Skip to content

Commit

Permalink
fix(ui): remove unneeded aria-label from NavToggleButton
Browse files Browse the repository at this point in the history
the button already has descriptive text so aria-label is not needed
  • Loading branch information
AliKdhim87 committed Mar 20, 2024
1 parent af117d0 commit 6f2c442
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions packages/ui/src/components/Navigation/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -86,7 +86,6 @@ export const Navigation = forwardRef(
text={toggleButton?.openText}
icon="hamburger"
aria-expanded={drawerVisible}
aria-label="Open menu"
onClick={() => setDrawerVisible(!drawerVisible)}
/>
) : null}
Expand All @@ -107,7 +106,6 @@ export const Navigation = forwardRef(
text={toggleButton?.closeText}
icon="close"
aria-expanded={drawerVisible}
aria-label="Close menu"
onClick={() => setDrawerVisible(!drawerVisible)}
/>
</NavigationList>
Expand Down

0 comments on commit 6f2c442

Please sign in to comment.