Bug 156256 - Battery widget in panel don't show percentage
Summary: Battery widget in panel don't show percentage
Status: RESOLVED FIXED
Alias: None
Product: plasma4
Classification: Plasma
Component: widget-battery (show other bugs)
Version: unspecified
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-01-20 19:52 UTC by Florian Sievert
Modified: 2008-01-27 11:25 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments
Small patch fixing the problem mentioned in the report. (676 bytes, patch)
2008-01-20 21:01 UTC, Florian Sievert
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Florian Sievert 2008-01-20 19:52:05 UTC
Version:            (using Devel)
Installed from:    Compiled sources
Compiler:          gcc (GCC) 4.1.2 20070925 (Red Hat 4.1.2-33) 
OS:                Linux

Add a "battery widget" to the panel. If you move the mouse cursor over the applet, it will show the current battery status correctly. Configure applet, checking "show the charge percentage on the battery".

I expected that the hint shown while hovering the widget is permently shown over the battery. Actually nothing seems to be displayed.

If you add a battery widget on the desktop and checking the same option, it works like expected.
Comment 1 Florian Sievert 2008-01-20 21:01:14 UTC
Created attachment 23164 [details]
Small patch fixing the problem mentioned in the report.

I am a complete newbie to things like this, however this patch solves the
problem. Orgin seems that vertical and horizontal panes are not allowed to
display the charage percentage as text. However, I am not sure why the autor
tries to avoid this - he might have his reasons!

Maybe the whole formFactor-check should be removed, so the text is always
displayed, if the size is big enough (currently 44 pixel)?
Comment 2 Sebastian Kügler 2008-01-26 21:20:16 UTC
Cannot reproduce this, it looks like it's fixed by different formfactor handling. Removed the offending code anyway, don't think that makes sense.
Comment 3 Florian Sievert 2008-01-27 11:25:33 UTC
Confirming. Revision 766875 seems to fix the problem. It is working now for me also. thanks.