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

Breadcrumbs aren't read individually by NVDA #1646

Open
adamwoodnz opened this issue Jun 23, 2023 · 0 comments · May be fixed by #1641
Open

Breadcrumbs aren't read individually by NVDA #1646

adamwoodnz opened this issue Jun 23, 2023 · 0 comments · May be fixed by #1641
Labels
[Component] Learn Theme Website development issues related to the Learn theme. [Type] Bug Something isn't working on the Learn website.

Comments

@adamwoodnz
Copy link
Contributor

adamwoodnz commented Jun 23, 2023

As explained by @alexstine

This is a Windows-specific bug. Voiceover and Windows screen readers work differently in the way that Voiceover traverses the content one item at a time where NVDA reads the links like a sentence due to the display: inline; set. I think that is okay for breadcrumbs, but to get the best verbosity results, it needs to be "Link 1, Link2, etc." Breadcrumbs can be tricky... I wonder what would happen if we tried rendering them as display: block;. Then there likely would not be a need for the screen reader text anymore.

We should try using display: block breadcrumbs within a flex row container if the attempted fix in #1641 doesn't work.

@adamwoodnz adamwoodnz added [Type] Bug Something isn't working on the Learn website. [Component] Learn Theme Website development issues related to the Learn theme. labels Jun 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Component] Learn Theme Website development issues related to the Learn theme. [Type] Bug Something isn't working on the Learn website.
Projects
Status: In Progress
Development

Successfully merging a pull request may close this issue.

1 participant