[Breeze] [Bug 455513] Small icons no longer respect the "Small Icon" size configurable in the Icons KCM

[Nate Graham]

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

Nate Graham <nate at kde.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |regression
           Assignee|dolphin-bugs-null at kde.org   |plasma-bugs at kde.org
                 CC|                            |noahadvs at gmail.com,
                   |                            |uhhadd at gmail.com
            Product|dolphin                     |Breeze
            Version|unspecified                 |5.25.3
          Component|general                     |QStyle
             Status|REOPENED                    |CONFIRMED
            Summary|Tiny icons in Dolphin tree  |Small icons no longer
                   |and in menu dropdown        |respect the "Small Icon"
                   |                            |size configurable in the
                   |                            |Icons KCM

--- Comment #10 from Nate Graham <nate at kde.org> ---
Right, that makes sense. The code assumes that a small icon is always 16,
probably because the author didn't know or forgot that the small icon size is
configurable in the Icons KCM. I really dislike this configurability and I
think it leads to people using it as a sort of ad-hoc scaling rather than using
the officially supported methods, which also reduces the pressure to fix the
official method for the cases where it doesn't work properly. In my ideal
world, we'd label that functionality obsolete and delete it entirely.

However in the absence of such a thing happening, we should fix this.

Probably what we should do is change the code to consult the Small Icon size
from the Icons KCM, and when when in Tablet mode, go up one size.

If instead we don't want to fix this, it's time to remove the Small Icons
configurability in the Icons KCM.

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