<table><tr><td style="">markg 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/D12218">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/D12218#247481" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">D12218#247481</a>, <a href="https://phabricator.kde.org/p/ngraham/" style="
border-color: #f1f7ff;
color: #19558d;
background-color: #f1f7ff;
border: 1px solid transparent;
border-radius: 3px;
font-weight: bold;
padding: 0 4px;">@ngraham</a> wrote:</div>
<div style="margin: 0;
padding: 0;
border: 0;
color: rgb(107, 116, 140);"><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/D12218#247400" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">D12218#247400</a>, <a href="https://phabricator.kde.org/p/markg/" style="
border-color: #f1f7ff;
color: #19558d;
background-color: #f1f7ff;
border: 1px solid transparent;
border-radius: 3px;
font-weight: bold;
padding: 0 4px;">@markg</a> wrote:</div>
<div style="margin: 0;
padding: 0;
border: 0;
color: rgb(107, 116, 140);"><p>As i said, it's all relative. I barely use any of the buttons but the one i do use is refresh.<br />
I only need it when i'm impatient (for instance when wanting to click on a file that is still being copied)</p></div>
</blockquote>
<p>Not really a valid use case; KDirWatcher should update the view for you automatically, and there's no practical benefit to mashing the reload button. The button isn't there to facilitate OCD. :) To do that, just hit <kbd style="display: inline-block; min-width: 1em; padding: 4px 5px 5px; font-weight: normal; font-size: 0.8rem; text-align: center; text-decoration: none; line-height: 0.6rem; border-radius: 3px; box-shadow: inset 0 -1px 0 rgba(71, 87, 120, 0.08); user-select: none; background: #f7f7f7; border: 1px solid #C7CCD9;">F5</kbd>. :)</p></div>
</blockquote>
<p>Not valid? I'd remove all those buttons on the left side except for refresh :)</p>
<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;">
<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>or when i'm browsing a slow network drive.</p></blockquote>
<p>The Reload functionality is still available via the <kbd style="display: inline-block; min-width: 1em; padding: 4px 5px 5px; font-weight: normal; font-size: 0.8rem; text-align: center; text-decoration: none; line-height: 0.6rem; border-radius: 3px; box-shadow: inset 0 -1px 0 rgba(71, 87, 120, 0.08); user-select: none; background: #f7f7f7; border: 1px solid #C7CCD9;">F5</kbd> as well as the context menu. Are those not enough for this use case? And how common is this, really? I'm not sure that "file manipulation from the open/save dialog on a slow network drive" is a common enough use case to justify taking up space in an extremely constrained UI to show a button for reloading the view, especially once that functionality is available via another GUI method too (it's always available via <kbd style="display: inline-block; min-width: 1em; padding: 4px 5px 5px; font-weight: normal; font-size: 0.8rem; text-align: center; text-decoration: none; line-height: 0.6rem; border-radius: 3px; box-shadow: inset 0 -1px 0 rgba(71, 87, 120, 0.08); user-select: none; background: #f7f7f7; border: 1px solid #C7CCD9;">F5</kbd>.</p>
<p>Ultimately the pressure here will be reduced by making the toolbar editable, so people who really want a visible Reload button can have one. But until then, we need to ask ourselves whether the <em>general user</em> benefits from having this button always visible, more than he or she might benefit from having something or more general usefulness visible in the toolbar (e.g. a dropdown menu button that exposes sorting options).</p></blockquote>
<p>I find refresh (much) more important then the preview toggle (that should imho be under the settings thingy, not as it's own button).<br />
Removing refresh is a mistake in my opinion.<br />
Note that <a href="https://phabricator.kde.org/F5" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">F5</a> is for some users FN + <a href="https://phabricator.kde.org/F5" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">F5</a> in case the user prefers to use the "FN" functions (in my case that is true on one keyboard where the volume keys are hidden under F8, F9 and F10). That's not an argument to either keep or remove it, but just to show that refresh doesn't always have to be a one-key thing. But i consider that to be a "it's your own fault" case as the user then knowingly chose this.</p>
<p>But perhaps we need more opinions.</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/D12218">https://phabricator.kde.org/D12218</a></div></div><br /><div><strong>To: </strong>ngraham, Frameworks<br /><strong>Cc: </strong>markg, broulik, rkflx, Dolphin, michaelh, ngraham, bruns<br /></div>