https://dev.entrouvert.org/https://dev.entrouvert.org/favicon.ico?15861920342022-06-01T12:07:05ZRedmine Entr’ouvertAuthentic 2 - Development #65864: auth_saml: poser une classe explicite sur le bouton de soumissionhttps://dev.entrouvert.org/issues/65864?journal_id=3842582022-06-01T12:07:05ZSerghei Mihai
<ul><li><strong>Fichier</strong> <a href="/attachments/66329">0001-auth_saml-add-explicit-class-for-submit-button-65864.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/66329/0001-auth_saml-add-explicit-class-for-submit-button-65864.patch">0001-auth_saml-add-explicit-class-for-submit-button-65864.patch</a> ajouté</li><li><strong>Statut</strong> changé de <i>Nouveau</i> à <i>Solution proposée</i></li><li><strong>Patch proposed</strong> changé de <i>Non</i> à <i>Oui</i></li></ul> Authentic 2 - Development #65864: auth_saml: poser une classe explicite sur le bouton de soumissionhttps://dev.entrouvert.org/issues/65864?journal_id=3842602022-06-01T12:07:36ZSerghei Mihai
<ul><li><strong>Lié à</strong> <i><a class="issue tracker-2 status-5 priority-4 priority-default closed" href="/issues/64671">Development #64671</a>: alpes-maritimes-2018: intégrer le bouton et le template de connexion SAML</i> ajouté</li></ul> Authentic 2 - Development #65864: auth_saml: poser une classe explicite sur le bouton de soumissionhttps://dev.entrouvert.org/issues/65864?journal_id=3846492022-06-02T13:10:27ZPaul Marillonnet
<ul></ul><p>J’avoue ne pas comprendre la convention utilisée sur le nommage de cette nouvelle classe. Instinctivement j’aurais plutôt vu un <code>"submit-button--{{ submit_name }}"</code> (plutôt que le double tiret est entre <code>submit</code> et <code>button</code> lesquels sont à la fin du nom de classe). Il y a une raison pour ce nommage ?</p> Authentic 2 - Development #65864: auth_saml: poser une classe explicite sur le bouton de soumissionhttps://dev.entrouvert.org/issues/65864?journal_id=3846572022-06-02T13:21:25ZFrédéric Pétersfpeters@entrouvert.com
<ul></ul><p>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.</p>
<p>À 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 <code>'login-%s-%s' % (self.id, instance_id)</code> (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="{{...}}" ?</p> Authentic 2 - Development #65864: auth_saml: poser une classe explicite sur le bouton de soumissionhttps://dev.entrouvert.org/issues/65864?journal_id=3846912022-06-02T13:58:41ZSerghei Mihai
<ul></ul><p>Paul Marillonnet a écrit :</p>
<blockquote>
<p>J’avoue ne pas comprendre la convention utilisée sur le nommage de cette nouvelle classe. Instinctivement j’aurais plutôt vu un <code>"submit-button--{{ submit_name }}"</code> (plutôt que le double tiret est entre <code>submit</code> et <code>button</code> lesquels sont à la fin du nom de classe). Il y a une raison pour ce nommage ?</p>
</blockquote>
<p>Je me suis emmelé les pinceaux avec le nom de la classe posé sur le div contenant le bloc: <code>block-saml_<instance_id></code> et je voulais suivre le nommage en suffixant avec <code>--submit-button</code>, mais c'est pas bon.</p>
<p>Je préfère ne pas toucher à la classe <code>submit-button</code> 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.</p> Authentic 2 - Development #65864: auth_saml: poser une classe explicite sur le bouton de soumissionhttps://dev.entrouvert.org/issues/65864?journal_id=3846922022-06-02T13:59:29ZSerghei Mihai
<ul><li><strong>Fichier</strong> <a href="/attachments/66380">0001-auth_saml-add-explicit-class-for-submit-button-65864.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/66380/0001-auth_saml-add-explicit-class-for-submit-button-65864.patch">0001-auth_saml-add-explicit-class-for-submit-button-65864.patch</a> ajouté</li></ul> Authentic 2 - Development #65864: auth_saml: poser une classe explicite sur le bouton de soumissionhttps://dev.entrouvert.org/issues/65864?journal_id=3847752022-06-02T16:32:09ZBenjamin Dauvergne
<ul></ul><p>Serghei Mihai a écrit :</p>
<blockquote>
<p>Je préfère ne pas toucher à la classe <code>submit-button</code> 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.</p>
</blockquote>
<p>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.</p>
<p>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...).</p> Authentic 2 - Development #65864: auth_saml: poser une classe explicite sur le bouton de soumissionhttps://dev.entrouvert.org/issues/65864?journal_id=3969052022-07-25T09:59:14ZBenjamin Dauvergne
<ul><li><strong>Statut</strong> changé de <i>Solution proposée</i> à <i>En cours</i></li><li><strong>Assigné à</strong> mis à <i>Serghei Mihai</i></li></ul> Authentic 2 - Development #65864: auth_saml: poser une classe explicite sur le bouton de soumissionhttps://dev.entrouvert.org/issues/65864?journal_id=4618922023-05-02T08:52:28ZRobot Gitea
<ul><li><strong>Statut</strong> changé de <i>En cours</i> à <i>Solution proposée</i></li></ul><p>Serghei Mihai (smihai) a ouvert une pull request sur Gitea concernant cette demande :</p>
<ul>
<li>URL : <a class="external" href="https://git.entrouvert.org/entrouvert/authentic/pulls/49">https://git.entrouvert.org/entrouvert/authentic/pulls/49</a></li>
<li>Titre : auth_saml: add explicit class for submit button (<a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Development: auth_saml: poser une classe explicite sur le bouton de soumission (Fermé)" href="https://dev.entrouvert.org/issues/65864">#65864</a>)</li>
<li>Modifications : <a class="external" href="https://git.entrouvert.org/entrouvert/authentic/pulls/49/files">https://git.entrouvert.org/entrouvert/authentic/pulls/49/files</a></li>
</ul> Authentic 2 - Development #65864: auth_saml: poser une classe explicite sur le bouton de soumissionhttps://dev.entrouvert.org/issues/65864?journal_id=4619852023-05-02T11:41:17ZRobot Gitea
<ul><li><strong>Statut</strong> changé de <i>Solution proposée</i> à <i>Solution validée</i></li></ul><p>Benjamin Dauvergne (bdauvergne) a approuvé une pull request sur Gitea concernant cette demande :</p>
<ul>
<li>URL : <a class="external" href="https://git.entrouvert.org/entrouvert/authentic/pulls/49">https://git.entrouvert.org/entrouvert/authentic/pulls/49</a></li>
</ul> Authentic 2 - Development #65864: auth_saml: poser une classe explicite sur le bouton de soumissionhttps://dev.entrouvert.org/issues/65864?journal_id=4619872023-05-02T11:43:08ZRobot Gitea
<ul><li><strong>Statut</strong> changé de <i>Solution validée</i> à <i>Résolu (à déployer)</i></li></ul><p>Serghei Mihai (smihai) a mergé une pull request sur Gitea concernant cette demande :</p>
<ul>
<li>URL : <a class="external" href="https://git.entrouvert.org/entrouvert/authentic/pulls/49">https://git.entrouvert.org/entrouvert/authentic/pulls/49</a></li>
<li>Titre : auth_saml: add explicit class for submit button (<a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Development: auth_saml: poser une classe explicite sur le bouton de soumission (Fermé)" href="https://dev.entrouvert.org/issues/65864">#65864</a>)</li>
<li>Modifications : <a class="external" href="https://git.entrouvert.org/entrouvert/authentic/pulls/49/files">https://git.entrouvert.org/entrouvert/authentic/pulls/49/files</a></li>
</ul> Authentic 2 - Development #65864: auth_saml: poser une classe explicite sur le bouton de soumissionhttps://dev.entrouvert.org/issues/65864?journal_id=4626622023-05-04T15:14:31ZTransition automatique
<ul><li><strong>Statut</strong> changé de <i>Résolu (à déployer)</i> à <i>Solution déployée</i></li></ul> Authentic 2 - Development #65864: auth_saml: poser une classe explicite sur le bouton de soumissionhttps://dev.entrouvert.org/issues/65864?journal_id=4780992023-07-09T02:42:02ZTransition automatique
<ul></ul><p>Automatic expiration</p>