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.

Sign up for GitHub

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

Jump to bottom

Add Context Menu Widget #87

Merged
merged 4 commits into from
Oct 17, 2024
Merged

Add Context Menu Widget #87

merged 4 commits into from
Oct 17, 2024
+255 −15

Conversation

tim-blackbird
Copy link
Contributor

@tim-blackbird tim-blackbird commented Oct 17, 2024 •
edited
Loading

Testing

cargo run -p bevy_context_menu --example simple

Showcase

image

Future Work

  • Prevent the menu from getting clipped at the edge of the window

@tim-blackbird tim-blackbird added C-Feature Make something new possible S-Needs-Review Needs reviewer attention (from anyone!) to move forward labels Oct 17, 2024
@tim-blackbird
Copy link
Contributor Author

I just realized I can easily get rid of the Arc<Mutex<>> I used but feel free to merge this if you get to it, Alice. I need to grab dinner anyways, I'll get to it later-ish

@alice-i-cecile alice-i-cecile added this pull request to the merge queue Oct 17, 2024
Merged via the queue into bevyengine:main with commit ea9d6de Oct 17, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Atlas16A Atlas16A Atlas16A approved these changes

Assignees
No one assigned
Labels
C-Feature Make something new possible S-Needs-Review Needs reviewer attention (from anyone!) to move forward
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@tim-blackbird @Atlas16A @alice-i-cecile