ngraham planned changes to this revision. ngraham added a comment.
In D12077#245758 <https://phabricator.kde.org/D12077#245758>, @rkflx wrote: > I know I already accepted this, but: > > > moves the preview button over to the right so it doesn't compete with the new view mode and sorting buttons. > > While you now could remove "sorting" from the summary, I wonder whether it would make sense to revisit moving Preview. How about this: > > F5806931: kio-preview.png <https://phabricator.kde.org/F5806931> > > After looking at your screenshot again, I'm not too fond of how Preview and Zoom Out relate to each other… > > One more thing which came to mind after revisiting the dialog: Why not use Detailed Tree View instead? This way the buttons would act as a kind of simple/advanced switcher. Also, this is what Dolphin does, too! Good points! REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D12077 To: ngraham, #frameworks, rkflx, #vdg, abetts Cc: rkflx, mmustac, broulik, michaelh, ngraham, bruns
