ngraham added a comment.
I see your point @rkflx, but I think it's a mistake to use an icon whose appearance suggests that it represents state when in fact it doesn't--that's the kind of thing that causes the misconception. Since ascending alphabetical order is one of the available states, using an icon that with "ascending" and "alphabetical" iconography is bound to confuse and frustrate a few people who mistakenly expect it to reflect the active sort order. Either way, since this contains a string change, it's headed for 5.47 anyway (thanks for being a stickler and reminding me to pay attention to this), so I think we have some time to get a better icon. REPOSITORY R241 KIO BRANCH arcpatch-D12337 REVISION DETAIL https://phabricator.kde.org/D12337 To: ngraham, #frameworks, #dolphin, #vdg, rkflx Cc: andreaska, markg, broulik, anemeth, michaelh, bruns
