Bug 189053 - Battery icons are shift rightwards on the battery level monitor widget
Summary: Battery icons are shift rightwards on the battery level monitor widget
Status: RESOLVED FIXED
Alias: None
Product: plasma4
Classification: Plasma
Component: widget-battery (show other bugs)
Version: unspecified
Platform: openSUSE Linux
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-04-07 19:08 UTC by Tamás Németh
Modified: 2009-06-11 17:24 UTC (History)
4 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
battery level monitor widget (29.54 KB, image/png)
2009-04-07 19:08 UTC, Tamás Németh
Details
Another screenshot (496.71 KB, image/png)
2009-04-07 20:15 UTC, Tamás Németh
Details
Widget after re-attaching the second battery (126.39 KB, image/png)
2009-04-07 23:14 UTC, Tamás Németh
Details
Another screenshot (with full batteries) (16.12 KB, image/png)
2009-04-18 19:10 UTC, Jaak Ristioja
Details
Patch for battery.cpp (2.69 KB, patch)
2009-06-03 15:39 UTC, Jaak Ristioja
Details
batterymonitorbug.png (3.59 KB, image/png)
2009-06-04 13:36 UTC, Robert Bredereck
Details
batterymonitorbugperc.png (4.70 KB, image/png)
2009-06-04 13:36 UTC, Robert Bredereck
Details
batterymonitorbugpopup.png (41.04 KB, image/png)
2009-06-04 13:37 UTC, Robert Bredereck
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Tamás Németh 2009-04-07 19:08:06 UTC
Version:            (using KDE 4.2.2)
OS:                Linux
Installed from:    SuSE RPMs

It was OK in KDE 4.2.1, but it's very unaesthetic (and unusable in addition) now in 4.2.2. See the attached image!
Comment 1 Tamás Németh 2009-04-07 19:08:41 UTC
Created attachment 32681 [details]
battery level monitor widget
Comment 2 Dario Andres 2009-04-07 19:17:44 UTC
Does this happen with every battery widget you add ? Also, can you try with putting it inside a Panel.. or changing the Plasma theme?
Thanks
Comment 3 Tamás Németh 2009-04-07 20:15:12 UTC
Created attachment 32684 [details]
Another screenshot

It behaves the same way both on the desktop and the panel, with every theme (some of them are even worse), and regardless of the number of displayed battery icons. (See this image.)
Comment 4 Dario Andres 2009-04-07 20:16:11 UTC
What Plasma theme are you using? Can you try with another one ? Thanks
Comment 5 Tamás Németh 2009-04-07 20:24:16 UTC
Currently oxigen, but I've tried EVERY single theme, and it was broken with each of them for me. I'm using x86_64 openSUSE11.1, and my KDE comes from http://download.opensuse.org/repositories/KDE:/KDE4:/Factory:/Desktop/openSUSE_11.1
Comment 6 Dario Andres 2009-04-07 20:37:05 UTC
Do you have two batteries?
Comment 7 Tamás Németh 2009-04-07 20:49:53 UTC
(In reply to comment #6)
> Do you have two batteries?

Yes, but when I set this widget not to show each of them (that is to show only one icon), then that single icon will be shift rightwards the same way.
Comment 8 Dario Andres 2009-04-07 21:33:28 UTC
That's an interesting detail. Thanks
Comment 9 Tamás Németh 2009-04-07 23:14:50 UTC
Created attachment 32688 [details]
Widget after re-attaching the second battery

OK, I've restarted my machine with a single battery, and it behaved the same way. Then I re-attached the second bettery on the fly (!), and the KDE battery widget immediately recognized it, but it become even more ugly (see the attached image).
Comment 10 Jaak Ristioja 2009-04-18 19:10:39 UTC
Created attachment 32914 [details]
Another screenshot (with full batteries)

This is the exact same bug that I also experienced. Here's a screenshot. I'm on Gentoo Linux, x86-64.

My batteries are actually full, but thats another other bug (batteries showing up empty when 100% charged), I think.
Comment 11 Tamás Németh 2009-05-07 13:14:45 UTC
Still applies to 4.2.3
Comment 12 Jaak Ristioja 2009-06-03 15:39:04 UTC
Created attachment 34241 [details]
Patch for battery.cpp

This is a patch for the plasma workspace file plasma/applets/battery/battery.cpp in KDE 4.2.3 which fixes this bug for me (both vertical and horizontal panels).

Switching whether it shows multiple batteries doesn't work fine, because the applet doesn't resize. Neither could I make it show multiple batteries on the desktop.

PS: I hope you fix this soon.
Comment 13 Robert Bredereck 2009-06-04 13:33:30 UTC
I can confirm this bug, too.

I'm using Jaunty on a "Thinkpad T61" with one system battery and one additional ultrabay battery. They are correctly detected, but:
- the battery symbols are displaced out of the widget's space in control panel (see screenshot batterymonitorbug.png)
- the battery symbols are also discplaced in the popup window of the widget (see screenshot batterymonitorbugpopup.png)

Hints:
- the capacity percentage captions/overlays are displayed correctly (see screenshot batterymonitorbugperc.png)
- the lighting-symbols (showing that main cable is connected) have also correct positions
- with only one battery plugged in everything works fine
Comment 14 Robert Bredereck 2009-06-04 13:36:04 UTC
Created attachment 34259 [details]
batterymonitorbug.png
Comment 15 Robert Bredereck 2009-06-04 13:36:33 UTC
Created attachment 34260 [details]
batterymonitorbugperc.png
Comment 16 Robert Bredereck 2009-06-04 13:37:34 UTC
Created attachment 34261 [details]
batterymonitorbugpopup.png
Comment 17 Artur Souza (MoRpHeUz) 2009-06-11 17:24:35 UTC
This bug is already fixed by Jaak's patch that was commited by aseigo on Revision 977704.