Skip to content

ETQ instructeur utilisant un terminal mobile, je veux que le détail des notifications soit lisible#13294

Open
inseo wants to merge 2 commits into
mainfrom
a11y-notif-details
Open

ETQ instructeur utilisant un terminal mobile, je veux que le détail des notifications soit lisible#13294
inseo wants to merge 2 commits into
mainfrom
a11y-notif-details

Conversation

@inseo

@inseo inseo commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

Problèmes

  • La listes des notifications n'est pas balisées comme telle.
  • Sur un terminal de faible dimensions le détail des notifications est difficilement lisible.

Après

notif-OK.mp4

Avant

notif-KO.mp4

@inseo
inseo force-pushed the a11y-notif-details branch 3 times, most recently from 5ba91f4 to cd24815 Compare June 16, 2026 10:51
@tchak

tchak commented Jun 16, 2026

Copy link
Copy Markdown
Member

En regardant ce code, je ne peux pas m'empêcher de penser que ce serait mieux sans JS, avec du CSS branché sur plusieurs breakpoints de largeur d'écran. Vous en pensez quoi ?

@inseo

inseo commented Jun 18, 2026

Copy link
Copy Markdown
Contributor Author

@tchak Justement, le faire en full CSS n'est pas possible car ça signifierait que la quantité d'informations visibles diffère suivant le terminal utilisé. Et c'est justement ce que je cherche à éviter. :)

Mais en relisant le code JS, je me rends compte que ça n'est pas le cas et que ça fonctionne bien au redimensionnement de la page, mais pas au chargement initial sur des fenêtres de différentes largeurs. :s

Je vais donc modifier le code JS pour que la valeur à partir de laquelle les éléments sont tronquées soit immuable, indépendamment du terminal utilisé. :)

@inseo
inseo marked this pull request as draft June 18, 2026 13:20
@inseo
inseo force-pushed the a11y-notif-details branch from cd24815 to e316c73 Compare June 18, 2026 14:34
@inseo
inseo requested a review from tchak June 18, 2026 14:34
@inseo
inseo marked this pull request as ready for review June 18, 2026 14:35
@inseo
inseo force-pushed the a11y-notif-details branch from e316c73 to ba553a1 Compare July 2, 2026 12:44
@inseo inseo moved this to En cours in The Kanban Jul 2, 2026
@inseo inseo self-assigned this Jul 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: En cours

Development

Successfully merging this pull request may close these issues.

3 participants