<table><tr><td style="">ngraham 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/D12333">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D12333#249352" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D12333#249352</a>, <a href="https://phabricator.kde.org/p/rkflx/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@rkflx</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>Please add a mockup of how your final vision of the dialog looks like after all changes in  <a href="https://phabricator.kde.org/T8552" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">T8552</a> are done, in particular "Move the File Name field to the top of the window" and any toolbar additions. Otherwise it's really hard to judge an individual change like this.</p>

<p>Also, as more and more things are added to the places panel, it might get a bit too short in the end, requiring too much scrolling?</p>

<p>Another point would be to research why it looks like it does today. After all, in KDE3 we already had something like you are proposing right now, so changing it back yet again should come with a good reason.</p></div>
</blockquote>

<p>I did some spelunking, and here's where the layout was first changed: <a href="https://cgit.kde.org/kdelibs.git/commit/kfile/kfilewidget.cpp?h=Active/Two&id=1c850c60d4bbe2c838941ea1ffff787a6e4ddb6e" class="remarkup-link" target="_blank" rel="noreferrer">https://cgit.kde.org/kdelibs.git/commit/kfile/kfilewidget.cpp?h=Active/Two&id=1c850c60d4bbe2c838941ea1ffff787a6e4ddb6e</a></p>

<p>It looks like the reason was to make the dialog "look better and more ergonomic". I'm not sure if I agree that making the toolbar not span the width of the top of the window actually accomplished this. I'm with Kai that consistency with Dolphin is important.</p>

<p>it's true that this reduces the amount of space available for Places a bit, but that's a problem Dolphin has too, and it should be alleviated by many of the tasks in <a href="https://phabricator.kde.org/T8552" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">T8552: Polish Open/Save dialogs</a>. Also we could consider making the defauly size a tad bigger. They are really quite small right now.</p>

<p>I'm willing to put "Move the File Name field to the top of the window, like GTK and Mac file pickers" on hold for now, or indefinitely. Your arguments in the meta-task were convincing.</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/D12333">https://phabricator.kde.org/D12333</a></div></div><br /><div><strong>To: </strong>ngraham, Frameworks, Dolphin, VDG<br /><strong>Cc: </strong>broulik, anemeth, rkflx, michaelh, bruns<br /></div>