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

Add compatibility with DING for Dynamic Transparency #1767

Merged
merged 1 commit into from
Jul 26, 2022

Conversation

skoruppa
Copy link

This patch adds compatibility with Desktop Icons NG for Dynamic Transparency functionality by taking into account that, under Wayland, the "skip-taskbar" property can be set only under very specific circumstances (that, currently, only DING does fulfill).

You can read about issue details at https://gitlab.com/rastersoft/desktop-icons-ng/-/issues/230

This patch adds compatibility with Desktop Icons NG for Dynamic Transparency functionality by taking into account that, under Wayland, the "skip-taskbar" property can be set only under very specific circumstances (that, currently, only DING does fulfill).
@3v1n0
Copy link
Collaborator

3v1n0 commented Jul 6, 2022

Mh, while I think this can be fine, I feel it could be too generic, and we could end up considering other windows other than DING, can we maybe use a stricter filter here?

@skoruppa
Copy link
Author

skoruppa commented Jul 7, 2022

Let me just quote DING's developer (@rastersoft) comment from lamarios/gnome-shell-extension-transparent-top-bar#9 (exact same thing but for a different extension)

Hi, I'm the creator of Desktop Icons ND (DING). Unfortunately, under Wayland, there aren't thinks like a "Desktop window" or similar, so I had to use several tricks to emulate a desktop with icons. But fortunately I think that there is a way of solving this: in Wayland, no application can set the "skip_from_window_list" flag, which is mandatory for things like a desktop window. But an extension can set it under some very specific circumstances (which were put there specifically for DING, BTW). So a solution is to check if we are under Wayland AND a window has the "skip_from_window_list" flag set, and in that case set the top bar as transparent.

So it seems no other windows will use this flag in Wayland. I based my change on his

@vanvugt
Copy link
Collaborator

vanvugt commented Jul 21, 2022

This looks like exactly the right fix.

I share Marco's concerns that the mutter window flags here are poorly named and vague, but that ship has sailed some years ago. If we want to fix that then we can discuss it separately.

@vanvugt vanvugt self-requested a review July 21, 2022 09:54
Copy link
Collaborator

@vanvugt vanvugt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks right to me, so approved. It should help with https://launchpad.net/bugs/1981837

@vanvugt
Copy link
Collaborator

vanvugt commented Jul 26, 2022

Confirmed to fix https://launchpad.net/bugs/1981837

@vanvugt vanvugt merged commit 19cc612 into micheleg:master Jul 26, 2022
@Coeur-Noir
Copy link

Unfortunately problem is back - noticed today on a fresh Ubuntu 22.04 installation.

gnome-shell-extension-ubuntu-dock is 72~ubuntu5.22.04.1
and
gnome-shell-extension-desktop-icons-ng is 43-2ubuntu1

@vanvugt
Copy link
Collaborator

vanvugt commented May 2, 2023

Since this bug is closed already, please open a new one.

@Coeur-Noir
Copy link

Not related to DiNG but… wslu
https://askubuntu.com/questions/1410764/ubuntu-22-04-fresh-install-desktop-entries-do-not-open

Removing wslu package restores expected behavior.

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

Successfully merging this pull request may close these issues.

4 participants