[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