ngraham added a comment.

  In D12077#244340 <https://phabricator.kde.org/D12077#244340>, @rkflx wrote:
  
  > Okay, did look at the code ;) But please get at least one extra approval 
from someone elseā€¦
  
  
  Yep, was planning on it.
  
  In D12077#244339 <https://phabricator.kde.org/D12077#244339>, @rkflx wrote:
  
  > Hm, seems like the width of the places panel is set in `kdeglobals` (and 
too small, BTW)
  
  
  Expect another patch to improve this  at some point soon. :)
  
  > As for Sorting: If a user does not click on the Configure button which then 
has "Sorting" in plain sight, why would he click on a Sorting icon? The problem 
is not that sorting is too hidden, but that users don't get the idea to 
actually click on any button in the toolbar. Moving sorting does not help at 
all with that.
  
  Well a button with text on a toolbar is more visible than a sub-menu hidden 
behind a menu hidden behind a text-less button in the corner that currently has 
an inappropriate icon. But this is a more general problem with KDE-style 
ToolButtons that...
  
  - Don't visibly look like buttons until you hover over them
  - Don't have text, just an icon
  - Use a small abstract monochrome line-art icon
  
  These buttons don't really communicate "Click me, I'm a button!" like they 
should. I plan to start a VDG conversation on the subject eventually.
  
  > For the view modes it's different, because users recognize the icon from 
Dolphin. I'd just add the view modes, and IMO sorting in the Detailed View is a 
good enough compromise. We could think about making this mode the default, 
because the horizontal scrolling of Short View is kinda difficult to use anyway.
  
  Agreed. I was also planning to change the default for Short View to put the 
icons on top rather than on the side, to match Dolphin's default behavior. 
That's gated behind a couple of bugs with that view mode that I have to fix 
first though. Details View as the general default also makes sense for me once 
D11993 <https://phabricator.kde.org/D11993> lands (without that, it's often 
almost unusable).

REPOSITORY
  R241 KIO

BRANCH
  show-view-mode-toggles-on-open-save-dialog-toolbar (branched from master)

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

To: ngraham, #frameworks, rkflx
Cc: rkflx, mmustac, broulik, michaelh, ngraham, bruns

Reply via email to