Skip to content

Commit

Permalink
src/gui/widgets/mpv/mpvwidget: fix screensaver not inhibiting on windows
Browse files Browse the repository at this point in the history
The screensaver hasn't ever been inhibited on Windows and nobody
noticed. You're all terrible at Japanese. Fixes the issue by calling
SetThreadExecutionState with ES_DISPLAY_REQUIRED instead of whatever
incorrect options were used before.

Closes #185
  • Loading branch information
ripose-jp committed Oct 27, 2023
1 parent d6a3c2e commit 37f101e
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/gui/widgets/mpv/mpvwidget.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -815,7 +815,7 @@ void MpvWidget::preventScreenDimming()
{
#if defined(Q_OS_WIN)
SetThreadExecutionState(
ES_CONTINUOUS | ES_SYSTEM_REQUIRED | ES_AWAYMODE_REQUIRED
ES_CONTINUOUS | ES_DISPLAY_REQUIRED
);
#elif defined(Q_OS_UNIX) && !defined(Q_OS_DARWIN)
QDBusInterface screenSaver(
Expand Down

1 comment on commit 37f101e

@Issue-maker-9000
Copy link

Choose a reason for hiding this comment

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

Who the heck uses a screensaver, and let those Japanese characters burn in to show your true dedication if you have an oled monitor.
This commit has to be reverted IMO!

Please sign in to comment.