Skip to content
New issue

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

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

Already on GitHub? # to your account

fix(VBreadcrumbsDivider): add aria-hidden attribute #20893

Merged
merged 1 commit into from
Jan 24, 2025

Conversation

mtdvlpr
Copy link
Contributor

@mtdvlpr mtdvlpr commented Jan 21, 2025

Description

A divider in a list of breadcrumbs is only visual. It should be ignored by screenreaders.

Markup:

<v-breadcrumbs :items="['Foo', 'Bar', 'Fizz']"></v-breadcrumbs>
<ul class="v-breadcrumbs v-breadcrumbs--density-default">
  <li class="v-breadcrumbs-item">Foo</li>
  <li class="v-breadcrumbs-divider" aria-hidden="true">/</li>
  <li class="v-breadcrumbs-item">Bar</li>
  <li class="v-breadcrumbs-divider" aria-hidden="true">/</li>
  <li class="v-breadcrumbs-item v-breadcrumbs-item--disabled">Fizz</li>
</ul>

@johnleider johnleider changed the title fix(VBreadcrumbsDivider): add aria-hidden to breadcrumbs divider fix(VBreadcrumbsDivider): add aria-hidden attribute Jan 24, 2025
@johnleider johnleider merged commit 301576b into vuetifyjs:master Jan 24, 2025
10 checks passed
@johnleider johnleider added this to the v3.7.x milestone Jan 24, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants