I think I did this change the wrong way. What we really need is a way to use different sizes for different panel widgets by default without making it impossible for users to configure the size.
- Group Reviewers
- R119:9705414e3ec3: Revert "[showdesktop][minimizeall] Reduce the maximum panel icon size"
Some users complained about the buttons being smaller or the size not being configurable when it previously was.
Whatever you change some users will always complain.
It's important not to knee-jerk react based on some comments if they go against the general trend.
Code-wise it's no worse than before your patches, so I'm happy to accept, but if you want to keep it as-is, I would support that too.
I was hesitant about making this change before I committed it, so it's not a knee jerk reaction, but I also don't think it's important for me to land this quickly. I might want to see if I can come up with the real solution.
That doesn't help the huge fraction of people who never touch the default settings though. What's especially silly IMO is that the default panel height is 36px, at which size everything with a colorful icon version gets the colorful version... but if you reduce the panel height just one tick to 34px, the icons get smaller and everything becomes monochrome and it all looks great.
I feel like we should either slightly reduce the default height of the panel to 34px, or change its icon sizing logic so that the threshold for everything turning colorful happens at some size that's greater than 36px.
Why the Notifications, Volume, Network, etc. icons are monochrome, but "Show desktop" and "Minimize all" icons is hard to make a monochrome?
Colored icon of "Show desktop" and "Minimize all" is bigger that other monochrome icons.
Those are in the System Tray, which always shows monochrome icons. Show Desktop is a standalone widget, which has logic to switch its icon between monochrome and colorful icons depending on the size of the container that the icon is inside.
It doesn't contain that logic in the code, it's just only capable of getting icons from the icon theme, for whatever reason. The icon theme happens to have monochrome icons at 16 & 22px and color at 32 & 64 px.
Maybe we could add an option to force a specific icon size. It would be literally the only option in the widget, but it would do the job and we could set a monochrome icon size by default. These applets should probably still get refactored some day though.
It seems like setting specific sizes for specific widgets should be a feature of the panel editing mode though. That way people could set any size for any widget. I don't think our current way of allowing users to set the global panel icon size is very discoverable and it's quite limiting.
I hope it doesn't come to that, add an option is a very KDE3 solution.
that icon kinda needs to be smaller when it's placed in its intended default position.
perhaps not when is somewhere else, i don't know
but this revert makes the default just appear... broken