La classe "label" pour les legend des "faux" fieldsets est trop générique #4857

Open
opened 2 months ago by rastapopoulos · 2 comments
Owner

Lors de l'introduction des fieldset.editer (fieldset ayant la forme d'un champ normal où la légende remplace le label), on a alors ajouté la classe "label" sur le legend.

C'est un terme en fait trop générique pour une classe, et cela provoque des conflits avec de l'existant. Par exemple Bootstrap 1, 2, et 3 ont un composant de ce nom, et je l'ai vu ailleurs aussi (mais ça implique déjà beaucoup d'utilisations rien qu'avec Bootstrap). Les éléments deviennent alors illisibles (blanc sur blanc), tous petits, etc.

D'autant plus que cela a donné un mauvais exemple, car au départ uniquement pour les fieldsets, ça a été ajouté (par cohérence ?) à tous les vrais labels (label.label) dans Saisies. Donc encore bien plus d'éléments pétés avec cette classe.

Le mieux serait de dire plus précisément "c'est un label de notre composant .editer", et donc que la classe soit plutôt "editer__label" (par exemple, ou "editer-label" mais pas "label" tout court).

Lors de l'introduction des fieldset.editer (fieldset ayant la forme d'un champ normal où la légende remplace le label), on a alors ajouté la classe "label" sur le legend. C'est un terme en fait trop générique pour une classe, et cela provoque des conflits avec de l'existant. Par exemple Bootstrap 1, 2, et 3 ont un composant de ce nom, et je l'ai vu ailleurs aussi (mais ça implique déjà beaucoup d'utilisations rien qu'avec Bootstrap). Les éléments deviennent alors illisibles (blanc sur blanc), tous petits, etc. D'autant plus que cela a donné un mauvais exemple, car au départ uniquement pour les fieldsets, ça a été ajouté (par cohérence ?) à tous les vrais labels (label.label) dans Saisies. Donc encore bien plus d'éléments pétés avec cette classe. Le mieux serait de dire plus précisément "c'est un label de notre composant .editer", et donc que la classe soit plutôt "editer__label" (par exemple, ou "editer-label" mais pas "label" tout court).
b_b added this to the 4.1 milestone 1 month ago
Owner

+1 pour editer__label en ce qui me concerne.

Quitte à ce qu'il y ait un mélange de règles (c'est déjà le cas avec les tirets et les underscores), autant avoir un peu de BEM dans le mix. Et petit à petit on pourrait tirer vers ce dernier.

+1 pour `editer__label` en ce qui me concerne. Quitte à ce qu'il y ait un mélange de règles (c'est déjà le cas avec les tirets et les underscores), autant avoir un peu de BEM dans le mix. Et petit à petit on pourrait tirer vers ce dernier.
Poster
Owner

Je n'ai pas d'avis tranché, mais pour aller vite, gardons plutôt le "editer-label" pour l'instant avec d'être coéhrent avec les autres trucs de forms "editer-groupe" etc.

Je n'ai pas d'avis tranché, mais pour aller vite, gardons plutôt le "editer-label" pour l'instant avec d'être coéhrent avec les autres trucs de forms "editer-groupe" etc.
Sign in to join this conversation.
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.