Skip to content

context_menu: Fix certain menu items not reacting to click events#49557

Merged
yeskunall merged 2 commits intomainfrom
fix-context-menu-c98d930c8e6514ee
Feb 23, 2026
Merged

context_menu: Fix certain menu items not reacting to click events#49557
yeskunall merged 2 commits intomainfrom
fix-context-menu-c98d930c8e6514ee

Conversation

@yeskunall
Copy link
Member

@yeskunall yeskunall commented Feb 19, 2026

Use Workspace::for_window instead of window.root().flatten().

bun-blank.2026-02-23.13.49.03.mp4
bun-blank.2026-02-23.13.43.05.mp4

Before you mark this PR as ready for review, make sure that you have:

  • Added a solid test coverage and/or screenshots from doing manual testing
  • Done a self-review taking into account security and performance aspects
  • Aligned any UI changes with the UI checklist

Release Notes:

  • Fixed certain context menu items not reacting to click events

Use `Workspace::for_window` instead of `window.root().flatten()`.
@yeskunall yeskunall requested a review from a team as a code owner February 19, 2026 03:26
@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Feb 19, 2026
@zed-community-bot zed-community-bot bot added the staff Pull requests authored by a current member of Zed staff label Feb 19, 2026
@yeskunall yeskunall enabled auto-merge (squash) February 23, 2026 18:52
@yeskunall yeskunall merged commit 3333c79 into main Feb 23, 2026
27 checks passed
@yeskunall yeskunall deleted the fix-context-menu-c98d930c8e6514ee branch February 23, 2026 18:58
Anthony-Eid pushed a commit to bobbymannino/zed that referenced this pull request Feb 25, 2026
…d-industries#49557)

Use `Workspace::for_window` instead of `window.root().flatten()`.



https://github.com/user-attachments/assets/3ea6dae3-166d-49e4-9fff-8256e13cc2bf



https://github.com/user-attachments/assets/eb8d9394-5f71-4df6-b0db-c1c8078f1b9a



Before you mark this PR as ready for review, make sure that you have:
- [x] Added a solid test coverage and/or screenshots from doing manual
testing
- [x] Done a self-review taking into account security and performance
aspects
- [x] Aligned any UI changes with the [UI
checklist](https://github.com/zed-industries/zed/blob/main/CONTRIBUTING.md#uiux-checklist)

Release Notes:

- Fixed certain context menu items not reacting to click events
tahayvr pushed a commit to tahayvr/zed that referenced this pull request Mar 4, 2026
…d-industries#49557)

Use `Workspace::for_window` instead of `window.root().flatten()`.



https://github.com/user-attachments/assets/3ea6dae3-166d-49e4-9fff-8256e13cc2bf



https://github.com/user-attachments/assets/eb8d9394-5f71-4df6-b0db-c1c8078f1b9a



Before you mark this PR as ready for review, make sure that you have:
- [x] Added a solid test coverage and/or screenshots from doing manual
testing
- [x] Done a self-review taking into account security and performance
aspects
- [x] Aligned any UI changes with the [UI
checklist](https://github.com/zed-industries/zed/blob/main/CONTRIBUTING.md#uiux-checklist)

Release Notes:

- Fixed certain context menu items not reacting to click events
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed The user has signed the Contributor License Agreement staff Pull requests authored by a current member of Zed staff

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant