Seems to do the job by providing a global menu and not clutering the UI
when plasma-workspace does not provide a global menu.
BUG: 400335
ngraham |
Elisa |
Seems to do the job by providing a global menu and not clutering the UI
when plasma-workspace does not provide a global menu.
BUG: 400335
Works with Plasma 5.14 and a global menu applet
Automatic diff as part of commit; lint not applicable. |
Automatic diff as part of commit; unit tests not applicable. |
Seems to work perfectly. My only UI suggestion is to hide the hamburger button in the window when exporting the menu like this.
Also, there seems to be some code redundancy between this menu and the in-window hamburger menu. Can they share the same code somehow?
Why is it like this:
instead of this?
I mean, can we have some more Menu instead of only Elisa ?
I will have to investigate to know what is possible.
Also, there seems to be some code redundancy between this menu and the in-window hamburger menu. Can they share the same code somehow?
The idea is to do the work to allow native menu to be usable and remove the old one. Currently, quite some things are not working. I should probably add a WIP prefix.
I wanted to get feedback on how well it would be working given it seemed working on my setup.
Currently, Elisa is not really an application built around the idea of having a big and complex application menu bar.
This is the reason why you see only one menu.
No need a complex menu bar, but there should be *some* ones. For example: File | View | Playlist | Settings | Help (just an example)
Probably best to get this in rather than making it perfect first. Iteration can come later.
The one thing that I think should be changed before getting it in is that the menu should have a different name. "Elisa Elisa" isn't very user-friendly. Maybe "Elisa Control" or "Elisa Actions"?
Thanks for your help.
I will do that and update the review.
Sorry for my late answer.
it would be hard to do much better given the current limitations of the native global menu components
I fear I lack time to work upstream on this even if that should be the only right way to handle this.