----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121915/#review73478 -----------------------------------------------------------
daemon/powerdevilbackendinterface.cpp <https://git.reviewboard.kde.org/r/121915/#comment51141> emit lidClosedChanged(d->isLidClosed); ? Also you might want to check for changes in the property, so you don't emit too much. - Aleix Pol Gonzalez On Jan. 8, 2015, 11 a.m., Daniel Vrátil wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/121915/ > ----------------------------------------------------------- > > (Updated Jan. 8, 2015, 11 a.m.) > > > Review request for Plasma. > > > Repository: powerdevil > > > Description > ------- > > There's no way to detect when lid has been closed other than listening to > `changed` signal org.freedesktop.UPower and then polling the Powerdevil for > new values. This patch adds a new signal to the PowerDevil interface to > notify about the change and provide new value right away. Makes it much > easier to use. > > > Diffs > ----- > > daemon/org.kde.Solid.PowerManagement.xml 53f77e5 > daemon/powerdevilbackendinterface.h 702b66b > daemon/powerdevilbackendinterface.cpp 6dd8c71 > daemon/powerdevilcore.h e255703 > daemon/powerdevilcore.cpp d51ab19 > > Diff: https://git.reviewboard.kde.org/r/121915/diff/ > > > Testing > ------- > > Tested with qdbus-monitor, signal is emitted when laptop lid is closed/opened. > > > Thanks, > > Daniel Vrátil > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
