<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><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#250194" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">D12333#250194</a>, <a href="https://phabricator.kde.org/p/abetts/" style="
border-color: #f1f7ff;
color: #19558d;
background-color: #f1f7ff;
border: 1px solid transparent;
border-radius: 3px;
font-weight: bold;
padding: 0 4px;">@abetts</a> wrote:</div>
<div style="margin: 0;
padding: 0;
border: 0;
color: rgb(107, 116, 140);"><p>I gave a +1 to the original idea because I feel that there isn't really much closeness that you can achieve with the open dialog. It is simple, straightforward.</p></div>
</blockquote>
<p>I find the pre-patch design also very streamlined.</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>If we wanted to do a strict fitt's law follow, then each back and forth icon would be next to each of the folders and not in a toolbar.</p></blockquote>
<p>Let's not throw the baby out with the bath water. Having the navigation buttons right above the breadcrumb bar is still better the having them crammed in the top left corner.</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>But we also have to see that there is stronger merit in organizing and looking for symmetry. Symmetry will sometimes help a user more than Fitt's guidelines.</p></blockquote>
<p>This statement may be true in general, but does not fit in our case, as I don't see much symmetry here, TBH. Panel and item view occupy most of the height, and thus are the most important elements. They are split asymmetrical in the horizontal direction, and rightly so. Adding a centered toolbar goes against this and breaks up the split, adding even more asymmetry.</p>
<p>Furthermore, users are able to hide the places panel. With the current design the spatial relation between buttons and item view stays constant, while with a centered approach it will jump around.</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>get user feedback.</p></blockquote>
<p>The file dialog is an important piece of the entire UI. We cannot endlessly change things back and forth, we have to get it right by logical thinking the first time (in this case it would even be the third time!). In KDE3 it was centered, based on feedback from a usability expert it was changed to the current design. In terms of <em>user</em> feedback, I'm not aware of any huge issue users have with it right now.</p>
<p>Do you want to change things around again until there <em>actually</em> is negative feedback? I'd agree with basing design decisions on user feedback if we had better telemetry or did some eye tracking studies, but our current method of hearing user's opinions over on Reddit and Bugzilla is not very effective in painting a broad picture, IMO.</p>
<hr class="remarkup-hr" />
<p>Anyway, I'll come back to this Diff once the Sort button is in and we decided on a final default dialog size. This should make the decision easier, because after all the main motivation was to create space for more buttons.</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>abetts, jtamate, broulik, anemeth, rkflx, michaelh, bruns<br /></div>