Skip to content

[62976] Two menus intefere with each other #320

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

Merged
merged 1 commit into from
Jun 12, 2025
Merged

Conversation

HDinger
Copy link
Collaborator

@HDinger HDinger commented Jun 10, 2025

What are you trying to accomplish?

Remove custom focus handling as it results in weird bug when two menus are on the same page. When the first is opened, and then the second one is activated, the focus is set on the first button again which will result in the second menu to closed immediatly again

https://community.openproject.org/projects/design-system/work_packages/62976/activity#comment-1271441

@HDinger HDinger added the bug Something isn't working label Jun 10, 2025
Copy link

changeset-bot bot commented Jun 10, 2025

🦋 Changeset detected

Latest commit: e5df72a

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@openproject/primer-view-components Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@HDinger HDinger force-pushed the fix/62976-two-menu-close branch 5 times, most recently from e3345d7 to 6e026d5 Compare June 12, 2025 07:29
…s are on the same page. When the first is opened, and then the second one is activated, the focus is set on the first button again which will result in the second menu to closed immediatly again
@HDinger HDinger force-pushed the fix/62976-two-menu-close branch from 6e026d5 to e5df72a Compare June 12, 2025 07:34
@HDinger HDinger merged commit 28cdbe9 into main Jun 12, 2025
27 checks passed
@camertron
Copy link
Collaborator

Thanks for this @HDinger 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working patch release
Development

Successfully merging this pull request may close these issues.

3 participants