<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#249819" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D12333#249819</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>Thanks for looking into this. There is one way the "more ergonomic" might have been meant: When navigating in the file view, the distance for the mouse to travel to the navigation buttons is much shorter when they are directly above the file view, i.e. like they are currently in the file dialog, but not in Dolphin.</p></div>
</blockquote>

<p>Ah, this makes a lot of sense.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>Also, if a patch refers to Celeste (see <a href="https://en.wikipedia.org/wiki/Celeste_Lyn_Paul" class="remarkup-link" target="_blank" rel="noreferrer">https://en.wikipedia.org/wiki/Celeste_Lyn_Paul</a>), that's often a good indication that the decisions made there were investigated thoroughly ;)</p></blockquote>

<p>Wow, it's great that we had someone like her working  on KDE stuff. I see she's now at the U.S. Department of Defense now, whoa!</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>I'm not yet giving your patch a -1, but perhaps we should just see how the default dialog size and the toolbar content turn out in the end, maybe it turns out that we won't need to move the toolbar (and can even find a way to change Dolphin to be consistent :D</p></blockquote>

<p>If we change Dolphin, we need to solve the same problem here: the toolbar becomes too small and space becomes precious.</p>

<p>Also IMHO it's a visual and consistency regression; in general toolbars span the whole horizontal width of the window, not just some of it. The presentation becomes muddy and awkward, especially given the lack of separators between elements in the standard breeze style. Things feel jumbled together and it's hard to tell where one thing ends and another begins (a separate issue in Breeze of course).</p>

<p>Perhaps we could make the toolbar span the full width, but align things such that the navigation buttons are always over the view?</p>

<p>Like this:</p>

<p><a href="https://phabricator.kde.org/F5813097" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F5813097: Very crude mockup.png</a></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>