Skip to content

Make close_popup take an Id and add close_all_popups #5820

Merged
lucasmerlin merged 3 commits intomasterfrom
lucas/fix-popup-with-context-menu
Mar 18, 2025
Merged

Make close_popup take an Id and add close_all_popups #5820
lucasmerlin merged 3 commits intomasterfrom
lucas/fix-popup-with-context-menu

Conversation

@lucasmerlin
Copy link
Collaborator

Ooops, fixes a regression introduced in #5799

  • I have followed the instructions in the PR template

@github-actions
Copy link

Preview available at https://egui-pr-preview.github.io/pr/5820-lucasfix-popup-with-context-menu
Note that it might take a couple seconds for the update to show up after the preview_build workflow has completed.

@lucasmerlin lucasmerlin changed the title Fix popup not opening if there is also a context menu Make close_popup take an Id and add close_all_popups Mar 18, 2025
@lucasmerlin lucasmerlin merged commit a2afc8d into master Mar 18, 2025
46 checks passed
@lucasmerlin lucasmerlin deleted the lucas/fix-popup-with-context-menu branch March 18, 2025 14:32
darkwater pushed a commit to darkwater/egui that referenced this pull request Aug 24, 2025
Ooops, fixes a regression introduced in emilk#5799

* [x] I have followed the instructions in the PR template
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants