Skip to content

try to more closely match freedesktop specifications: distinguish bet…#84

Open
rmottola wants to merge 1 commit into
masterfrom
freedesktop_levels
Open

try to more closely match freedesktop specifications: distinguish bet…#84
rmottola wants to merge 1 commit into
masterfrom
freedesktop_levels

Conversation

@rmottola

Copy link
Copy Markdown
Member

…ween submenus and torn-off menus; attempt to set toolbar instead of dock for the main menu

…ween submenus and torn-off menus; attempt to set toolbar instead of dock for the main menu
@rmottola rmottola requested a review from fredkiefer as a code owner June 17, 2026 00:13
@rmottola

Copy link
Copy Markdown
Member Author

I think this branch is quite correct, however we cannot test it until GUI distinguishes (and sets) tear-off and submenus correctly

@fredkiefer

Copy link
Copy Markdown
Member

As I wrote in the gui PR, we need to change all the backends not just x11. Also this change needs to be checked on many window managers, especially on the one I am using (KDE). Otherwise we have an improvement for some, but a regression for others.

@rmottola

Copy link
Copy Markdown
Member Author

As I wrote in the gui PR, we need to change all the backends not just x11. Also this change needs to be checked on many window managers, especially on the one I am using (KDE). Otherwise we have an improvement for some, but a regression for others.

Of course, we need testing. Right now it appears to do no harm on windowmaker (but doesn't fix an annoying layering detail either). On Mac-X11 now all menus are movable, ont what we like, but more consistent. Will test xfce, please test kde.

I think freedesktop adherence is good. Reading the documentation, these styles were added specifically for NeXT style interface (!)
Currently, even with the gui change, submenus and teared-off menus are still the same, I'd like to fix that. Try on your side with props.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants