Skip to content

Comments

add WakeLockSentinel#6331

Merged
WithoutPants merged 3 commits intostashapp:developfrom
feederbox826:wake-sentinel
Dec 2, 2025
Merged

add WakeLockSentinel#6331
WithoutPants merged 3 commits intostashapp:developfrom
feederbox826:wake-sentinel

Conversation

@feederbox826
Copy link
Collaborator

@feederbox826 feederbox826 commented Nov 27, 2025

prevents screen from sleeping ONLY in secure contexts (localhost, https) while videos are playing and in focus

closes #2884

prevents screen from sleeping ONLY in secure contexts (localhost, https)

closes stashapp#2884
@smith113-p
Copy link
Contributor

I would probably mention in the description that this is specifically while a video is playing

@WithoutPants WithoutPants added the improvement Something needed tweaking. label Nov 28, 2025
@WithoutPants WithoutPants added this to the Version 0.30.0 milestone Nov 28, 2025
@WithoutPants WithoutPants added the needs testing Pull requests that require testing label Nov 28, 2025
@feederbox826
Copy link
Collaborator Author

yes, needs testing please. I couldn't get android, iOS or W11 to timeout and blank out the screen, I'm assuming they have some hooks into background media playback

@discourse-stashapp
Copy link

This pull request has been mentioned on Stash. There might be relevant details there:

https://discourse.stashapp.cc/t/stash-v0-30-release-development-thread/4484/1

@feederbox826
Copy link
Collaborator Author

I think I got this validated by just leaving a scene playing for ~45mins and my phone not locking

release wakelock on dispose and end, call out secure contexts in error message
@WithoutPants WithoutPants merged commit 90dd0b5 into stashapp:develop Dec 2, 2025
2 checks passed
@feederbox826 feederbox826 deleted the wake-sentinel branch December 2, 2025 02:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement Something needed tweaking. needs testing Pull requests that require testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug Report] Device display does not stay awake while viewing content

4 participants