broulik created this revision.
broulik added a reviewer: Plasma.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
broulik requested review of this revision.

REVISION SUMMARY
  The model was changed a while ago to return a QString if possible to have 
icon processing done in the delegate QML on-demand. This broke the drag 
helper's drag pixmap, however, as the QML would pass in `model.decoration` 
which might be a `string`.

TEST PLAN
  5.12
  
  - Have proper drag pixmaps for all entries again
  - Custom icons (e.g. favicon) still work

REPOSITORY
  R112 Milou

REVISION DETAIL
  https://phabricator.kde.org/D14545

AFFECTED FILES
  lib/draghelper.cpp
  lib/draghelper.h

To: broulik, #plasma
Cc: plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, mart

Reply via email to