Re: [SPIP][Notifications/Commit Core] [spip ↪ 4.0] Un selecteur CSS + restrictif car sinon cela impacte (...)

il me semble que formidable utilise en l’occurence uniquement la norme
des formulaires de spip, non ?

Un selecteur CSS + restrictif car sinon cela impacte par exemple tous
les .label d’un formidable dans le texte d’un article (mais aussi, c’est
peut-être une erreur de formidable d’utiliser un nommage de classe si
generique a cet endroit)

/Modifié/
prive/themes/spip/content.css.html

Détails :

Il n’y a pas de class .label sur les label dans la charte

(c’est un peu redondant…)


Cédric
Le 12 juil. 2021 à 14:10 +0200, Maïeul Rouquette via Discuter de SPIP noreply@discuter.spip.net, a écrit :

Maïeul Rouquette maieul
Juillet 12
il me semble que formidable utilise en l’occurence uniquement la norme
des formulaires de spip, non ?

Un selecteur CSS + restrictif car sinon cela impacte par exemple tous
les .label d’un formidable dans le texte d’un article (mais aussi, c’est
peut-être une erreur de formidable d’utiliser un nommage de classe si
generique a cet endroit)
/Modifié/
prive/themes/spip/content.css.html
Détails :
Voir le sujet ou répondre à ce courriel pour répondre.
Vous recevez ce courriel car vous avez activé la liste de diffusion.
Pour se désabonner de ces courriels, cliquez ici.

Elle est utilisée sur les légendes des fieldset.editer, et je crois que saisies/formidable en ont déduit qu’on pouvait l’utiliser sur tous les labels.

Du coup sa présence ailleurs a dû être prise en compte dans les CSS des formulaires, mais un peu sous la pression :stuck_out_tongue:

Mais c’est pas plus mal que ça soit plus restrictif dans#wysiwyg de toute façon.

ah oui, c’est aussi que cela permettait de simplifier d’autres css par ailleurs, si mes souvenirs sont bons. C’est @nicod je crois qui a fait ca

Mais si seul Saisies fait ça, et que c’est pas une norme du core, bah ça permet pas vraiment de simplifier des CSS puisque les forms non faits en Saisies on doit toujours styler le vrai label non ? Donc ça n’aide pas grand chose.

Par ailleurs c’est pas juste redondant, mais ça peut casser de l’existant. Par exemple j’ai un site qui a Bootstrap 3, ce dernier ayant un composant CSS « .label » (notamment avec le texte en blanc, inline, etc) : après mise à jour, tous les forms fait avec Saisies ont alors tous les labels invisibles à cause de cette classe (+ taille réduite et plein d’autres merde).

En fait cette modif a bien cassé des sites en prod (tous les sites utilisant Bootstrap 3 déjà). :frowning:

Ça aurait plutôt dû être « editer__label » comme classe à vrai dire, pour ne pas avoir un nom trop générique, et ça marche très bien autant pour les champs normaux que pour les fieldset.editer (qui sont dont .editer aussi, donc avec un .editer__label dedans).

À changer pour 4.1 ?

1 « J'aime »