Skip to content

improvement: add aria-label(ledby) for lists #5030

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

WofWca
Copy link
Collaborator

@WofWca WofWca commented Apr 23, 2025

So that screen readers announce "messages list" or "chats list"
instead of just "list" when you focus an item inside of it.

For easier review, turn on "hide whitespace".

TODO:

  • Merge the MR this is based on.
  • Add a changelog entry.

@WofWca WofWca force-pushed the wofwca/a11y-label-lists branch from abc4343 to 760261e Compare April 23, 2025 20:48
@WofWca WofWca force-pushed the wofwca/a11y-label-lists branch 3 times, most recently from 7b70ba8 to fdae968 Compare April 25, 2025 14:30
@WofWca WofWca force-pushed the wofwca/a11y-label-lists branch from fdae968 to 475cf1e Compare April 25, 2025 14:45
@WofWca WofWca force-pushed the wofwca/a11y-label-lists branch from 475cf1e to 39abaa9 Compare April 29, 2025 19:57
@WofWca WofWca marked this pull request as ready for review April 29, 2025 19:59
@WofWca WofWca force-pushed the wofwca/a11y-label-lists branch from 39abaa9 to 0d3a9ff Compare April 30, 2025 16:43
@WofWca WofWca mentioned this pull request May 2, 2025
3 tasks
WofWca added 3 commits May 4, 2025 15:50
So that screen readers say "Chats list" instead of just "list"
when the focus enters such a list.
So that screen readers announce "messages list"
instead of just "list" when you focus an item inside of it.
@WofWca WofWca force-pushed the wofwca/a11y-label-lists branch from 0d3a9ff to a17896d Compare May 4, 2025 11:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant