antibes: fix search cell button not under textfield in mobile (#71841) #44

csechet · 2022-11-29T16:17:16+01:00

csechet commented

2022-11-29 16:17:16 +01:00

No description provided.

tjund commented

2022-12-13 10:08:45 +01:00

Je ne comprends pas vraiment ce ticket.
Il est noté

sous le champ en mobile

Et le code modifie le comportement du champ sur tous les viewport mais uniquement pour la sidebar.

Le ticket projet lié demande

Pour la cellule "recherche" en page d'accueil :

Passer le bouton "recherche" sous le champ de saisie

mettre le champ de saisie à la même taille que le bouton Recherche

Et c'est déjà le cas actuellement.

Ensuite par rapport à ce que fait le patch : donner au bouton et à l'input toute la largeur de la cellule lorsqu'elle est postionnée dasn la sidebar, il suffit simplement de

#sidebar div.searchcell {
	button {
		width: 100%;
	}
}

donner 100% au bouton

Je ne comprends pas vraiment ce ticket. Il est noté > sous le champ en mobile Et le code modifie le comportement du champ sur tous les viewport mais uniquement pour la sidebar. Le ticket projet lié demande > Pour la cellule "recherche" en page d'accueil : > * Passer le bouton "recherche" sous le champ de saisie > * mettre le champ de saisie à la même taille que le bouton Recherche Et c'est déjà le cas actuellement. Ensuite par rapport à ce que fait le patch : donner au bouton et à l'input toute la largeur de la cellule lorsqu'elle est postionnée dasn la sidebar, il suffit simplement de ```CSS #sidebar div.searchcell { button { width: 100%; } } ``` donner 100% au bouton

csechet force-pushed wip/71841-antibes-corrections-theme from e36e245bb9 to b28cf98001

2022-12-13 16:23:23 +01:00

Compare

csechet commented

2022-12-13 16:32:13 +01:00

Je ne comprends pas vraiment ce ticket.

C'est parce que tu ne parles pas l'Antibien.

Il est noté

sous le champ en mobile

Et le code modifie le comportement du champ sur tous les viewport mais uniquement pour la sidebar.

Oui, car la cellule recherche dont ils parlent est dans la sidebar.

Le ticket projet lié demande

Pour la cellule "recherche" en page d'accueil :

Passer le bouton "recherche" sous le champ de saisie

mettre le champ de saisie à la même taille que le bouton Recherche

Et c'est déjà le cas actuellement.

Pas dans les grandes résolutions en mobile, c'est ce qu'ils cherchent à exprimer quand ils disent "la zone de texte sur smartphone est trop petite. Le bouton peut-il aller à la ligne".

Ensuite par rapport à ce que fait le patch : donner au bouton et à l'input toute la largeur de la cellule lorsqu'elle est postionnée dasn la sidebar, il suffit simplement de
#sidebar div.searchcell {
	button {
		width: 100%;
	}
}
donner 100% au bouton

Ca ne change pas grand chose de mon point de vue, mais soit.

> Je ne comprends pas vraiment ce ticket. C'est parce que tu ne parles pas l'Antibien. > Il est noté > > > sous le champ en mobile > > Et le code modifie le comportement du champ sur tous les viewport mais uniquement pour la sidebar. Oui, car la cellule recherche dont ils parlent est dans la sidebar. > Le ticket projet lié demande > > > Pour la cellule "recherche" en page d'accueil : > > * Passer le bouton "recherche" sous le champ de saisie > > * mettre le champ de saisie à la même taille que le bouton Recherche > > Et c'est déjà le cas actuellement. Pas dans les grandes résolutions en mobile, c'est ce qu'ils cherchent à exprimer quand ils disent "la zone de texte sur smartphone est trop petite. Le bouton peut-il aller à la ligne". > Ensuite par rapport à ce que fait le patch : donner au bouton et à l'input toute la largeur de la cellule lorsqu'elle est postionnée dasn la sidebar, il suffit simplement de > > ```CSS > #sidebar div.searchcell { > button { > width: 100%; > } > } > ``` > donner 100% au bouton > Ca ne change pas grand chose de mon point de vue, mais soit.

tjund commented

2022-12-14 09:47:05 +01:00

C'est parce que tu ne parles pas l'Antibien.

Merci pour les explications 👍

Ca ne change pas grand chose de mon point de vue, mais soit.

Ce que je propose ce n'est pas de remplacer la prop flex par width mais de tout virer et de ne garder que width sur le bouton.

  #sidebar div.searchcell {
-	form {
-		display: flex;
-		flex-wrap: wrap;
-	}
-
	button {
-		flex-grow: 1;
-		margin-right: 1rem;
+		width: 100%;
	}
  }

> C'est parce que tu ne parles pas l'Antibien. Merci pour les explications 👍 > Ca ne change pas grand chose de mon point de vue, mais soit. Ce que je propose ce n'est pas de remplacer la prop flex par width mais de tout virer et de ne garder que width sur le bouton. ``` diff #sidebar div.searchcell { - form { - display: flex; - flex-wrap: wrap; - } - button { - flex-grow: 1; - margin-right: 1rem; + width: 100%; } } ```

csechet commented

2022-12-14 10:52:22 +01:00

Ce que je propose ce n'est pas de remplacer la prop flex par width mais de tout virer et de ne garder que width sur le bouton.

Désolé, j'avais mal compris. Je fais ça.

> Ce que je propose ce n'est pas de remplacer la prop flex par width mais de tout virer et de ne garder que width sur le bouton. Désolé, j'avais mal compris. Je fais ça.

csechet force-pushed wip/71841-antibes-corrections-theme from b28cf98001 to 77fa8da51c

2022-12-14 10:54:34 +01:00

Compare

tjund approved these changes 2022-12-14 11:23:28 +01:00

csechet merged commit 0e7ad03977 into main

2022-12-14 12:13:12 +01:00

Sign in to join this conversation.

No reviewers

No Label

No Milestone

No Assignees

2 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/publik-base-theme#44