----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106691/#review19815 -----------------------------------------------------------
Actually, I don't think this works. Better to re-disable dpms on return from idle - I think disabling DPMS just wakes the screen. I know I should have tested this on the real desktop, but there was yet another dpms-related issue that was blocking testing of this one. Sorry. I'll investigate further and update this when I have something that works. - Oliver Henshaw On Oct. 2, 2012, 6:03 p.m., Oliver Henshaw wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/106691/ > ----------------------------------------------------------- > > (Updated Oct. 2, 2012, 6:03 p.m.) > > > Review request for Solid. > > > Description > ------- > > Prevent button action from always enabling dpms > > DPMS needs to be enabled before a call to DPMSForceLevel, but it should not be > left on if it was previously disabled. Fix this in > PowerDevilDPMSAction::triggerImpl(), which is only called by the "Turn off > screen" button action. > > > Diffs > ----- > > powerdevil/daemon/actions/dpms/powerdevildpmsaction.cpp > a16bf7ee254b9a40c6f033c93e5ca63226469e6e > > Diff: http://git.reviewboard.kde.org/r/106691/diff/ > > > Testing > ------- > > Tested interaction of powerbutton action and timed screen powersaving in > 4.9.x VM. > > Note: used virtual cirrus graphics, which claims to implement dpms but > doesn't blank the screen - I used 'xset -q' to view dpms state. > > > Thanks, > > Oliver Henshaw > >
_______________________________________________ Kde-hardware-devel mailing list Kde-hardware-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-hardware-devel