fix: windowmenu role did not show the window list on macOS #16387
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description of Change
Fix #15912 properly. The whole menu needs to have
window
role set, as the OS appends the window list at the end of the submenu instead of replacing a single menu item. In case ofwindowmenu
role, we cannot change the role towindow
, so we need to check forwindowmenu
in the menu controller.The default window menu regressed in #15930, when it switched from using
window
role towindowmenu
.Before
After
/cc @codebytere, @sindresorhus
Checklist
npm test
passesRelease Notes
Notes: Fixed
windowmenu
role not showing the window list on macOS