ngraham added a comment.
In D12337#249528 <https://phabricator.kde.org/D12337#249528>, @rkflx wrote: > I know you feel strongly about putting Sort functionality right in the face of users ;) Let me play the devil's advocate to work out whether another solution would be feasible too: > > - Introducing a new string to the UI comes at a cost: It draws attention away from what's really important, i.e. the files. I'm not saying sorting is not important, but is it really //that// important like you claim? > - Why would only a single icon get text added next to it? > - The text may be partly motivated by the lack of an appropriate icon. Shouldn't we improve the icons instead? > - This conflicts with Detailed View, where some of the options are more easily accessible by clicking on the table headers. Isn't this confusing for users? Being able to easily change the sort order between alphabetical and date modified is a critically important feature. Those are the two most frequently used and most commonly useful sort orders. Right now, for Detailed Tree View, this takes one click: good. But for Short View, it's quite hidden: you need to right-click in the view, or click on the settings icon in the corner that evidence suggests people don't use. And both require the at least-intermediate mousing skill of being able to use a sub-menu. Adding text to the button is motivated by the lack of a good icon, that's true. But I'm not sure this is a surmountable problem. I don't think I've ever seen an icon that adequately communicated "sorting options here!" all by itself. Have you? An icon-only button would not be sufficient here unless we can find a //perfect// icon. In D12337#249528 <https://phabricator.kde.org/D12337#249528>, @rkflx wrote: > - Solve the problem where toolbar menu buttons don't get a downwards pointing arrow with Breeze. Definitely. That's been on my to-do list for a while. Currently, they get an arrow only if the button is set up to show a menu on a click-and-hold, but not on a regular click. That's just a regular old bug IMHO. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D12337 To: ngraham, #frameworks, #dolphin, #vdg, rkflx Cc: markg, broulik, anemeth, michaelh, bruns
