Skip to content
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

fix(menu): Bring the user back to the open/close button when reaching the last item in mobile - FRONT-4288 #3276

Merged
merged 5 commits into from Mar 20, 2024

Conversation

planctus
Copy link
Contributor

@planctus planctus commented Mar 5, 2024

There is still an issue in the storybook demo, using the "open in a new canvas" the problem can still be reproduced, suprisingly enough only the first time, if you keep tabbing from the second iteration the trap works as expected.
But this is not a real use case, we don't have anything after the menu in the demo, but in the real world there is an entire webpage to set the focus on and in that case the functionality works as expected.

Copy link

github-actions bot commented Mar 5, 2024

@github-actions github-actions bot temporarily deployed to pull request March 18, 2024 12:01 Inactive
@github-actions github-actions bot temporarily deployed to pull request March 18, 2024 14:27 Inactive
@emeryro emeryro merged commit 690a775 into v4-dev Mar 20, 2024
7 checks passed
@emeryro emeryro deleted the FRONT-4288-Menu-keyboard branch March 20, 2024 09:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants