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 handler "reload" to do a change of wallpaper by one hyprctl exec #173

Merged

Conversation

barrrguzin
Copy link
Contributor

Add handler "reload" to do a change of wallpaper by one hyprctl execution. Handler gets the string with monitor name and path to wallpaper: "MON, /tmp/pic.png"

@vaxerski
Copy link
Member

whats the use though?

@barrrguzin
Copy link
Contributor Author

unload previous picture, load next one and display it on screen by using 3 calls on hyprctl its too long sequence to change the wallpaper. i dont see examples of wallpaper changing script that preload all pictures from folder and than change it by "wallpaper" command, i concluded that the most of people use 3 command sequence in their scripts. so, this handler can be used instead of 3 separated, its more user friendly, and faster

Copy link
Member

@vaxerski vaxerski left a comment

Choose a reason for hiding this comment

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

I guess, alright. Needs a wiki mr, code is fine

@barrrguzin
Copy link
Contributor Author

keyword description added: hyprwm/hyprland-wiki#662

@vaxerski vaxerski merged commit 2c57525 into hyprwm:main May 25, 2024
1 check passed
vaxerski pushed a commit to hyprwm/hyprland-wiki that referenced this pull request May 25, 2024
@barrrguzin barrrguzin deleted the hyprpaper-with-reload-wallpaper-feature branch June 8, 2024 14:36
vaxerski pushed a commit that referenced this pull request Jul 17, 2024
…rctl exec (#173)

* Add handler "reload" to do a change of wallpaper by one hyprctl execution

* fixed contain parameter handling in "handleReload"
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