[FO - Formulaire - Accessibilité] Revue des liens

[emilschn]

Ticket

#2546

Description

Revue des liens dans le formulaire de signalement

Changements apportés

• ajustement des title quand on ouvre une nouvelle fenêtre
• remplacement de composants liens par des composants boutons quand on ouvre une modal

Tests

• Vérifier les différents liens des différents parcours
• Vérifier le fonctionnement des boutons En savoir plus

[hmeneuvrier]

Le bouton "En savoir plus" ouvre bien la modale, mais je pense qu'il y a un peu de css à passer.

En prod :

Sur ta branche :

[emilschn]

Le bouton "En savoir plus" ouvre bien la modale, mais je pense qu'il y a un peu de css à passer.

En prod :

Sur ta branche :

Ah oui, ça j'ai vu... mais c'est pas possible d'avoir un bouton qui a le style d'un badge.
Donc c'est un bouton qui a le style d'un bouton et l'indicateur d'info.
Tu penses que c'est trop problématique ?

[numew]

Mêmes remarques que héléne, plus les lien d'édition à revoir je pense (pas de title) a moins que c soit couvert pas un autre cas ?

[numew]

Le bouton "En savoir plus" ouvre bien la modale, mais je pense qu'il y a un peu de css à passer.
En prod :

Sur ta branche :

Ah oui, ça j'ai vu... mais c'est pas possible d'avoir un bouton qui a le style d'un badge. Donc c'est un bouton qui a le style d'un bouton et l'indicateur d'info. Tu penses que c'est trop problématique ?

Ca s'affiche comme un badge simplement si on retire la classe "ft-btn" à première vue

[hmeneuvrier]

Ah oui, ça j'ai vu... mais c'est pas possible d'avoir un bouton qui a le style d'un badge.
Donc c'est un bouton qui a le style d'un bouton et l'indicateur d'info.
Tu penses que c'est trop problématique ?

Je trouve que ça prend beaucoup de place... peut-être au moins en plus petit, en fr-btn--sm ?

[emilschn]

Le bouton "En savoir plus" ouvre bien la modale, mais je pense qu'il y a un peu de css à passer.
En prod :

Sur ta branche :

Ah oui, ça j'ai vu... mais c'est pas possible d'avoir un bouton qui a le style d'un badge. Donc c'est un bouton qui a le style d'un bouton et l'indicateur d'info. Tu penses que c'est trop problématique ?

Ca s'affiche comme un badge simplement si on retire la classe "ft-btn" à première vue

Ah oui tiens ; je suis sûr d'avoir testé l'autre fois sans succès. Je vais faire ça ! Merci !

[edit] Ah non, c'est surtout que fr-button est déjà dans le composant Bouton, par défaut...

[emilschn]

@hmeneuvrier @numew
J'ai mis le bouton En savoir plus en fr-btn--sm comme préconisé par Hélène (le badge, c'était compliqué structurellement, sans changer plein de trucs)
J'ai modifié les boutons du breadcrumb et les boutons d'édition en fin de parcours

[sonarcloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

[hmeneuvrier]

Relecture et tests OK pour moi.
Le fr-btn--sm ça me suffit. Mathilde verra si elle préfère passer sur un fr-btn--secondary ou pas

[numew]

Ok (c'est un peu filou d'intervertir lien et boutons pour s'en sortir mais ca me va)