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

Reply via email to