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
refactor: Add a more accessible dropdown menu #12745
refactor: Add a more accessible dropdown menu #12745
Conversation
SonarCloud Quality Gate failed. 9 Bugs No Coverage information |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works as expected!
//Used to switch to mobile view | ||
const MAX_WIDTH = 640; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We have a helper that could be used here to check for mobile devices instead of checking the window size.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
was this RP tested with RTL layout. ? |
It was not. I just checked it again and released a fix for the issue you created. Thanks! |
chore: missing package-lock.json changes from #12745
What does this PR do?
Add's a more accessible / simplified dropdown menu than the current implementation.
Closes Issue(s)
#12573, #12574