Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Submenus are decorated when using twm #84

Open
scaramacai opened this issue Apr 22, 2024 · 3 comments
Open

Submenus are decorated when using twm #84

scaramacai opened this issue Apr 22, 2024 · 3 comments
Assignees

Comments

@scaramacai
Copy link

This is a minor problem.
Cannot say if this is a flaw of agar or it is due to twm not supporting modern wm protocols. Anyway here is a screeshot:
Screenshot_2024-04-22_10-05-34

@scaramacai
Copy link
Author

Decorations are present also in combos

@ghost
Copy link

ghost commented Apr 24, 2024

I can also veryify this with Fluxbox. My thought on this was that by default the window used to create the menu wasn't coded with the flags implemented in the function that need to hide this or that you need to add them in your code (AG_WINDOW_PLAIN or AG_WINDOW_NOTITLE ?). A menu has those flags but a popup menu not so much I guess. AG_MenuNew(void *parent, Uint flags) vs. AG_PopupNew(void *obj)

@JulNadeauCA JulNadeauCA self-assigned this May 6, 2024
@JulNadeauCA
Copy link
Owner

The relevant code is under SetHints() in gui/drv_glx.c. There are hints specific to some different WMs such as Motif, KWM and Gnome for disabling window decorations, though most modern WMs will use the EWMH-compliant type (the wmType field of AG_Window that is used to set X properties such as _NET_WM_WINDOW_TYPE_MENU or _NET_WM_WINDOW_TYPE_DOCK).

I'm wondering, does twm / Fluxbox recognize a decoration-disabling hint (e.g., a property that could be set with XChangeProperty()) or do they provide another way to disable window decorations?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants