Bug 301877 - battery plasmoid icon does not scale below a certain size
Summary: battery plasmoid icon does not scale below a certain size
Status: RESOLVED FIXED
Alias: None
Product: plasma4
Classification: Plasma
Component: widget-battery (show other bugs)
Version: 4.8.90 (beta2)
Platform: Arch Linux Linux
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords:
: 301885 302034 (view as bug list)
Depends on:
Blocks:
 
Reported: 2012-06-14 06:25 UTC by Dave Lepore
Modified: 2012-06-20 10:39 UTC (History)
11 users (show)

See Also:
Latest Commit: https://projects.kde.org/projects/kde/kde-workspace/repository/revisions/14d57a26d9c486d2bc291e103a66744b930bd8e9
Version Fixed In: 4.9

Attachments
explanatory screenshot (8.21 KB, image/png)
2012-06-14 06:26 UTC, Dave Lepore 		Details
View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Dave Lepore 2012-06-14 06:25:10 UTC 
starting from 4.9 beta 2 the battery plasmoid icon does not scale correctly. 

Reproducible: Always

Steps to Reproduce:
1. update to 4.9 beta 2
2. put the battery plasmoid in system tray
3. reduce the size of the panel until the battery icon stops scaling
Actual Results:  
the icon does not scale

Expected Results:  
the icon should scale as in kde 4.8.* and 4.9 beta 1
Comment 1 Dave Lepore 2012-06-14 06:26:48 UTC 
Created attachment 71816 [details]
explanatory screenshot
Comment 2 Dominik Cermak 2012-06-14 07:15:44 UTC 
Confirmed.
Comment 3 Nikolay Rysev 2012-06-14 07:31:10 UTC 
Same here. Also, there are was power management on-off slider in beta1, which is now replaced by a checkbox (why?).
Comment 4 Romain Perier 2012-06-14 07:47:13 UTC 
I can confirm the bug, the icon size is wrong, it's not scaled correctly and sometimes it is outside the systray.
Comment 5 David Edmundson 2012-06-14 09:32:03 UTC 
*** Bug 301885 has been marked as a duplicate of this bug. ***
Comment 6 Christian (Fuchs) 2012-06-14 17:01:47 UTC 
Also confirmed on Gentoo, 4.9 Beta 2, was fine in Beta1.
Comment 7 FBrown 2012-06-16 15:29:38 UTC 
Fixed in this revision I think?

https://projects.kde.org/projects/kde/kde-workspace/repository/revisions/14d57a26d9c486d2bc291e103a66744b930bd8e9

Applied those changes, and icon here scales correctly again.
Comment 8 Andrea Scarpino 2012-06-16 15:34:40 UTC 
(In reply to comment #7)
> Fixed in this revision I think?
> 
> https://projects.kde.org/projects/kde/kde-workspace/repository/revisions/
> 14d57a26d9c486d2bc291e103a66744b930bd8e9

Confirmed. Fixed!
Comment 9 Christian (Fuchs) 2012-06-16 15:39:28 UTC 
Fixed, nice thing of qml: can be done on the fly, no compilation needed. 

Thanks :)
Comment 10 Mark 2012-06-19 22:48:13 UTC 
It might be fixed, but the minimum size is now set as 32x32. I often have a smaller panel then that. Could the minimum constraint just be removed? or 16x16 as minimum.
Comment 11 Mark 2012-06-19 22:53:30 UTC 
*** Bug 302034 has been marked as a duplicate of this bug. ***
Comment 12 Rex Dieter 2012-06-20 04:11:24 UTC 
look closer at the patch, the size limitation sholdn't apply to system tray.

confirmed fixed, thanks.
Comment 13 Mark 2012-06-20 10:39:58 UTC 
(In reply to comment #12)
> look closer at the patch, the size limitation sholdn't apply to system tray.
> 
> confirmed fixed, thanks.

I just take your word for it :) I haven't actually tried it yet.