ngraham accepted this revision.
ngraham added a reviewer: Dolphin.
ngraham added a comment.
This revision is now accepted and ready to land.


  Wow, a truly trivial fix! Since Dolphin is currently where the config UI for 
this lives, I think it makes sense to re-use that information here.
  
  I was worried about the use cases where Dolphin isn't installed (e.g. the 
case of someone using Kate or Gwenview or something in their GNOME or MATE 
environment) but I tested that out that seems to work just fine, and it gets 
the correct set of default plugins. Also works fine in the new user use case 
where Dolphin is installed, but the user opens a file dialog before Dolphin.
  
  #dolphin <https://phabricator.kde.org/tag/dolphin/> folks, does this look 
sane?

REPOSITORY
  R241 KIO

BRANCH
  dolphin_thumbs (branched from master)

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

To: anemeth, #frameworks, #vdg, ngraham, #dolphin
Cc: ngraham, #frameworks, michaelh, bruns

Reply via email to