Pass aria-label down to correct element #1101
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR patches a bug introduced in my last TabNav PR. I refactored the components so that the
nav
component is the inner component (as it should be & to match implementation in .com), but I forgot to forwardaria-label
down to thenav
which was throwing axe errors in Memex.This PR fixes that and updates the tests to be more specific, making sure the aria-label gets applied to the
nav
and not the wrapping element ☮️Screenshots
Please provide before/after screenshots for any visual changes
Merge checklist
index.d.ts
) if necessaryTake a look at the What we look for in reviews section of the contributing guidelines for more information on how we review PRs.