Project

General

Profile

Development #65864

auth_saml: poser une classe explicite sur le bouton de soumission

Added by Serghei Mihai 8 months ago. Updated 6 months ago.

Status:
En cours
Priority:
Normal
Assignee:
Category:
-
Target version:
-
Start date:
01 June 2022
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

Description

Pour faciliter la personnalisation dans les thèmes.


Files


Related issues

Related to Intégrations graphiques Publik - Development #64671: alpes-maritimes-2018: intégrer le bouton et le template de connexion SAMLFermé28 April 2022

Actions

History

#1

Updated by Serghei Mihai 8 months ago

#2

Updated by Serghei Mihai 8 months ago

  • Related to Development #64671: alpes-maritimes-2018: intégrer le bouton et le template de connexion SAML added
#3

Updated by Paul Marillonnet 8 months ago

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 ?

#4

Updated by Frédéric Péters 8 months ago

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="{{...}}" ?

#5

Updated by Serghei Mihai 8 months ago

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.

#7

Updated by Benjamin Dauvergne 8 months ago

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...).

#8

Updated by Benjamin Dauvergne 6 months ago

  • Status changed from Solution proposée to En cours
  • Assignee set to Serghei Mihai

Also available in: Atom PDF