Skip to content

Focusout for Dropdown, Menubar and ContextMenu #22

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

Draft
wants to merge 12 commits into
base: main
Choose a base branch
from

Conversation

wheregmis
Copy link
Contributor

@wheregmis wheregmis commented May 8, 2025

Attempt for #18 & #28

@wheregmis wheregmis changed the title focusout Focusout for Dropdown and ContextMenu May 9, 2025
@wheregmis wheregmis marked this pull request as draft May 15, 2025 13:03
@wheregmis wheregmis marked this pull request as ready for review May 18, 2025 18:10
@wheregmis
Copy link
Contributor Author

Maybe we need something similar for menubar

@wheregmis
Copy link
Contributor Author

One main drawback for this is, to make accessibility work with this. I am not sure if we should go ahead with funtionality or accessibility. Until we find the sweet spot

@wheregmis wheregmis changed the title Focusout for Dropdown and ContextMenu Focusout for Dropdown, Menubar and ContextMenu May 30, 2025
@wheregmis wheregmis marked this pull request as draft May 30, 2025 20:45
@wheregmis wheregmis marked this pull request as ready for review May 30, 2025 20:45
@wheregmis wheregmis marked this pull request as draft May 31, 2025 18:01
@wheregmis
Copy link
Contributor Author

#28

@wheregmis wheregmis marked this pull request as ready for review June 2, 2025 13:55
@ealmloff
Copy link
Member

ealmloff commented Jun 2, 2025

When I serve this branch locally, I'm not able to interact with any of the buttons in the context menu dropdown. I'm guessing because when you click the button, the div loses focus?

Screen.Recording.2025-06-02.at.4.17.41.PM.mov

@wheregmis wheregmis marked this pull request as draft June 2, 2025 23:25
@wheregmis
Copy link
Contributor Author

Thank you @ealmloff for pointing it out. Looks like inert was making it not work for some reason. But we can leave it commented for now. Should be okay now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants