DISIC / observatoire

🕵️‍♀️ Observatoire de la qualité des démarches en ligne
https://observatoire.numerique.gouv.fr/
GNU Lesser General Public License v3.0
10 stars 4 forks source link

Textes explicatifs sur les pages d'inscription et connexion à implémenter avec 2 paragraphes #1144

Closed lucaa closed 2 years ago

lucaa commented 2 years ago

L'espacement entre les 2 doit être donné par les marges des paragraphes, standard DSFR (1.5 rem).

à explorer la possibilité de changer la marge de tous les éléments de contenu (paragraphes, ul, etc) XWiki / bootstrap à cette marge DSFR, pour avoir le look & feel DSFR appliqué partout, notamment sur les pages explicatives (CGU, etc.).

Il y a des risques des effets de bord de cela (pour tous les endroits ou la marge est celle par défaut et la nouvelle est trop grande), mais globalement ça vaut la peine d'essayer, vu que en général la marge est contrôlée par du CSS spécifique si ce n'est pas un flux de contenu par défaut.

clemdee commented 2 years ago

Les pages de login et de registration ont été mise à jour afin d'utiliser deux paragraphes au lieu de balises <br />. La marge inférieur des paragraphes a été implémentée à 1.5rem globalement dans le DSFRTheme, après vérification des pages principales et autres pages types, je n'ai pas constaté de problème d'espacement majeur. À l'inverse, les paragraphes des pages de contenu du wiki sont maintenant correctement espacé.

Il faudra tout de même vérifier plus en profondeur afin de s'assurer que rien n'a été cassé.

Concernant les règles DSFR liées au espacements des paragraphes, la logique est très simple et la marge de 1.5rem est présente partout, sauf dans de rares cas particuliers à l'intérieur de composants dsfr. Un cas qui pourrait nous concerner est la présence de <p> à l'intérieur de boites d'alerte, où celle-ci est mise à 0. Cependant puisque nous utilisons le markup bootstrap, cela ne nous concerne pas directement, mais pourrait dans le cas où l'on voudrait utiliser un paragraphe à l'intérieur d'un alert. Il n'y pour l'instant aucun cas comme ça, je laisse donc ça de coté et ça sera implémenté si jamais le besoin s'en fait.

lucaa commented 2 years ago

Cependant puisque nous utilisons le markup bootstrap, cela ne nous concerne pas directement, mais pourrait dans le cas où l'on voudrait utiliser un paragraphe à l'intérieur d'un alert. Il n'y pour l'instant aucun cas comme ça, je laisse donc ça de coté et ça sera implémenté si jamais le besoin s'en fait.

(j'espère que je comprends ça correctement): en fait, il y a des paragraphes tout le temps dans les alertes, dès qu'on utilise plus qu'un seul paragraphe. Mais la plateforme XWiki corrige cela https://jira.xwiki.org/browse/XWIKI-13947 , on devrait être safe pour les alertes...

ugodtn commented 2 years ago

@lucaa Ok pour la tâche décrite. Ceci étant dit, il y a toujours des différences de mise en page entre connexion et inscription : le titre ne se retrouve pas à la même hauteur.

lucaa commented 2 years ago

@ugodtn tu penses à #1143 ? C'est censé être corrigé et prêt à tester en staging. Tu peux décrire plus en détail le problème que tu vois? (avec capture?)

ugodtn commented 2 years ago

Ah oui parfait merci, je vais faire mes retours sur cette tâche alors !