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

Hypr Ecosystem/hyprpaper: rewrite reload docs #964

Merged
merged 1 commit into from
Feb 5, 2025

Conversation

fxzzi
Copy link
Contributor

@fxzzi fxzzi commented Feb 3, 2025

This PR rewrites and extends the reload keyword hyprpaper documentation.

This section was originally very short, with no example provided and only a very short description, this led to a lot of people missing this option and instead going with the old "unload, preload, set" combo which isn't needed and probably causes issues with the transition.

Examples of people missing this config option is seen in the discord server and hyprpaper issue tracker:

https://canary.discord.com/channels/961691461554950145/1215490894804029450/1335447955339087882

And I also missed it too when i first started using hyprpaper.

I also added a small piece of documentation about how to randomize your wallpaper using the reload keyword. This is in response to hyprwm/hyprpaper#194 where people (including vaxry lmfao) mentioned sed'ing the config and relaunching hyprpaper as a good solution to it. The simple script I wrote is instead much better with no reloading of hyprland being needed. Simply change the wallpaper directory to wherever your ones are stored. (It also checks to make sure it doesn't set the same wallpaper as what is currently applied!)

Let me know if you need any changes or if I should make the script more in depth, i.e. check for only .png, .jpg, .jpeg, or whatever. imo it's fine as it is but I could use another persons opinion.

@fxzzi fxzzi force-pushed the hyprpaper-reload-docs branch from ba6de37 to bb28e58 Compare February 3, 2025 22:12
@fxzzi fxzzi force-pushed the hyprpaper-reload-docs branch from bb28e58 to 2f70059 Compare February 3, 2025 22:13
@vaxerski vaxerski requested a review from fufexan February 4, 2025 21:35
Copy link
Member

@fufexan fufexan left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@fufexan fufexan merged commit d30c163 into hyprwm:main Feb 5, 2025
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.

2 participants