misc: improve a11y of timetable widget (#74067) #592

fpeters · 2023-08-12T20:42:56+02:00

fpeters commented

2023-08-12 20:42:56 +02:00

les intitulés des boutons d'heure ne sont pas assez pertinent (ils devraient préciser la date)

C'est fait, en mettant la date dans un <span class="sr-only"> pour concerner uniquement les lecteurs d'écran;

les dates ne sont pas implémentées comme des titres

C'est fait, dans des <h5> pour être sûr de se trouver sous le niveau des sous-titres (+ modif css pour que ne rien changer).

Aussi, amélioration pour les flèches de navigation, pour plutôt avoir un texte accessible (en .sr-only aussi), et modification en passant du texte de celles-ci, pour être jours suivants/précédents, plutôt que semaine suivante/précédente, dans la mesure où la navigation n'est pas toujours précisément par semaine.

> les intitulés des boutons d'heure ne sont pas assez pertinent (ils devraient préciser la date) C'est fait, en mettant la date dans un `<span class="sr-only">` pour concerner uniquement les lecteurs d'écran; > les dates ne sont pas implémentées comme des titres C'est fait, dans des `<h5>` pour être sûr de se trouver sous le niveau des sous-titres (+ modif css pour que ne rien changer). Aussi, amélioration pour les flèches de navigation, pour plutôt avoir un texte accessible (en .sr-only aussi), et modification en passant du texte de celles-ci, pour être jours suivants/précédents, plutôt que semaine suivante/précédente, dans la mesure où la navigation n'est pas toujours précisément par semaine.

fpeters added 1 commit 2023-08-12 20:42:57 +02:00

gitea/wcs/pipeline/head This commit looks good Details

66cc81a691 misc: improve a11y of timetable widget (#74067 )

vdeniaud requested changes 2023-10-02 16:35:41 +02:00

vdeniaud left a comment

(+ modif css pour que ne rien changer).

Je vois la modif dans admin.scss mais ça ne s'applique qu'au rendu BO il me semble, donc le rendu front est laissé modifié (en moins bien).

> (+ modif css pour que ne rien changer). Je vois la modif dans admin.scss mais ça ne s'applique qu'au rendu BO il me semble, donc le rendu front est laissé modifié (en moins bien).

fpeters commented

2023-10-02 16:50:33 +02:00

(+ modif css pour que ne rien changer).

Je vois la modif dans admin.scss mais ça ne s'applique qu'au rendu BO il me semble, donc le rendu front est laissé modifié (en moins bien).

Oui il y a une branche côté publik-base-theme avec le même code, que je comptais pousser en même temps. (2a8151f267, que je n'avais pas poussée).

> > (+ modif css pour que ne rien changer). > > Je vois la modif dans admin.scss mais ça ne s'applique qu'au rendu BO il me semble, donc le rendu front est laissé modifié (en moins bien). Oui il y a une branche côté publik-base-theme avec le même code, que je comptais pousser en même temps. (https://git.entrouvert.org/entrouvert/publik-base-theme/commit/2a8151f2671c53d0a12cc367ab79fc70a1442e1f, que je n'avais pas poussée).

fpeters requested review from vdeniaud 2023-10-02 16:50:36 +02:00

vdeniaud approved these changes 2023-10-02 16:55:16 +02:00

vdeniaud left a comment

Dac, j'avais un moyen de voir cette branche ?

fpeters commented

2023-10-02 16:55:59 +02:00

Dac, j'avais un moyen de voir cette branche ?

Non, je l'avais juste en local...

> Dac, j'avais un moyen de voir cette branche ? Non, je l'avais juste en local...

fpeters merged commit a6cf14d3f0 into main

2023-10-02 17:52:46 +02:00

fpeters deleted branch wip/74067-timetable-a11y

2023-10-02 17:52:46 +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#592