Produits Entr'ouvert » Intégrations graphiques Publik

88b298e6e

Pou référence c'est #68359 (pas dans publik-base-theme mais dans authentic).

Perso je pense que oui ça peut être rangé dans le {% block %} (et que ça serait mieux).

Cela étant le balisage produit restera le même, et donc le problème de ce ticket présent. (le problème vient parce que fc-button-wrapper pose une hauteur de 60px, et le bouton prend cette hauteur, mais le fc-button-wrapper se fait prendre par .block h2 + div qui ajoute du padding; (en plus de la modif sur le {% block %}), je serais ici pour

 div#fc-button-wrapper {
        margin-top: 20px;
-       height: 60px;
 }

(aussi je ne capte pas vraiment l'alignement, avec le Qu'est-ce que centré mais le bouton pas centré, il doit y avoir autre chose à faire là)

Thomas Jund a écrit :

J'ai p-e ouvert une boite de pandor.

Plusieurs thèmes sont impactés depuis 88b298e6e, où "explanation.html" a été posé en dehors du {% block login %}
Par exemple celui de la CNIL.
Si pas de justif à avoir fait cela comme ça, je range l'include au sein du block.

Oui, ma faute, ça me paraissait logique que le texte d’explication ne fasse pas partie du bloc de login à proprement parler, mais j’ai pas réalisé que ça allait casser tous les thèmes où on ajoutait un titre à ce bloc. Mes excuses.

Le rétablissement du texte explicatif dans le bloc login est proposé dans #70386.

ok, ce qui règle déjà le bug, en replaçant le titre avant le texte.

mais j’ai pas réalisé que ça allait casser tous les thèmes où on ajoutait un titre à ce bloc

Notre front est un univers à la fois merveilleux et si fragile :p

Je vais essayer de clarifier les différents problèmes.
Trop de choses imbriquées, besoin de poser le contexte pour essayer de d'améliorer les choses.

Le bug apparait pour les thèmes qui surchargent "authentic2_auth_fc/login.html" pour lui ajouter un titre :

{% block login %}
  <h2>S’identifier avec FranceConnect</h2>
  {{ block.super }}
{% endblock %}

Le CSS présent dans _cell.scss l.95 s'active alors

.column .a2-block,
.block {
    & h2 + div,
        padding: 1rem;
    }
}

En ajoutant un padding supplémentaire qui vient s'ajouter à la div qui suit le titre (maintenant la div.fc-explanation-text, avant div.fc-bouton-wrapper).
Pourquoi ? Parceque .block doit hériter de l'apparence des cellules combo.
Lorque le padding s'ajoute au button-wrapper, la taille de la boite en est modifié (en cause la hauteur fixe pointé par Fred)

Mais on peut aussi remettre eu cause l'ajout d'un h2 personnalisé à cet endroit.

En effet, ce template est importé dans les pages /login et /register.
Et pas toujours évident de trouver un titre appoprié aux 2 contextes, sachant qu'en plus, sur /register un titre en dur est déjà ajouté 1 niveau au dessus, mais pas sur /login.
Il faut donc jouer du display:none sur les pages où l'on ne veut pas de ce titre personnalisé ajouté mais également pour masquer celui en dur qui ne convient pas.
Et après cela, se rend compte que le padding ajouté par `.block h2 > div` sera quand même appliqué, même si le titre est masqué. Mais uniquement sur /login et pas sur /register qui utilise la class `.column .a2-block`.

Donc très casse gueule de proposer une personnalisation du titre du block fc via "authentic2_auth_fc/login.html". Mais pas d'alternative en l'état.

Je remonte #53264 pour pourrait aider.

Après plusieurs tests, en jouant sur les variables cells-* et title-*, je conclue que surcharger "authentic2_auth_fc/login.html" pour ajouter un titre au bloc fc est simplement une très mauvaise idée.
Et donc, inutile d'ajouter des lignes de code au core pour patcher un cas de figure qui est une mauvaise chose.
Ce ticket a pointé qu'il y a un (gros ?) travail de mise à jour du markup des templates a2 pour les mettre en conformité avec le markup des cellules combo.
Mais que cela devra se faire après la validation de #53264 et #65803 qui offrent une solution pour permettre une personnalisation les titres des blocs de connexion et d'inscription.

#70386 règle le bug.
Je suis d'avis de fermer ce ticket.