Panel icon sizing bug in 4.4

[todd rme]

On Sun, May 23, 2010 at 9:29 PM, Mike Kasick <mkasick-pd at club.cc.cmu.edu> wrote:
> On Sun, May 23, 2010 at 10:28:00AM -0700, Chani wrote:
>
>> one of the bug reports mentioned that KDE's default icon size should be
>> respected, although it's not clear whether that was implemented.
>
> It wasn't.
>
>> if that setting is not followed, then you have found a bug, and patches
>> are welcome.  :)
>
> OK, attached are three fairly trivial patches.
>
> The first patch, against kdebase-runtime (both Debian 4.4.3-2 and KDE 4.5
> SVN trunk) reenables the ability to change the "Panel" icon size in
> kcm_icons.  It also disables the "Animate Icons" checkbox as that checkbox
> is disabled for most of the other icon types, and appers to be obsolete in
> KDE 4.
>
> The second and third patches do the same thing, but the second applies to
> Debian 4.4.3-2 while the third applies to KDE 4.5 SVN trunk.  Both patches,
> against kdelibs, changes IconWidget::sizeHint to return
> IconSize(KIconLoader::Panel) instead of KIconLoader::SizeMedium when no
> preferred icon size has already been set (4.5 only).  Note that
> KIconLoader::Panel defaults to KIconLoader::SizeMedium, so there's no
> change unless someone modifies the setting in the Appearance dialog.
>
> I've tested the first two patches against Debian 4.4.3-2 and they resolve
> the problem, allowing the user to configure the capped panel icon size.
> plasma-desktop has to be restarted for a new capped icon size to take
> effect, since IconWidget::sizeHint is only queried on applet
> initialization.  The effect is also immediate (without plasma-desktop
> restart) on any new applet added to the panel.
>
> I've not tested the patches against KDE SVN trunk, I've just ensured they
> apply cleanly.  Unfortunately I'm not setup to build and test trunk right
> now, if someone else can try these patches that would be great.  If you do
> a partial build, the modified output files are kcm_icons.so and
> libplasma.so.3.0.0.
>
> So actually, I haven't confirmed that the icon size cap problem exists in
> KDE 4.5.  I suspect it does, since it's definitely not user configurable
> right now.  It looks like a bunch of code was added to IconWidget::sizeHint
> to allow individual applets to set a preferred icon size.  However the
> "icon" and "kickoff" applets appear not to touch this.
>
> Are the first two patches fair game for KDE 4.4.4?  I'd really love to see
> a user configurable panel icon size in Debian for accesibility purposes,
> and 4.5 is going to hit too late for squeeze.
>
> I'm assuming the first and third patches are appropriate for KDE 4.5.
> Again, they're fairly trivial, but need to be tested.
>
> Thanks everyone!


I apologize of this is a silly question, or already discussed, but if
the problem is the width of vertical panels, why don't we just have
the size hint depend on whether the panel is vertical or horizontal?
So for horizontal panels it resizes freely, but for vertical panels it
only goes up to the maximum size specified?  Wouldn't that fix the
problems with the vertical panel while not making the horizontal panel
any more difficult to use?

-Todd