a11y: add group role to blocks (#88620) #1312

Merged
fpeters merged 1 commits from wip/88620-a11y-block-group-role into main 2024-03-29 08:33:25 +01:00
Owner
No description provided.
fpeters added 1 commit 2024-03-25 13:11:58 +01:00
gitea/wcs/pipeline/head There was a failure building this commit Details
7ed2d05f07
a11y: add group role to blocks (#88620)
fpeters force-pushed wip/88620-a11y-block-group-role from 7ed2d05f07 to 07e6d62f4e 2024-03-25 14:13:37 +01:00 Compare
fpeters force-pushed wip/88620-a11y-block-group-role from 07e6d62f4e to 6d6bae428e 2024-03-25 14:51:15 +01:00 Compare
fpeters force-pushed wip/88620-a11y-block-group-role from 6d6bae428e to 4e38acb959 2024-03-26 07:40:46 +01:00 Compare
fpeters force-pushed wip/88620-a11y-block-group-role from 4e38acb959 to 92ce9390ec 2024-03-26 08:23:56 +01:00 Compare
fpeters force-pushed wip/88620-a11y-block-group-role from 92ce9390ec to acab2c5f0a 2024-03-26 08:34:49 +01:00 Compare
fpeters force-pushed wip/88620-a11y-block-group-role from acab2c5f0a to acabf97512 2024-03-26 08:39:19 +01:00 Compare
fpeters reviewed 2024-03-26 08:42:46 +01:00
@ -501,0 +504,4 @@
@property
def a11y_role(self):
# don't mark block as a group if it has no label
Author
Owner

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.

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>')
Author
Owner

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.

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.
fpeters changed title from WIP: a11y: add group role to blocks (#88620) to a11y: add group role to blocks (#88620) 2024-03-26 08:45:19 +01:00
pmarillonnet approved these changes 2024-03-26 09:42:09 +01:00
fpeters merged commit 6f6859098a into main 2024-03-29 08:33:25 +01:00
fpeters deleted branch wip/88620-a11y-block-group-role 2024-03-29 08:33:25 +01:00
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/wcs#1312
No description provided.