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

Reset timers after inhibitor has been taken off if idled #72

Merged
merged 1 commit into from
Jun 22, 2024
Merged

Reset timers after inhibitor has been taken off if idled #72

merged 1 commit into from
Jun 22, 2024

Conversation

Elvyria
Copy link
Contributor

@Elvyria Elvyria commented Jun 22, 2024

This addresses #26.
Got annoyed with monitor staying on after video is over. It would be better to use idle-inhibit-unstable-v1 with a fake surface and let Hyprland just reset and halt timers, but I'm not writing that. The conditions shouldn't trigger too often anyways, so with <10 hooks on average it should be harmless.

Tested for about 2 days, but I don't do c++ and wayland-proto, so check for sanity please. If you don't like that copy-paste and want me to create clean methods for clearing notifications and re-registering them separately, do tell.
(and if someone manages to hot-reload config in an idle state, that's on them)

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.

thanks!

@vaxerski vaxerski merged commit 7c5747b into hyprwm:main Jun 22, 2024
1 check passed
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