Bug 15824

Summary: Default KDE width for icon on panel is too big (256)
Product: Mageia Reporter: Daniel Napora <napcok>
Component: RPM PackagesAssignee: Luc Menut <lmenut>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: Normal CC: doktor5000, mageia, marja11
Version: Cauldron   
Target Milestone: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Source RPM: mageia-kde4-config CVE:
Status comment:

Description Daniel Napora 2015-05-02 16:33:10 CEST
Default panel icon size is set to 256, should be probably 32.

You can see this by installing plasma-applet-homerun-kicker, and adding it to panel.

Default settings comes from:
http://gitweb.mageia.org/software/desktop/kde/config/tree/kde4-profiles/common/share/config/kdeglobals



Reproducible: 

Steps to Reproduce:
Comment 1 Daniel Napora 2015-05-02 16:54:29 CEST
*** Bug 15825 has been marked as a duplicate of this bug. ***
Comment 2 David Walser 2015-05-03 18:53:11 CEST
I've not seen it use 256, that sounds crazy.  Even on test installs at work I get 32.  Maybe a screenshot would help here too.

CC: (none) => mageia
Assignee: bugsquad => lmenut

Comment 4 Rémi Verschelde 2015-05-03 19:38:07 CEST
I haven't seen that either, and the screenshot shown in the forum topic linked in the errata is using a custom theme and mentions that it's how it looked like after an upgrade.

Florian, could you comment some more about this issue so that we can have a better understand of it?

CC: (none) => doktor5000

Samuel Verschelde 2015-05-03 20:11:43 CEST

Summary: Default KDE panel icon size to big (256) => Default KDE panel icon size too big (256)

Comment 5 Marja Van Waes 2015-05-03 20:22:28 CEST
I tried to reproduce it in cauldron, when napcok told me about the issue. 
The only odd thing I saw was, that I could not add Homerun Kicker directly to the panel. I had to put it on my desktop first (where it already had a small size), and then drag it to the panel.

CC: (none) => marja11

Comment 6 Daniel Napora 2015-05-03 20:45:32 CEST
I reproduced it many times. It also affects Mageia 4 - see workaround in errata.

You can see it here: https://youtu.be/RX4VxDYIVCw
Comment 7 Samuel Verschelde 2015-05-03 20:58:59 CEST
Oh, it's not about icon size after all, it's about the width it takes in the panel?
Comment 8 Rémi Verschelde 2015-05-03 21:01:33 CEST
(In reply to Daniel Napora from comment #6)
> I reproduced it many times. It also affects Mageia 4 - see workaround in
> errata.
> 
> You can see it here: https://youtu.be/RX4VxDYIVCw

Thanks, the video is helpful to see how the bug should be reproduced. I can't reproduce it on cauldron locally, even when adding the Homerun launcher to my taskbar. I'll try to test in a clean VM.
Comment 9 Daniel Napora 2015-05-03 21:04:47 CEST
(In reply to Samuel VERSCHELDE from comment #7)
> Oh, it's not about icon size after all, it's about the width it takes in the
> panel?

Yes, but source of this problem is line from http://gitweb.mageia.org/software/desktop/kde/config/tree/kde4-profiles/common/share/config/kdeglobals

[PanelIcons]
Size=256
Comment 10 Marja Van Waes 2015-05-03 21:10:19 CEST
I can't see the video, but understand now that this is about the additional space this starter (and some others, too) claims on the panel: 

yes, I have that, too
Comment 11 Marja Van Waes 2015-05-03 21:42:54 CEST
I edited 
/var/lib/mageia/kde4-profiles/common/share/config/kdeglobals

and changed 

  [PanelIcons]
  Size=256

into

  [PanelIcons]
  # Size=256
  Size=16

after logging out and back in, the width the icon claims is normal.

However, there is a downside:
applets that need more space on the panel, like plasma-applet-yawp don't get the space they need.

Summary: Default KDE panel icon size too big (256) => Default KDE width for icon on panel is too big (256)

Comment 12 Nicolas Lécureuil 2015-05-03 22:09:27 CEST
do we need to fix a size ?
Comment 13 Marja Van Waes 2015-05-03 22:13:36 CEST

(In reply to Marja van Waes from comment #11)

> 
> after logging out and back in, the width the icon claims is normal.
> 
> However, there is a downside:
> applets that need more space on the panel, like plasma-applet-yawp don't get
> the space they need.

It isn't a problem, I should  have remembered to first put plasma-applet-yapw on the desktop, and then move it to the panel.


(In reply to Nicolas Lécureuil from comment #12)
> do we need to fix a size ?

Here it seems that 16 is fine, even if you add a large applet on the panel
Comment 14 Rémi Verschelde 2015-05-03 22:19:32 CEST
The default size of my taskbar icons seem to be 32x32, so I guess 32 would be a good replacement. 16px is really small on a 1920x1080 screen.
Comment 15 Nicolas Lécureuil 2015-05-03 22:23:41 CEST
the pb by fixing default size is that it breaks applet needing a little more space.

I removed this for the moment.

Status: NEW => RESOLVED
Resolution: (none) => FIXED

Comment 16 Florian Hubold 2015-05-04 02:15:59 CEST
(In reply to Nicolas Lécureuil from comment #15)
> the pb by fixing default size is that it breaks applet needing a little more
> space.

Why is the default size/width set to 256? I'd consider it a bug if an applet will not even allocate the minimum size it needs for itself (plus a little room to the left and the right) but using 256 seems totally overkill.
Can you please provide an example of an applet that requires 256px minimum height/width and can not allocate it by itself?

But removing it completely also seems wrong. Why not set it to 32px?