Skip to content
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

A11y Refactor: MeetupList Component #8580

Closed
TylerAPfledderer opened this issue Nov 13, 2022 · 5 comments · Fixed by #9576
Closed

A11y Refactor: MeetupList Component #8580

TylerAPfledderer opened this issue Nov 13, 2022 · 5 comments · Fixed by #9576
Assignees
Labels
a11y ♿️ This issue relates to accessibility Status: Stale This issue is stale because it has been open 30 days with no activity.

Comments

@TylerAPfledderer
Copy link
Contributor

TylerAPfledderer commented Nov 13, 2022

Objective

Improving accessibility and UI markup for the MeetupList component, including UX with screen readers.

Description

Now that this component has been migrated to ChakraUI for the UI Epic, there might be further improvements that can be made.

  • As this is a list, it could be rendered with Chakra's List components and possibly an aria-label to clarify what the list is, else a screen reader would not declare to the user what group of items they are navigating into (cognitive load consideration).
  • The search input should include at least hidden text for the screen reader to state that "results will update as you type". This text can be in a separate element connected to the input via aria-describedby (maybe the preferred way)
  • If a user makes a search in the input that turns up no results, there should be a live region surrounding the info banner that immediately displays to have the screen reader assertively read out it's message.
  • Should there be search results, because the list is in a List component as suggested above, the user will be given the number of items that rendered once they navigate into the list.
@minimalsm minimalsm added the a11y ♿️ This issue relates to accessibility label Nov 28, 2022
@TylerAPfledderer TylerAPfledderer changed the title Improve Accessibility of MeetupList Component A11y Refactor: MeetupList Component Nov 28, 2022
@github-actions
Copy link
Contributor

This issue is stale because it has been open 45 days with no activity.

@github-actions github-actions bot added the Status: Stale This issue is stale because it has been open 30 days with no activity. label Jan 13, 2023
@corwintines corwintines removed the Status: Stale This issue is stale because it has been open 30 days with no activity. label Feb 9, 2023
@corwintines
Copy link
Member

This sounds great @TylerAPfledderer. Is this something you would be interested in taking on?

@TylerAPfledderer
Copy link
Contributor Author

@corwintines Yes sir!

@corwintines
Copy link
Member

Wicked! Ill assign to you :)

@github-actions
Copy link
Contributor

This issue is stale because it has been open 45 days with no activity.

@github-actions github-actions bot added the Status: Stale This issue is stale because it has been open 30 days with no activity. label Mar 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a11y ♿️ This issue relates to accessibility Status: Stale This issue is stale because it has been open 30 days with no activity.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants