Bug 15824 - Default KDE width for icon on panel is too big (256)
Summary: Default KDE width for icon on panel is too big (256)
Status: RESOLVED FIXED
Alias: None
Product: Mageia
Classification: Unclassified
Component: RPM Packages (show other bugs)
Version: Cauldron
Hardware: All Linux
Priority: Normal normal
Target Milestone: ---
Assignee: Luc Menut
QA Contact:
URL:
Whiteboard:
Keywords:
: 15825 (view as bug list)
Depends on:
Blocks:
 
Reported: 2015-05-02 16:33 CEST by Daniel Napora
Modified: 2015-05-04 02:15 CEST (History)
3 users (show)

See Also:
Source RPM: mageia-kde4-config
CVE:
Status comment:


Attachments

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?

Note You need to log in before you can comment on or make changes to this bug.