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

[Alexander Kernozhitsky]

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

--- Comment #12 from Alexander Kernozhitsky <sh200105 at mail.ru> ---
> 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.
The code is also Breeze-specific, meaning that if I switch the application
theme to something else, then it will use the icon size configured in the KCM
(which resides in SmallIcons section in ~/.config/kdeglobals).

Also, magic hard-coded values in the source code are usually bad.

I don't really know how Tablet mode is made, but I assume that this fix should
go to System settings or whatever sets the icon sizes. If the Tablet mode is
Breeze-specific itself, then making the icons one step bigger is also a good
solution.

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