Skip to content

docs(range), feat(range): corrige quelques coquilles dans la documentation et proposition d'améliorations#1407

Open
7studio wants to merge 5 commits intoGouvernementFR:mainfrom
7studio:docs/range
Open

docs(range), feat(range): corrige quelques coquilles dans la documentation et proposition d'améliorations#1407
7studio wants to merge 5 commits intoGouvernementFR:mainfrom
7studio:docs/range

Conversation

@7studio
Copy link
Copy Markdown
Contributor

@7studio 7studio commented Mar 24, 2026

Hello,

Voici une petite PR pour :

  1. rendre fonctionnel le contrôle Storybook "size" du composant Curseur.
    URL pour tester le dysfonctionnement actuel : https://www.systeme-de-design.gouv.fr/v1.14/storybook/index.html?path=/story/range--range-story&args=size:sm&globals=viewport:lg
  2. utiliser la liaison for/id comme préconisé dans la partie accessibilité du composant.
  3. corriger les exemples du curseur double.
  4. corriger coquille nommage dans l'exemple avec succès.

Je profite de cette PR pour proposer deux améliorations :

  1. cacher la valeur visible du curseur aux APIs d'accessibilité car celle-ci n'est pas vocalisée lors de sa mise à jour.
  2. privilégier l'utilisation d'un ensemble de champs via role="group au double aria-labelledby sur les éléments HTML <input>. Cela permet d’alléger la vocalisation et d'éviter de faire pointer le aria-labelledby sur son propre élément (je n'avais jamais vu ça avant).

@7studio 7studio changed the title docs(range): corrige le contrôle Storybook "size" docs(range), feat(range): corrige quelques coquilles dans la documentation et proposition d'améliorations Apr 8, 2026
@keryanS
Copy link
Copy Markdown
Contributor

keryanS commented Apr 14, 2026

Merci @7studio pour cette PR,

Je suis ok avec tous tes points, et ça m'a l'air top, sauf le dernier.
C'est certainement intéressant, mais sans prendre le temps de tester ça et de reboucler avec access42, cela me semble risqué, je ne préfère pas l'intégrer ici.

Si tu peux retirer ce changement, je pousserai la PR en 1.15

keryan

Comment thread src/dsfr/component/range/template/ejs/range.ejs Outdated
@7studio
Copy link
Copy Markdown
Contributor Author

7studio commented Apr 16, 2026

Hello @keryanS,

J'ai supprimé le dernier commit et ajouté la correction que tu as demandé sur la branche.

Je pousserais l'idée du groupe de champs dès que la 1.15 sera là 😉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants