Review Request: Add a tooltip to battery monitor applet

[Sebastian Kügler]

On Thursday 26 November 2009 18:01:19 Aaron J. Seigo wrote:
> On November 26, 2009, Kåre Särs wrote:
> > I also thought about extragear/base/plasma/applets/, but that one is
> > empty (dead?).
> 
> anything that you wish to package, ship and support can go into extragear
>  (and yes, we do need to open a plasma module in extragear again). a forked
>  version won't happen in addons, however. i mean, really, two battery
>  monitors in the default packages?
> 
> in this particular case, i'm ok with a hidden config option. usually i
>  really dislike these things, but it's trivial, will get this issue off my
>  back and will be a nice example later on for why hidden config options do
>  or don't work.

Here's a patch that re-enables this option, with the hidden config option 
"showRemainingTime=true". I've commented out the i18n for the label since we're in 
string freeze.

It basically moves around a bit of code I had disabled temporarily when I reworked 
the layout of the popup and adapts to the better alignment. No tooltip in there, 
however.

Should I commit this one?
-- 
sebas

 http://www.kde.org | http://vizZzion.org |  GPG Key ID: 9119 0EF9 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: battery-applet-showremainingtime.diff
Type: text/x-patch
Size: 8535 bytes
Desc: not available
Url : http://mail.kde.org/pipermail/plasma-devel/attachments/20091127/1c885039/attachment-0001.diff 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 489 bytes
Desc: This is a digitally signed message part.
Url : http://mail.kde.org/pipermail/plasma-devel/attachments/20091127/1c885039/attachment-0001.sig