Bug 301877

Summary:	battery plasmoid icon does not scale below a certain size
Product:	[Unmaintained] plasma4	Reporter:	Dave Lepore <dave.lepore>
Component:	widget-battery	Assignee:	Plasma Bugs List <plasma-bugs>
Status:	RESOLVED FIXED
Severity:	normal	CC:	arthur, d.cermak, francisbrwn9, kde, kde, mad.f3ka, markg85, rdieter, romain.perier, russianneuromancer, scarpino
Priority:	NOR
Version:	4.8.90 (beta2)
Target Milestone:	---
Platform:	Arch Linux
OS:	Linux
Latest Commit:	https://projects.kde.org/projects/kde/kde-workspace/repository/revisions/14d57a26d9c486d2bc291e103a66744b930bd8e9	Version Fixed In:	4.9
Sentry Crash Report:
Attachments:	explanatory screenshot

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.