[Tracker] use 32x32 icons with font size >= 18pt.

[diver]

proof of concept patch

[anevilyak]

Updated version of patch

[anevilyak]

Attached an updated version of the original patch, which modifies the two menu item classes to dynamically calculate their spacing based on an icon_which instead of hardcoding various constants. As far as making this switchable completely on the fly though, a few complications remain. The first question is whether a simple "font size > 18.0" heuristic is good enough for deciding between the smaller vs larger icon size, and also, there may be other places in Tracker that need some adjustments to take that into account, as IconMenuItem allows one to pass in an icon_size, which might be getting used elsewhere in libtracker.

In the long term, it'd be nice to rewrite this entirely to use HVIF to get an icon that's a more precise match for the font size, but that'd involve significant changes to the icon cache and various other pieces.

[waddlesplash]

Following hrev52496 I'm not quite sure what relevance this patch still has, so -- axeld, what should be done here after your change?

[jscipione]

Fixed in hrev52496 (a while back)