D12077: Show view mode buttons and a sort menu in the open/save dialog's toolbar

[Henrik Fehlauer]

rkflx added a comment.


  I brought this up before, but let me repeat: It is of utmost importance to test with the default settings, i.e. default font size and default dialog size (remove `~/.config/kate*` for that). Your current version does not look good, despite what you show in your screenshot in the summary:
  
  F5803932: kio-toolbar.png <https://phabricator.kde.org/F5803932>
  
  Note that I'm not saying your intention in this patch is wrong. But as there is only so much space available, you need to prioritize what can be shown. By default, there should be no overflow arrow in the toolbar too. Ideas:
  
  - Sorting is irritating, this needs an icon, e.g. `view-sort-descending`, `object-order-back` or an entirely new icon. OTOH I wonder if this entry is needed at all, because for Detailed View you could simply click on the table headers.
  - The slider should get a minimum size.
  - There should still be a good amount of whitespace between what's on the left and on the right side of the toolbar. This separation into groups results in faster navigation in the UI and helps in making it feel less crowded.
  
  Please resist the urge to put everything in the toolbar. Configuring the dialog is //not// a regular task, so using the Configure button is fine too. Only the most important settings should be shown.

REPOSITORY
  R241 KIO

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

To: ngraham, #frameworks
Cc: rkflx, mmustac, broulik, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180411/17e53775/attachment.html>