<table><tr><td style="">rkflx added a comment.
</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/D8332" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>This change caused a little bit of fallout for #Gwenview. Apparently the review focussed more on the code, but less so on the behaviour in users of the class. I'm not complaining, but given one of our focus goals is on usability and quality of the basic apps, it would be great if:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">changes were tested more broadly in the future in addition to only looking at the code</li>
<li class="remarkup-list-item">there was some help to fix the fallout</li>
</ul>

<p>Please head over to <a href="https://bugs.kde.org/show_bug.cgi?id=387824" class="remarkup-link" target="_blank" rel="noreferrer">https://bugs.kde.org/show_bug.cgi?id=387824</a> if you can help Gwenview, thanks!</p>

<hr class="remarkup-hr" />

<p>In addition to Gwenview I also looked on lxr and did some testing based on what I found:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">The sidebar in <tt style="background: #ebebeb; font-size: 13px;">KDirSelectDialog</tt> is now awful to use, because<ul class="remarkup-list">
<li class="remarkup-list-item">The devices entry (which for some users is more important/useful/frequently used than the search entries) is hidden from view (bad) and requires scrolling (annoying). → We should discuss reordering or (even better) adding collapsing and then collapsing some groups by default.</li>
<li class="remarkup-list-item">The additional scrollbar makes the sidebar so small that you can't read most of the entries. → Add splitter and improve default width.</li>
</ul></li>
<li class="remarkup-list-item">To a lesser extent, this also applies to the normal file dialog (no scrollbar by default would be nice).</li>
<li class="remarkup-list-item">Filesystem sidebars in <a href="https://phabricator.kde.org/tag/kdevelop/" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">#KDevelop</a>, #Okteta, <a href="https://phabricator.kde.org/tag/kile/" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">#Kile</a>, <a href="https://phabricator.kde.org/tag/kate/" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">#Kate</a> and <a href="https://phabricator.kde.org/tag/krusader/" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">#Krusader</a>: Some of the entries do not make sense in some of those apps at all, e.g. Videos/Images/…. Also some entries show an error message or are broken / show nothing at all (while the same entry works fine in Dolphin).</li>
<li class="remarkup-list-item">That's it at first sight, luckily ;)</li>
</ul>

<p>Would be nice to fix those too… Let me know what's the best way forward here, i.e. what are generic issues and where we'd need bugs filed against individual apps.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R241 KIO</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D8332" rel="noreferrer">https://phabricator.kde.org/D8332</a></div></div><br /><div><strong>To: </strong>renatoo, Frameworks, Dolphin, KDE Applications, dvratil, VDG, ngraham, ervin, mlaurent, dfaure, mwolff<br /><strong>Cc: </strong>rkflx, mwolff, dfaure, ervin, usta, mlaurent, dvratil, ngraham, Frameworks<br /></div>