--87fa0ad8dc6a46e6b18c59f13f781c65 Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="ascii" Mime-Version: 1.0 davidedmundson added a comment. > KIconLoader does, but even Qt's own ones QSvgIconEngine and QPixmapIconEngine don't currently. I did fix QPixmapIcon and QSvgIconEngine, This should mean we can change QIcon::fromTheme to check isNull() which will then implicitly solve this. REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D5109 To: broulik, #plasma, hein Cc: rkflx, smartins, ngraham, davidedmundson, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart --87fa0ad8dc6a46e6b18c59f13f781c65 Content-Transfer-Encoding: quoted-printable Content-Type: text/html; charset="ascii" Mime-Version: 1.0
davidedmundson added a comment. | View Revision=
KIconLoader does, but even Qt'= s own ones QSvgIconEngine and QPixmapIconEngine don't currently.
= blockquote>I did fix QPixmapIcon and QSvgIconEngine,
This should mean we can change QIcon::fromTheme to check isNull() which = will then implicitly solve this.