New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[D8][UX] Update the dated icons for sorting tables #3906
Comments
I was about to ask for a review of both PRs here, but when I tried to refresh the sandboxes I got problems again 😓 (see #4351) |
Either pr looks good to me. What do other people prefer? Are there other places to change if we use the new ones? |
I like the We already use (the |
I've restarted the PR sandboxes. Let's hope that things will come back green 🤞 @olafgrabienski I hear you re the arrows being to narrow/small. I have also thought of these: https://fontawesome.com/icons/sort-alpha-up-alt?style=solid But these would only make sense when sorting alphanumeric things, like node titles etc. - it wouldn't work so great with numbers/dates, which means that we'd need a way to tell things apart, and then use the numeric equivalents: https://fontawesome.com/icons/sort-numeric-up-alt?style=solid I also hear you re the ...but I wasn't able to find a free alternative for it in fontawesome 😞 ...anyway, that should be a separate issue. |
Can I please get some feedback here? ...there are 2 PRs, each one using different icons. Can people please test and chime in to say what they think? ...do you agree with the change? ...do you have an alternative set of icons to propose instead? PS: I've rebased both PRs, and confirmed that tests are passing and sandboxes spin up w/o issues (at least the Tugboat ones). |
The sorting icons containing (latin) characters or numbers don't seem optimal to me, as they're not generic and might be the wrong choice for different languages, especially character sets. But I don't know what might represent sorting in Chinese anyway. 😉 Regarding the other ones (arrows with or without bars, or carets), I don't have any preference. Whatever fits. |
I understand my suggestion (below) is a larger issue, but if we're considering changing icons then it should be considered now, rather than needing to change things again in future... I propose using icon fonts rather than images, since we're discussing dark themes in other issues, and one of the sticking points is that most of our icons/images are designed only for light-coloured backgrounds. An icon font will make it easier to allow different coloured icons on different coloured backgrounds. At least keep that in mind when deciding how to implement new icons here. As for which icons to use, I like the arrows next to the varying-length bars. |
Hmm, it seems that SVGs were discussed there, and that would solve the same problem here (allowing icon colours to change)... |
Looking again into it, I guess I have changed my mind completely:
Apart of that, both PRs seem to put the icons the wrong way round. Example: |
Hehe, I guess it's the way people look at things: do the buttons convey the current sorting order, or the sorting you will get after you click them? 😅 Anyway, I'm happy to change the order, but let's get some more feedback first.
It's not only that the current ones look dated (which can be a totally subjective opinion):
That would take time though, and I have found it that it's harder to get bigger sets of changes in 😓 ...I don't want to wait long for changes that are "easy wins". |
@klonos I agree that the icons, apart of the "dated" question, are not ideal. Personally, I don't have strong feelings about which icon to choose. If the goal is an 'easy win', I'd suggest the arrows without sorting symbol, so that we don't have to (wait for a) change (of) the Admin bar "Structure" icon. Re "do the buttons convey the current sorting order?": I'm skeptical if we should change the behavior people are used to. However, if your change was intentional, it should also be mentioned in the issue description. |
Nope, not intentional at all. Happy to change it if others agree re it being the "wrong" direction. Just waiting for more feedback.
I hear you 👍 ...I'd like to fix both these things though. We never seem to have found a good icon for "Structure", so we went with the current one, which is not ideal at all. |
If we're going to change icons, the new icons should point in the same direction as the old ones. I see only confusion otherwise. We can open a separate issue to discuss different arrow directions if needbe, but I don't think it should be part if this change. |
Here's the PR that uses |
...and the PR that uses |
...please provide feedback and up/down-vote on the comments above^^, using 👍 / 👎 as reaction. |
I didn't think to ask this earlier, but would you mind adding an option for the regular fontawesome arrow? I have a feeling it wont look as good, but at least we can see the comparison. No worries if that's out of the question. |
Am I bikeshedding with this? Riffing on #3906 (comment) I did a quick mock with the bars visually unsorted: |
I like that icon @laryn ...is it fontawesome, and if so, then which one is it? (it would have to be black or some dark gray color though, since the unsorted headers are light gray and we'd have accessibility issues) |
@klonos It's taken from the fontawesome icons you were using and then tweaked in Photoshop. I'm sure we can adjust colors on the SVG and then export as a PNG if we know which colors to use. |
@laryn I admire your creativity! ...you've made an icon that I find to be quite accurately denoting "unsorted" 👏🏼 Can you please provide a transparent 64px .png in |
@klonos Are we moving this back to "Needs work"? What's your feeling on it. |
@laryn I was hoping to get more feedback before adding the "unsorted" icon, but it can be a follow-up. The currently approved variation is the one with |
When I look at both and ask myself "which conveys sorting alphabetically versus inverse alphabetically?", I'm not sure from either. I'll just click it and then go "oops, I wanted the other way" and click again. (So no preference.) |
I've just had a look at both PRs. The alternative icon |
Thanks everyone for chiming in. Can someone please add the 1.19.4 milestone to this issue? |
Same @olafgrabienski |
Thank you @klonos for your continued attention to detail with this issue and PR. And thanks for reviewing and testing, @herbdool and @olafgrabienski, and for feedback @indigoxela, @philsward, and @bugfolder. It seems opinions on the regular or |
@robertgarrigos I agree with you, if you look at the icons in an analytical way. But when I just have a quick look at the icons, the first icon set seems more appropriate to me. I guess, that's because in the first set both elements change, and it's easier to recognize the direction of the arrow. |
@olafgrabienski I agree that for a quick look, the arrow is very graphic, but then it shouldn't be contradictory when you look at it closely. If we use those icons just because of the arrow, let's forget about anything else which is giving wrong information. |
Well, I'm open for that, and it reminds me at the begin of the discussion in this thread and some comments below, e.g. #3906 (comment). Anyway, as this issue here is fixed and closed, maybe open a new issue to put the decision into question? |
Those caret icons are dated, and they are being used all over the place:
It does make sense when used to expand/collapse things:
...but we should be using more universally acceptable icons for sorting.
Here's what is being implemented in the new D8 admin theme that is being developed at the moment:
The text was updated successfully, but these errors were encountered: