Skip to content
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

Ambigous column sort icons in DataTable #4708

Closed
fprijate opened this issue Oct 28, 2023 · 3 comments
Closed

Ambigous column sort icons in DataTable #4708

fprijate opened this issue Oct 28, 2023 · 3 comments
Assignees
Labels
Status: Discussion Issue or pull request needs to be discussed by Core Team Type: New Feature Issue contains a new feature or new component request

Comments

@fprijate
Copy link

fprijate commented Oct 28, 2023

IMHO current icons for column sort are ambigous.
I don't know is it arrow that counts or lines (Ok it is arrow meaning the biggest at the tip . Why then distracting lines)

  1. Left icon is interpreted by me: start at the top with the smallest value an go down to the largest
  2. Right icon is interpreted by me: start at the bottom with the largest value an go up to the smallest

Which has the same meaning to me - sort top to bottom (A-Z sort)
I would flip the left part (lines) in the right icon and change the meaning - the left icon is (A-Z), the right (Z-A)

downUp

There are some other possibilities, which seem more logical to me:

Simplest: Only one arrow - meaning, we start with smallest at the begining of the arrow and go down/up to the largest

Other possibilities (maybe reversing one of the arrows)

1, Capture3,
2. Capture1
3. Capture2
4. Capture4

Is your feature request related to a problem?

As described

Describe the solution you'd like

No response

Describe alternatives you have considered

No response

Additional context

No response

@fprijate fprijate added Status: Discussion Issue or pull request needs to be discussed by Core Team Type: New Feature Issue contains a new feature or new component request labels Oct 28, 2023
@FlipWarthog
Copy link
Contributor

This has come up recently. See this closed issue for context and an example showing how to override

#3978

@cagataycivici cagataycivici self-assigned this Oct 30, 2023
@cagataycivici cagataycivici modified the milestone: 3.38.2 Oct 30, 2023
@Voltra
Copy link

Voltra commented Nov 13, 2023

This has also come up in a few other issues and PRs, for instance #3732

@onursenture
Copy link
Member

onursenture commented Dec 20, 2023

It's already fixed here: #4981.

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Discussion Issue or pull request needs to be discussed by Core Team Type: New Feature Issue contains a new feature or new component request
Projects
None yet
Development

No branches or pull requests

5 participants