Skip to content

Only use KIconLoader on Plasma

Nicolas Fella requested to merge work/nonplasmaicons into master

On Plasma QIcon::fromTheme is backed by KIconLoader, however on other desktops a different icon loader is used by Qt, which sometimes behaves differently.

Using KIconLoader directly here means that there can be a behavior difference between icons loaded by QIcon::fromTheme and Kirigami.Icon.

This leads to unexpected surprises so we better avoid that.

Merge request reports