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

wallpaper: change layer to BOTTOM for proper working in KDE6 #400

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

cristiancolosimo
Copy link

(sorry, it's first pull request, I don't know what I am doing)

Relative to the Issue #393

@LGFae
Copy link
Owner

LGFae commented Mar 1, 2025

I do not think this is the proper fix. The Background layer is so called because it is supposed to be used for backgrounds. Setting it to Bottom could potentially cause issue with things like status bars.

@cristiancolosimo
Copy link
Author

cristiancolosimo commented Mar 1, 2025

Thanks for reply,

Yeah, I thought so too, but on kde I'm not having any problem, I have no way to try it on other environments at the moment. Tomorrow I'll try to test it on gnome(and if i can with some tiling window manager). Can you give me some examples of status bars to test, which I install and test with the default configs.

@Axlefublr
Copy link

I think ideologically this is incorrect. As the author said "the background layer is made for backgrounds".
However you seem to have some bug on your end that would be fixed by this, so I don't think this should be ignored.
What about adding a flag that lets you pick the layer for the wallpaper to be in?
That way, people can fix their de / wm's bug without us all having to switch to a default that makes less sense.

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.

3 participants