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

Xwayland menus open with toplevel animation #2007

Closed
spl237 opened this issue Nov 7, 2023 · 10 comments · Fixed by #2026
Closed

Xwayland menus open with toplevel animation #2007

spl237 opened this issue Nov 7, 2023 · 10 comments · Fixed by #2026
Labels
Milestone

Comments

@spl237
Copy link

spl237 commented Nov 7, 2023

Describe the bug
When a GTK application is running under Xwayland, opening a menu causes it to be shown as a window expanding outwards from a point, as with toplevel windows.

To Reproduce
Steps to reproduce the behavior:
Open a GTK application under Xwayland, e.g. GDK_BACKEND=x11 geany, and open a menu in the application.
Does not seem to happen with Qt applications such as VLC.

Expected behavior
The menu should simply appear at the correct place with no animation.

Wayfire version
0.8.0 release

@spl237 spl237 added the bug label Nov 7, 2023
@ammen99
Copy link
Member

ammen99 commented Nov 7, 2023

What are your settings for the animate plugin?

@spl237
Copy link
Author

spl237 commented Nov 7, 2023

Unchanged from the defaults, so whatever is in the metadata, I guess. I have no [animate] section in my config.

@spl237
Copy link
Author

spl237 commented Nov 7, 2023

Just tried copying the enabled_for line from the metadata to my config so I can edit it.

The default in the metadata is '(type equals "toplevel" | (type equals x-or) & focusable equals true))'

If I change that to just 'type equals "toplevel"', that fixes the problem, but I'm not sure what other animations that might affect?

@ammen99
Copy link
Member

ammen99 commented Nov 7, 2023

Does http://ix.io/4KT9 fix the issue?

@spl237
Copy link
Author

spl237 commented Nov 7, 2023

Those changes don't build for me - looks like they rely on some other changes elsewhere.

@ammen99
Copy link
Member

ammen99 commented Nov 7, 2023

I am on latest master, 9bed909

@ammen99
Copy link
Member

ammen99 commented Nov 7, 2023

Oh, I just realized that I haven't updated all sites where the variable is needed, sorry. I'll send you an updated patch soon.

@ammen99
Copy link
Member

ammen99 commented Nov 7, 2023

Ok, so find references in my current NeoVim setup doesn't find all usages .. anyway, here is a patch which seems to build here http://ix.io/4KTW

@ammen99 ammen99 added this to the 0.8.1 milestone Nov 8, 2023
@ammen99
Copy link
Member

ammen99 commented Nov 17, 2023

@spl237 ping, does the patch help?

@spl237
Copy link
Author

spl237 commented Nov 17, 2023

Sorry - not had a chance to look at this yet; I've been dealing with some more urgent matters on the currently-shipping release, so work on 0.8 is on hold at the moment.

This was referenced Nov 20, 2023
ammen99 added a commit that referenced this issue Nov 23, 2023
ammen99 added a commit that referenced this issue Nov 23, 2023
ammen99 added a commit that referenced this issue Mar 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants