Development #82789
Questions markup radiogroup et checkboxesgroup
0%
Description
Je me penche sur #81487 et besoin d'avis.
L'origine est un constat graphique que je souhaite corriger : les inputs de nos formulaires ont un petit margin top et bottom. Une légère séparation avec le label et la description du widget (cf capture).
Mais pas les radios et checkboxes: pas d'espace entre le label d'un widget et le premier label d'un bouton radio et checkbox.
J'en arrive donc à :
1- Le markup des templates radiosGroup et des checkboxesGroup différent, une ul > li pour les checkboxes et une suite de labels pour les radios. Y'a-t-il un intérêt à garder 2 markups différents ou non, simplifier pour uniformiser ?
2- Le markup du group ne semble pas correct avec une description :
<div class="content " role="radiogroup" aria-describedby="form_label_f18"> <div id="form_hint_f18" class="hint"><p>Remarque pour un radio group</p></div> <label for="f18_op_0"><input id="f18_op_0" aria-describedby="form_hint_f18" type="radio" name="f18" value="choix 1"><span>choix 1</span></label> ...
La description (hint) se trouve au sein du group mais n'en fait pas partie. Je pense à quelque chose comme
<div class="content"> <div id="form_hint_f18" class="hint"><p>Remarque pour un radio group</p></div> <div class="radiogroup" role="radiogroup" aria-describedby="form_label_f18"> <label for="f18_op_0"><input id="f18_op_0" aria-describedby="form_hint_f18" type="radio" name="f18" value="choix 1"><span>choix 1</span></label> ... </div>
3. Pour y arriver il faut déplacer les `widget.content_extra_attributes.items` sur le nouveau tag ajouté au sein du {% block widget-control %}. Il me semble que seul RadiobuttonsWidget et CheckboxesWidget utilisent cette option mais je veux bien une confirmation
Fichiers
Demandes liées
Révisions associées
scss: adapt for list markup in radio buttons (#82789)
Historique
Mis à jour par Frédéric Péters il y a 6 mois
1- Le markup des templates radiosGroup et des checkboxesGroup différent, une ul > li pour les checkboxes et une suite de labels pour les radios. Y'a-t-il un intérêt à garder 2 markups différents ou non, simplifier pour uniformiser ?
Il me semble que ça pourrait systématiquement être une liste.
2- Le markup du group ne semble pas correct avec une description : (...)
Pas d'opinion particulière ici.
3. Pour y arriver il faut déplacer les `widget.content_extra_attributes.items` sur le nouveau tag ajouté au sein du {% block widget-control %}. Il me semble que seul RadiobuttonsWidget et CheckboxesWidget utilisent cette option mais je veux bien une confirmation
je ne confirme pas, il y a également
class CompositeWidget(quixote.form.CompositeWidget): content_extra_attributes = {'role': 'group'}
et
class ConditionWidget(CompositeWidget): (...) @property def content_extra_attributes(self):
Mis à jour par Frédéric Péters il y a 6 mois
- Tracker changé de Development à Autre
- Sujet changé de Questiosn markup radiogroup et checkboxesgroup à Questions markup radiogroup et checkboxesgroup
Mis à jour par Robot Gitea il y a 5 mois
- Tracker changé de Autre à Development
- Statut changé de Nouveau à En cours
- Assigné à mis à Frédéric Péters
Frédéric Péters (fpeters) a ouvert une pull request sur Gitea concernant cette demande :
- URL : https://git.entrouvert.org/entrouvert/wcs/pulls/845
- Titre : WIP: widgets: put radiobuttons in a <ul>, like checkboxes (#82789)
- Modifications : https://git.entrouvert.org/entrouvert/wcs/pulls/845/files
Mis à jour par Frédéric Péters il y a 5 mois
1- Le markup des templates radiosGroup et des checkboxesGroup différent, une ul > li pour les checkboxes et une suite de labels pour les radios. Y'a-t-il un intérêt à garder 2 markups différents ou non, simplifier pour uniformiser ?
Il me semble que ça pourrait systématiquement être une liste.
C'est ce que la PR modifie.
2- Le markup du group ne semble pas correct avec une description : (...)
Pas d'opinion particulière ici.
À regarder l'exemple dans https://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA/roles/radiogroup_role il n'apparait pas qu'il faille exclusivement les boutons radio dans le radiogroup, l'exemple donné y inclut un libellé global, ça donne même l'impression inverse, qu'il faudrait remonter le radiogroup sur le <div> général du champ. Je laisserais ça à un autre ticket.
Je laisse à un autre ticket aussi le traitement des champs avec des images, qui ont été ajoutés sans <ul>.
Mis à jour par Robot Gitea il y a 5 mois
- Statut changé de Solution proposée à Solution validée
Thomas Jund (tjund) a approuvé une pull request sur Gitea concernant cette demande :
Mis à jour par Robot Gitea il y a 5 mois
- Statut changé de Solution validée à Résolu (à déployer)
Frédéric Péters (fpeters) a mergé une pull request sur Gitea concernant cette demande :
- URL : https://git.entrouvert.org/entrouvert/wcs/pulls/845
- Titre : widgets: put radiobuttons in a <ul>, like checkboxes (#82789)
- Modifications : https://git.entrouvert.org/entrouvert/wcs/pulls/845/files
Mis à jour par Transition automatique il y a 5 mois
- Statut changé de Résolu (à déployer) à Solution déployée
Mis à jour par Serghei Mihai il y a 5 mois
- Lié à Development #84549: adapter la feuille de style au nouveau markup des radiogroup et checkboxesgroup ajouté
widgets: put radiobuttons in a <ul>, like checkboxes (#82789)