<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/D12333">View Revision</a></tr></table><br /><div><div><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>

<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>

<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 style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>we could consider making the defauly size a tad bigger.</p></blockquote>

<p>Yup, especially the default height should be increased slightly if we decide to go with <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Detailed (Tree) View</span></span></span>.</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>put "Move the File Name field to the top of the window, like GTK and Mac file pickers" on hold</p></blockquote>

<p>Good to know ;)</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>