D12218: Remove Reload button from the file dialogs' toolbar

Nathaniel Graham noreply at phabricator.kde.org
Mon Apr 16 17:49:39 BST 2018


ngraham added a comment.


  In D12218#247400 <https://phabricator.kde.org/D12218#247400>, @markg wrote:
  
  > As i said, it's all relative. I barely use any of the buttons but the one i do use is refresh.
  >  I only need it when i'm impatient (for instance when wanting to click on a file that is still being copied)
  
  
  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 [F5]. :)
  
  > or when i'm browsing a slow network drive.
  
  The Reload functionality is still available via the [F5] 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 [F5].
  
  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 //general user// 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).

REPOSITORY
  R241 KIO

REVISION DETAIL
  https://phabricator.kde.org/D12218

To: ngraham, #frameworks
Cc: markg, broulik, rkflx, #dolphin, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20180416/81e1129e/attachment.htm>


More information about the kfm-devel mailing list