Produits Entr'ouvert » Authentic 2

J’avoue ne pas comprendre la convention utilisée sur le nommage de cette nouvelle classe. Instinctivement j’aurais plutôt vu un "submit-button--{{ submit_name }}" (plutôt que le double tiret est entre submit et button lesquels sont à la fin du nom de classe). Il y a une raison pour ce nommage ?

Supposément la logique serait d'avoir nom-du-composant, posé en classe à la racine (donc ici sur le <form>), par exemple class="auth-saml-login-form", puis dans le contenu, sur le bouton en question, auth-saml-login-form--submit-button.

À propos de {{submit_name}} dans le nom, j'imagine que c'est pour pouvoir différencier deux blocs, ça irait pour moi alors plutôt à la racine, mais je trouve un peu redondante sa constitution 'login-%s-%s' % (self.id, instance_id) (qui va produire login-saml-quelquechose). Je n'ai pas d'avis tranché ici, peut-être que ça pourrait aller sur le <form>, en data-instance-id="{{...}}" ?

Paul Marillonnet a écrit :

J’avoue ne pas comprendre la convention utilisée sur le nommage de cette nouvelle classe. Instinctivement j’aurais plutôt vu un "submit-button--{{ submit_name }}" (plutôt que le double tiret est entre submit et button lesquels sont à la fin du nom de classe). Il y a une raison pour ce nommage ?

Je me suis emmelé les pinceaux avec le nom de la classe posé sur le div contenant le bloc: block-saml_<instance_id> et je voulais suivre le nommage en suffixant avec --submit-button, mais c'est pas bon.

Je préfère ne pas toucher à la classe submit-button au cas où il y aurait des intégrations qui se basent dessus pour styler le bouton (à priori pas dans publik-base-theme, mais peut-être ailleurs). Plutôt rajouter une nouvelle classe en suivant ta proposition.

Serghei Mihai a écrit :

Je préfère ne pas toucher à la classe submit-button au cas où il y aurait des intégrations qui se basent dessus pour styler le bouton (à priori pas dans publik-base-theme, mais peut-être ailleurs). Plutôt rajouter une nouvelle classe en suivant ta proposition.

Tu peux laisser les classes "legacy" et ajouter des classe conformes à la nouvelle politique de nommage, ce n'est pas bien grave je pense et ça permet d'avancer.

Sinon ce qui serait bien ce serait d'avoir quelque chose d'un peu uniforme au niveau des classes entre OIDC, SAML (à base de auth-form, auth-form--submit-button, auth-form--type-saml/oidc/fc, auth-form--title, auth-form--description pour les petits textes qu'on pourra afficher etc...).