Skip to content

Commit

Permalink
fix: breadcrumb isCurrent disabled should be false
Browse files Browse the repository at this point in the history
  • Loading branch information
anuraghazra committed Oct 5, 2020
1 parent 6c1c0ec commit 98ef3b6
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 7 deletions.
4 changes: 0 additions & 4 deletions src/breadcrumbs/BreadcrumbLink.ts
Original file line number Diff line number Diff line change
Expand Up @@ -20,10 +20,6 @@ export const useBreadcrumbLink = createHook<
compose: useLink,
keys: BREADCRUMB_LINK_KEYS,

useOptions(options) {
return { disabled: options.disabled || options.isCurrent, ...options };
},

useProps({ isCurrent }, htmlProps) {
return { "aria-current": isCurrent && "page", ...htmlProps };
},
Expand Down
3 changes: 0 additions & 3 deletions src/breadcrumbs/__tests__/Breadcrumbs.test.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -65,10 +65,7 @@ describe("Breadcrumb", () => {
<li>
<a
aria-current="page"
aria-disabled="true"
href="https://www.w3.org/TR/wai-aria-practices-1.1/#breadcrumb"
style="pointer-events: none;"
tabindex="-1"
>
Breadcrumb Pattern
</a>
Expand Down

0 comments on commit 98ef3b6

Please sign in to comment.