rkflx added a comment.
In D12337#256813 <https://phabricator.kde.org/D12337#256813>, @ngraham wrote: > Now we have a problem. With no text in the toolbar button, the actual items don't reveal their purpose so readily unless we change all of their strings to say "Sort by Name" (etc.). But if we do that, then the context menu will have "Sorting > Sort by name. Thoughts? What about injecting a header as proposed in D12337#249821 <https://phabricator.kde.org/D12337#249821>? Or could we simply remove the context menu (only for the file dialog)? REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D12337 To: ngraham, #frameworks, #dolphin, #vdg, rkflx Cc: andreaska, markg, broulik, anemeth, michaelh, bruns