search

demarches-simplifiees / demarches-simplifiees.fr

Dématérialiser et simplifier les démarches administratives
https://www.demarches-simplifiees.fr
GNU Affero General Public License v3.0
190 stars 85 forks source link

ETQ usager: je ne peux ajouter qu'une unique PJ pour les anciennes procédures qui l'exigent #10511

Open Benoit-MINT opened 3 weeks ago

Benoit-MINT commented 3 weeks ago

En réponse à l'issue https://github.com/demarches-simplifiees/demarches-simplifiees.fr/issues/10186

Ceci concerne donc que les anciennes démarches où l'admin pouvait indiquer si une ou plusieurs PJ étaient autorisées.

Le pb venait de l'association entre le for dans la balise du label et l'id de l'input associé, qui rendait donc cliquable le label, permettant ainsi à l'usager de sélectionner une nouvelle PJ.

La modification proposée introduit une exception pour le type de champ PJ et lorsque les PJ multiples ne sont pas autorisées, avec pour conséquence de ne plus forcer la valeur de l'attribut for.

codecov[bot] commented 3 weeks ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 80.69%. Comparing base (bc6e2a4) to head (9086cb0). Report is 77 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #10511 +/- ## ========================================== - Coverage 81.08% 80.69% -0.40% ========================================== Files 1229 1234 +5 Lines 26011 26188 +177 Branches 4703 4718 +15 ========================================== + Hits 21092 21132 +40 - Misses 4919 5056 +137 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

colinux commented 3 weeks ago

Enlever l'attribut for, ça ne va pas le faire niveau accessibilité. Et l'importance de l'accessibilité est supérieure à l'impact de ce bug (on a eu qu'une remontée ou 2), sachant que ça ne concerne que d'anciennes démarches (publiées avant décembre 2022). Donc à moins de trouver une autre façon de le corriger (et pour pas trop "cher" vu le faible enjeu), je préfère que ça reste comme c'est

Benoit-MINT commented 1 week ago

L'autre logique proposée pour ne pas dégrader l'accessibilité est de jouer avec un attribut "disabled" sur l'input.

Pour info ici, contrairement à mon premier commentaire, ce bug se retrouve également pour les démarches actuelles (PJ multiples), en permettant d'insérer plus de 10 PJ (nb fixé arbitrairement dans le code).