fix: gif & colorscheme persistence #2
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The
~/.local/state/caelestia/wallpaper/path.txt
wasn't actually updating on gif selection, while the CLI get function was, so restarting the shell didn't apply the previously set gif, but instead the last valid static image.is_valid_image()
in the wallpaper.py, from the caelestia-cli-git would need to be updated to also include gifs for it to not error and actually write the path.This PR is really just a quick bypass of the CLI, instead just writing to the .txt for the path, and JSON for the colors.
Actually updating the CLI would be better, but I'm not sure how much that actually gets updated/merged PRs, but this workaround feels a bit like one of those "yeah, just use the CLI" haha
(of course I pulled in the upstream updates just because I merged yours in to a clean branch lol)