[Information Panel] Adjust painting icons for high dpi scaling

Authored by broulik on Feb 15 2018, 10:10 AM.



Instead of creating our own KIconLoader (which takes ~5ms here) just use QIcon::fromTheme which already does everything right.

BUG: 390488

Test Plan

Multi-selection before

Multi-selection after

Search before

Search after



Overlays are also correctly painted with D9814

Diff Detail

R318 Dolphin
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.
broulik requested review of this revision.Feb 15 2018, 10:10 AM
broulik created this revision.
apol accepted this revision.Feb 15 2018, 12:26 PM
apol added a subscriber: apol.

Although it should probably have a ::setIcon rather than a ::setPixmap(hugePixmap)

This revision is now accepted and ready to land.Feb 15 2018, 12:26 PM
This revision was automatically updated to reflect the committed changes.

So, what's wrong with KIconLoader? Shouldn't we fix it?

Shouldn't we fix it?

Sure. I started working on that in D6313 but it's quite complex to get right. I need to add tons of overloads taking a "scale" argument while keeping API compatibility, etc.