wails icon indicating copy to clipboard operation
wails copied to clipboard

Support more menu roles

Open leaanthony opened this issue 3 years ago • 10 comments

See https://www.electronjs.org/docs/latest/api/menu-item/#roles

leaanthony avatar Nov 07 '21 18:11 leaanthony

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Jan 03 '22 20:01 stale[bot]

Reopen

leaanthony avatar Jan 04 '22 08:01 leaanthony

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Apr 16 '22 12:04 stale[bot]

@leaanthony I assume you still want this open. Maybe stale-bot should ignore issues which have a certain label (eg "not-stale")

binyamin avatar May 03 '22 21:05 binyamin

Hi! Is this issue still up for grabs?

SubhanShah1 avatar Aug 21 '23 15:08 SubhanShah1

This is now being deferred to v3. But yes it's up for grabs if you want it :-)

leaanthony avatar Oct 25 '23 08:10 leaanthony

I'd still like to do this. Just have a few questions, have went through the code a bit to try and understand what's happening and it seems like the application menu (SetApplicationMenu)is only being set for Darwin? The definitions exist for Windows and Linux but I don't see them being used anywhere.

I saw that as I was trying to figure out how the role value interacts with the application (i.e how does the OS know what menu to show). Now I'm wondering if the purpose of the ticket is just to add the roles for future use or if it's to add them but it's okay that they only get used on Darwin (which I'm assuming is MacOS only)?

SubhanShah1 avatar Nov 07 '23 14:11 SubhanShah1

SetApplicationMenu is on all platforms. Darwin has this concept of an "application menu" which is the app specific one with CMD+Q Quit menu item if that's what you mean? Roles are a group of menu items. Like Electrons: https://www.electronjs.org/docs/latest/api/menu-item#roles

leaanthony avatar Nov 08 '23 10:11 leaanthony