Development #38192
accessibilité du dé/pliage des cellules
0%
Description
Le dé/pliage d'une cellule marquée foldable/folded se fait par clic sur un <h2>, il n'y a rien côté accessibilité (aria-) pour indiquer ça, le focus ne s'arrête pas sur ces titres, etc.
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 4 ans
- Projet changé de Intégrations graphiques Publik à Combo
Mis à jour par Frédéric Péters il y a plus de 4 ans
- Fichier 0001-a11y-add-support-for-foldable-folding-cells-38192.patch 0001-a11y-add-support-for-foldable-folding-cells-38192.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Ajout d'un tab-index=0 pour que le clavier puisse s'arrêter dessus, événement keydown pour que taper sur entrée plie/déplie, ajout attribut aria-expanded parce que c'est ça qui semble recommandé / le mieux convenir. Comme on se trouve ainsi à devoir changer le balisage, on passe d'un événement global, commun à toutes les cellules (présentes et à venir (ajax)), à un événement local à une cellule.
Mis à jour par Thomas Jund il y a plus de 4 ans
Quelques retours
Ajouter un role=button au titre
$cell_title.attr('tabindex', '0'); + $cell_title.attr('role', 'button');
Tester si la cell est `.foldable` directement en début de fonction pour éviter de stocker des vars pour rien, éviter une imbrication et exit le plus tôt possible
function prepare_foldable(cell) { + if (!$cell.is('.foldable')) { + return; + } … var $cell_title = $cell.find('> div > h2:first-child'); - if ($cell.is('.foldable')) { … return false; }); - }
J'aurais eu tendance à faire une function pour
if ($cell.is('.folded')) { $cell_title.attr('aria-expanded', 'false'); } else { $cell_title.attr('aria-expanded', 'true'); }
et ainsi éviter de dupliquer du code.
Enfin je ne suis pas sûr de l'intérêt de mettre un .keydown sur l'image sachant qu'elle n'est pas focusable.
Mis à jour par Frédéric Péters il y a plus de 4 ans
- Fichier 0001-a11y-add-support-for-foldable-folding-cells-38192.patch 0001-a11y-add-support-for-foldable-folding-cells-38192.patch ajouté
Commentaires pris en compte.
Mis à jour par Thomas Jund il y a plus de 4 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Frédéric Péters il y a plus de 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 43b61611599cd0729fb63a4620f634e9c4c2dc0b Author: Frédéric Péters <fpeters@entrouvert.com> Date: Wed Dec 4 20:42:56 2019 +0100 a11y: add support for foldable/folding cells (#38192)
Mis à jour par Frédéric Péters il y a plus de 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
a11y: add support for foldable/folding cells (#38192)