Development #65864
auth_saml: poser une classe explicite sur le bouton de soumission
0%
Description
Pour faciliter la personnalisation dans les thèmes.
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Serghei Mihai il y a presque 2 ans
- Fichier 0001-auth_saml-add-explicit-class-for-submit-button-65864.patch 0001-auth_saml-add-explicit-class-for-submit-button-65864.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Serghei Mihai il y a presque 2 ans
- Lié à Development #64671: alpes-maritimes-2018: intégrer le bouton et le template de connexion SAML ajouté
Mis à jour par Paul Marillonnet il y a presque 2 ans
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 ?
Mis à jour par Frédéric Péters il y a presque 2 ans
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="{{...}}" ?
Mis à jour par Serghei Mihai il y a presque 2 ans
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 entresubmit
etbutton
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.
Mis à jour par Serghei Mihai il y a presque 2 ans
Mis à jour par Benjamin Dauvergne il y a presque 2 ans
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...).
Mis à jour par Benjamin Dauvergne il y a presque 2 ans
- Statut changé de Solution proposée à En cours
- Assigné à mis à Serghei Mihai
Mis à jour par Robot Gitea il y a 12 mois
- Statut changé de En cours à Solution proposée
Serghei Mihai (smihai) a ouvert une pull request sur Gitea concernant cette demande :
- URL : https://git.entrouvert.org/entrouvert/authentic/pulls/49
- Titre : auth_saml: add explicit class for submit button (#65864)
- Modifications : https://git.entrouvert.org/entrouvert/authentic/pulls/49/files
Mis à jour par Robot Gitea il y a 12 mois
- Statut changé de Solution proposée à Solution validée
Benjamin Dauvergne (bdauvergne) a approuvé une pull request sur Gitea concernant cette demande :
Mis à jour par Robot Gitea il y a 12 mois
- Statut changé de Solution validée à Résolu (à déployer)
Serghei Mihai (smihai) a mergé une pull request sur Gitea concernant cette demande :
- URL : https://git.entrouvert.org/entrouvert/authentic/pulls/49
- Titre : auth_saml: add explicit class for submit button (#65864)
- Modifications : https://git.entrouvert.org/entrouvert/authentic/pulls/49/files
Mis à jour par Transition automatique il y a 12 mois
- Statut changé de Résolu (à déployer) à Solution déployée
auth_saml: add explicit class for submit button (#65864)