Skip to content

Fix stuck menu when submenu vanishes#7589

Merged
lucasmerlin merged 2 commits intomainfrom
lucas/fix-popup-not-closing-when-submenu-vanishes
Oct 7, 2025
Merged

Fix stuck menu when submenu vanishes#7589
lucasmerlin merged 2 commits intomainfrom
lucas/fix-popup-not-closing-when-submenu-vanishes

Conversation

@lucasmerlin
Copy link
Copy Markdown
Collaborator

This fixes a bug where a menu could get stuck, not closing at all, when the currently open submenu stops being shown.
I also added a way to reproduce this to the demo, as well as a test ensuring that there is no race condition in the fix.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Oct 6, 2025

Preview available at https://egui-pr-preview.github.io/pr/7589-lucasfix-popup-not-closing-when-submenu-vanishes
Note that it might take a couple seconds for the update to show up after the preview_build workflow has completed.

View snapshot changes at kitdiff

@lucasmerlin lucasmerlin added bug Something is broken egui labels Oct 6, 2025
Copy link
Copy Markdown
Owner

@emilk emilk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

awesome!

@lucasmerlin lucasmerlin changed the title Fix suck menu when submenu vanishes Fix stuck menu when submenu vanishes Oct 7, 2025
@lucasmerlin lucasmerlin merged commit 6524901 into main Oct 7, 2025
48 of 49 checks passed
@lucasmerlin lucasmerlin deleted the lucas/fix-popup-not-closing-when-submenu-vanishes branch October 7, 2025 08:16
Masterchef365 pushed a commit to Masterchef365/egui that referenced this pull request Apr 3, 2026
* Closes rerun-io/rerun#11301

This fixes a bug where a menu could get stuck, not closing at all, when
the currently open submenu stops being shown.
I also added a way to reproduce this to the demo, as well as a test
ensuring that there is no race condition in the fix.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something is broken egui

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Context menu doesn't disappear

2 participants