<table><tr><td style="">ngraham added inline comments.
</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/D25856">View Revision</a></tr></table><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D25856#inline-145880">View Inline</a><span style="color: #4b4d51; font-weight: bold;">meven</span> wrote in <span style="color: #4b4d51; font-weight: bold;">kdialog.cpp:865</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">The trash recentlyused tags and desktop ioslaves can't be save filename' folder since they are not writable.<br />
So I believe this should be undone here.</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;"><tt style="background: #ebebeb; font-size: 13px;">kdialog</tt> can be used to show open dialogs as well as save dialogs, where writability is not an issue. Also, the default dialog will show these ioslaves and let you try to write into them, so I believe if we want to do that, they should be filtered out there in the KIO places model, rather than here.</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R229 KDialog</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D25856">https://phabricator.kde.org/D25856</a></div></div><br /><div><strong>To: </strong>ngraham, Frameworks, Dolphin<br /><strong>Cc: </strong>meven<br /></div>