a11y: add group role to blocks (#88620) #1312
No reviewers
Labels
No Label
No Milestone
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: entrouvert/wcs#1312
Loading…
Reference in New Issue
No description provided.
Delete Branch "wip/88620-a11y-block-group-role"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
7ed2d05f07
to07e6d62f4e
07e6d62f4e
to6d6bae428e
6d6bae428e
to4e38acb959
4e38acb959
to92ce9390ec
92ce9390ec
toacab2c5f0a
acab2c5f0a
toacabf97512
@ -501,0 +504,4 @@
@property
def a11y_role(self):
# don't mark block as a group if it has no label
Mon intention initiale était d'également prendre en compte la présence d'au minimum 2 champs de données (cf https://git.entrouvert.org/entrouvert/wcs/commit/acab2c5f0) mais il me semble que se baser sur la présence d'un libellé affiché pour décréter que c'est un groupe de champs est suffisant par rapport aux usages.
@ -569,1 +580,3 @@
return htmltag('span', **attrs) + htmltext('</span>')
# add a tag even if there's no label to display as it's used as an anchor point
# for links to errors.
return htmltag('div', **attrs) + htmltext('</div>')
J'étais parti pour totalement cacher ça (parce que le span prenait de la hauteur et je ne voyais pas son utilité), j'ai retrouvé l'utilité (servir d'ancre) mais j'ai transformé en div pour obtenir une hauteur de 0.
WIP: a11y: add group role to blocks (#88620)to a11y: add group role to blocks (#88620)