anthonyfieroni created this revision. anthonyfieroni added reviewers: mart, broulik, thsurrel, Plasma. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. anthonyfieroni requested review of this revision.
REVISION SUMMARY This is 2nd patch of fixes annoying side effect (D16918 <https://phabricator.kde.org/D16918> was first one), yeah about me it's not a bug but it depend of driver even on display reaction time. I try to wake-up by simulating activity, but that may not enough, so delay screen brightness gives a little bit time to display to react. TEST PLAN What the side effect is: 1. Dimm display and screen energy saving are enabled 2. Let's time activation in configuration of energy saving is longer 3. Unplug adapter 4. Wait to dimming to set screen to 0 5. If now wake-ups screen - all is good screen is not goes to sleep 6. So we wait to screen saving to be activated 7. Now screen sleeps, just plug adapter in makes screen to wake-up 8. If dimm action sets screen brightness too early is just ignored (by driver or display who knows) REPOSITORY R122 Powerdevil REVISION DETAIL https://phabricator.kde.org/D17874 AFFECTED FILES daemon/actions/bundled/dimdisplay.cpp daemon/powerdevilcore.cpp To: anthonyfieroni, mart, broulik, thsurrel, #plasma Cc: plasma-devel, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart