leinir added a comment.
In https://phabricator.kde.org/D5764#108019, @apol wrote: > Don't we also have to add support for when DesktopIcon::m_source is a QIcon? That's already covered in DesktopIcon::updatePaintNode's big switch statement, where it checks for what type is actually inside m_source (heh, i thought it was in the patch, but that whole switch statement's not changed in the end) :) https://cgit.kde.org/kirigami.git/tree/src/desktopicon.cpp#n256 REPOSITORY R169 Kirigami REVISION DETAIL https://phabricator.kde.org/D5764 To: leinir, mart, apol Cc: plasma-devel, apol