misc: add button role to tracking code link (#78150) #359

Merged
fpeters merged 1 commits from wip/78150-tracking-code-a11y into main 2023-06-12 09:37:36 +02:00
Owner
No description provided.
fpeters force-pushed wip/78150-tracking-code-a11y from f8406e3709 to 3fca81a2c4 2023-06-04 12:29:20 +02:00 Compare
Author
Owner

Changement d'approche en cours de route, plutôt que juste ce lien je prends le commentaire de l'audit d'accessibilité au-delà du code de suivi,

Le liens code de suivi ouvre une popup et devrais donc être un button

et tous les liens qui ont le js pour faire une ouverture en popup, je les modifie pour avoir role="button" + le js d'activation au clavier.

Changement d'approche en cours de route, plutôt que juste ce lien je prends le commentaire de l'audit d'accessibilité au-delà du code de suivi, > Le liens code de suivi ouvre une popup et devrais donc être un button et tous les liens qui ont le js pour faire une ouverture en popup, je les modifie pour avoir role="button" + le js d'activation au clavier.
fpeters changed title from WIP: misc: add button role to tracking code link (#78150) to misc: add button role to tracking code link (#78150) 2023-06-05 10:36:48 +02:00
lguerin approved these changes 2023-06-06 19:02:58 +02:00
fpeters merged commit 2c312e5e79 into main 2023-06-12 09:37:36 +02:00
fpeters deleted branch wip/78150-tracking-code-a11y 2023-06-12 09:37:36 +02: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#359
No description provided.