ETQ usager, je veux que les erreurs et les descriptions soient reliées aux champs concernés

[inseo]

Déplace les descriptions contribuées à l'extérieur des étiquettes

Après

Avant

Précise le contexte du bouton de suppression d'une pièce jointe

Après

Avant

Uniformise le rendu des descriptions utilisant des balises HTML

Après

Avant

Applique les recommandations du DSFR concernant la liaison des messages d'erreur aux champs et aux regroupements

En cas de regroupement (balise fieldset)

• L'étiquette (le label), la description (si elle existe) et l'erreur (si elle est présente) sont liés au fieldset via l'attribut aria-labelledby.
• Les champs en erreur compris dans le regroupement ne sont pas liés à l'erreur ou à la description.
• Référence : Recommandation DSFR pour les regroupements.
  

En cas de champ simple (input, select, textarea)

• L'étiquette (le label) est lié au champ via le couple d'attribut for / id.
• La description (si elle existe) et l'erreur (si elle est présente) sont liés au champ via l'attribut aria-describedby
• Référence : Recommandation DSFR pour les champs simples.
  

[codecov]

Codecov Report

Attention: Patch coverage is 97.50000% with 1 line in your changes missing coverage. Please review.

Project coverage is 84.47%. Comparing base (6e1a3b7) to head (f329833).
Report is 50 commits behind head on main.

Files with missing lines	Patch %	Lines
app/components/dsfr/input_errorable.rb	90.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #10991      +/-   ##
==========================================
- Coverage   84.48%   84.47%   -0.02%     
==========================================
  Files        1187     1187              
  Lines       26265    26267       +2     
  Branches     4948     4953       +5     
==========================================
- Hits        22189    22188       -1     
- Misses       4076     4079       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[tchak]

je propose d'ajouter une méthode dans EditableChamp::EditableChampBaseComponent

def describedby
  "#{@champ.describedby_id} #{@champ.error_id}"
end

[colinux]

déjà, super cette grosse avancée pour le form usager !

quelques petites remarques, je ne sais pas lesquelles sont vraiment adressables

Aussi on avait fait un markup du champ "case à cocher", qui affichait correctement le cas champ obligatoire + texte sur plusieurs lignes car c'est un cas fréquent. Ici l'astérisque passe à la ligne :

[colinux]

c'est pas génant d'avoir des éléments référencés dans aria-describedby qui ne sont pas dans le DOM ? Je veux dire que s'il n'y a pas d'erreur, il n'y a pas cet élément, mais on le liste ici.

[LeSim]

Alors je crois que c'est bon, tu confirmes @inseo ?

[inseo]

Alors, le problème s'applique surtout aux champs qui n'ont pas de description, car sauf erreur de ma part, le bloc d'erreur est présent mais simplement vide lorsqu'il n'y a pas d'erreur.

Les tests que j'avais fait semblait dire que ça ne pose pas de problème.
Je vais encore juste faire une repasse de vérification avec TalkBack et NVDA pour être sûre.

[inseo]

Je confirme. 👍
J'ai vérifié avec les différentes technologies d'assistance et ça passe bien :

Quand il y a une description (blabla), mais pas d'erreur

nvda_description-sans-erreur.mp4

Quand il y a une erreur (RNF doit être rempli), mais pas de description

nvda_erreur-sans-description.mp4

[inseo]

@colinux : Merci pour la remarque sur le passage à la ligne de l'astérisque en cas de texte multiligne. J'étais passée à côté...
J'ai poussé un fix qui utilise une classe existante et ai supprimé le morceau de code custom ajouté à form.scss.

[colinux]

super et merci pour les précieux tests !