<table><tr><td style="">ngraham accepted this revision.<br />ngraham added a reviewer: Dolphin.<br />ngraham added a comment.<br />This revision is now accepted and ready to land.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D12389">View Revision</a></tr></table><br /><div><div><p>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.</p>
<p>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.</p>
<p><a href="https://phabricator.kde.org/tag/dolphin/" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">#dolphin</a> folks, does this look sane?</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R241 KIO</div></div></div><br /><div><strong>BRANCH</strong><div><div>dolphin_thumbs (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D12389">https://phabricator.kde.org/D12389</a></div></div><br /><div><strong>To: </strong>anemeth, Frameworks, VDG, ngraham, Dolphin<br /><strong>Cc: </strong>ngraham, Frameworks, michaelh, bruns<br /></div>