Development #55297
templates: attribuer un identifiant au bloc contenant le libellé du champ de formulaire
0%
Description
Cela permet d'expliciter aux lecteurs d'écran que c'est le libellé du champ de formulaire.
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Serghei Mihai il y a presque 3 ans
- Fichier 0001-templates-set-id-for-widget-labels-55297.patch 0001-templates-set-id-for-widget-labels-55297.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Serghei Mihai il y a presque 3 ans
- Lié à Development #54642: avoir une indication permettant de déterminer l'appartenance d'un bouton radio à un groupe ajouté
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a presque 3 ans
J'ai hésité sur le fait de mettre l'id sur le wrapper `div.title` à la place de le mettre directement sur `label`.
Mais ça semble fonctionner avec `aria-labelledby`, la valeur aria sera alors `text label (optional)`
J'ai encore un doute sur le choix de {{ field.name }} comme id. Est-ce que {{field.name}} a la garanti d'être unique comparativement à {{field.id_for_label}} ?
Et enfin, le label du titre conserve son attribut "for", ce qui ne me semble pas opportun dans le cas d'un radiogroup.
Mis à jour par Serghei Mihai il y a presque 3 ans
Thomas Jund a écrit :
J'ai hésité sur le fait de mettre l'id sur le wrapper `div.title` à la place de le mettre directement sur `label`.
Mais ça semble fonctionner avec `aria-labelledby`, la valeur aria sera alors `text label (optional)`
Oui, je souhaite avoir comme résultat `text label (optional)`.
J'ai encore un doute sur le choix de {{ field.name }} comme id. Est-ce que {{field.name}} a la garanti d'être unique comparativement à {{field.id_for_label}} ?
Non, comme on n'a pas de garantie que {{field.id_for_label}} est unique. id_for_label est la combinaison de "id_" + field.label.
Sauf à avoir plusieurs fois le même formulaire sur une page, les id resteront uniques.
Et enfin, le label du titre conserve son attribut "for", ce qui ne me semble pas opportun dans le cas d'un radiogroup.
On ne sait pas au niveau du template s'il s'agit d'une "liste" avec rendu en boutons radio, surtout que le libellé est généré par django: {{ field.label_tag }}.
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a presque 3 ans
Oui, je souhaite avoir comme résultat `text label (optional)`.
ah ben nickel :)
le libellé est généré par django: {{ field.label_tag }}
On est pas obligé de l'utilisé.
On ne sait pas au niveau du template s'il s'agit d'une "liste" avec rendu en boutons radio
ok, je pensais qu'il y avait peut-être quelque chose du côté de {{ field.template_name }}
Mis à jour par Serghei Mihai il y a presque 3 ans
Thomas Jund a écrit :
le libellé est généré par django: {{ field.label_tag }}
On est pas obligé de l'utilisé.
En effet, mais ça fera l'objet d'un autre ticket.
Mis à jour par Thomas Jund (congés, retour le 29/04) il y a presque 3 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Serghei Mihai il y a presque 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
- Assigné à mis à Serghei Mihai
commit 7ceef600aee1a85550559d4f0d149e76ed12c4ce (origin/main)
Author: Serghei Mihai <smihai@entrouvert.com>
Date: Thu Jul 1 11:59:49 2021 +0200
templates: set id for widget labels (#55297)
Mis à jour par Serghei Mihai il y a presque 3 ans
Serghei Mihai a écrit :
En effet, mais ça fera l'objet d'un autre ticket.
En m'appretant à faire le ticket je me rends compte que for est utile notamment pour les cases à cocher uniques: cliquer sur le libellé permet de cocher ou décocher la case.
Mis à jour par Frédéric Péters il y a presque 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
templates: set id for widget labels (#55297)