[PATCH] Lock/Logout plasmoid: configure shown buttons
Konstantinos Smanis
kon.smanis at gmail.com
Sun Feb 8 15:08:13 CET 2009
HI everybody,
I have created a patch that implements wishlist #164687
(https://bugs.kde.org/show_bug.cgi?id=164687), namely provide the UI to
configure the buttons which are shown in the Lock/Logout plasmoid (Lock,
Logout or both).
I wasn't sure whether I should commit it (plasma is quite a "sensitive" area
that I wouldn't like to mess), so I send it to you, the Plasma devs to review
it. It's quite small btw.
I have followed the coding styles and all these things, but there were some
things that I wasn't sure for. For example, under Windows the logout button is
not shown so the configuration UI should not apply there. Thus I have put all
config-related code in the appropriate #ifndef Q_OS_WIN. Still, I don't really
like this (I am afraid I abused it and spread it all over the code), so feel
free to make suggestions.
Cheers,
Konstantinos Smanis
-------------- next part --------------
A non-text attachment was scrubbed...
Name: lockout.patch
Type: text/x-patch
Size: 6548 bytes
Desc: not available
Url : http://mail.kde.org/pipermail/plasma-devel/attachments/20090208/3509203a/attachment.patch
More information about the Plasma-devel
mailing list