[dolphin] [Bug 455513] Tiny icons in Dolphin tree and in menu dropdown

[gianluca.pettinello at gmail.com]

https://bugs.kde.org/show_bug.cgi?id=455513

--- Comment #9 from gianluca.pettinello at gmail.com <g_pet at hotmail.com> ---
Hello Nate,

I found the root cause:


it is in kstyle/breezestyle.cpp

***
//______________________________________________________________
int Style::pixelMetric(PixelMetric metric, const QStyleOption *option, const
QWidget *widget) const
{
    // handle special cases
    switch (metric) {
    case PM_MenuHMargin:
    case PM_MenuVMargin:
        return Metrics::MenuItem_HighlightGap;

    // small icon size
    case PM_SmallIconSize:
        if (isTabletMode()) {
            return 22;
        } else {
            return 16; //<-- here is the root cause
        }
***

This override causes small icons to be insensitive to a change in size in
system settings.
I'm not a programmer but I think that forcing these numbers is not elegant.
I suggest to first read the size and then increase by one level for tablet mode
instead of fixing absolute values
What  do you think?
Regards
Gianluca

-- 
You are receiving this mail because:
You are on the CC list for the bug.