Skip to content
This repository has been archived by the owner on Jul 11, 2023. It is now read-only.

Add support for KDE Plasma DE with fix when the script worked only once #3

Open
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

jiriks74
Copy link

I added support for KDE Plasma in a similar way as in #2 . But I'm still creating this pull request, as I fixed the issue mentioned in the said pull request, where the script would set the wallpaper only once.

I was able to find that KDE probably caches the wallpaper, so when the script downloads new and sets it, KDE ignores it or uses a cached one. I was able to get rid of this by creating a blank file, setting it as wallpaper temporarily and then setting the new wallpaper.

I also deleted the build directory, as it should be in .gitignore IMO.

Hope this works for you as well.

@jiriks74
Copy link
Author

Don't worry about the other commits. I left config for my pc and subreddits, some leftover files from vscode and build files. The fix itself is 8b08bc0

@jiriks74 jiriks74 changed the title Adding support for KDE Plasma DE Add support for KDE Plasma DE with fix when the script worked only once Jul 26, 2021
@jiriks74
Copy link
Author

@roudra-das

Hi, i saw your comment in the other pull request. I have a working version here, try it out.

@jiriks74
Copy link
Author

I just added a check where if user deleted the blank file or this script was used before the patch it now creates the blank file
(it checked for the wallpaper image path not for the blank file path)

Signed-off-by: Jiří Štefka <jiri@stefka.eu>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant