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

Grid Multisort behavior unintuitive #1197

Closed
clivemoj opened this issue Jan 13, 2020 · 10 comments · Fixed by #3401
Closed

Grid Multisort behavior unintuitive #1197

clivemoj opened this issue Jan 13, 2020 · 10 comments · Fixed by #3401
Assignees
Labels

Comments

@clivemoj
Copy link

Version: Vaadin Flow 14.1.2
Component: Grid

User sorting on several columns works back to front.
Latest column clicked becomes the no 1 sorting column, which is very counter-intuitive.
This is true even when the user just change the sorting direction.

Let's say you have a list of people with FIRST and LAST name and the user decides that he wish to sort by LAST name and then FIRST name, which is a pretty normal situation.
He is then forced to first click FIRST and then LAST, the opposite to what he was looking to do!
And, when he got the hang of that he changes his mind and want the FIRST name in descending order and clicks FIRST name again, then that column not only changes direction but also becomes the no 1 sorting column.

This needs to be fixed or possible to control via settings.

PS.
We are very happy about the improvement that allows the user to remove sorting.
DS.

@mehdi-vaadin mehdi-vaadin transferred this issue from vaadin/flow Jan 15, 2020
@mehdi-vaadin
Copy link
Contributor

Thank you @clivemoj for reporting this. I moved it to the correct repository.

@vaadin-bot vaadin-bot transferred this issue from vaadin/vaadin-grid-flow Oct 6, 2020
@vaadin-bot vaadin-bot transferred this issue from vaadin/vaadin-grid May 19, 2021
@vaadin-bot vaadin-bot transferred this issue from vaadin/web-components May 21, 2021
@vaadin-bot vaadin-bot added enhancement New feature or request vaadin-grid labels May 21, 2021
@KasparScherrer
Copy link

Hi, I very much agree with clivemoj here. This is very counter-intuitive!

The first column to be sorted by the user should ALWAYS remain the first sorting until the user removes the same again. Each subsequently applied column-sorting should not be priorized to the first one.

When a user wants to sort a Grid by lastName, then firstName, then age, the user instinctly clicks on lastName, then firstName, then age. Because every sortable Grid would behave this way. But in a Vaadin Application, this would now lead to a sorting of age, then firstName, then lastName. Very strange behaviour! Even (or especially) computer-savy users have reported problems with grid sorting in my applications.

I, too, would love to see this fixed or at least controllable with a setting/configuration.

@skiedrowski
Copy link

this is still true using Vaadin 21.0.2 - and it is very confusing

@skiedrowski
Copy link

Reproducer:

Expected: "order by firstName, lastName"
But is: "order by lastName, firstName"

@mictaege
Copy link

mictaege commented Oct 8, 2021

In my opinion the current behavior is a showstopper. If multisort would work the intuitive way, it would be a great and usefull feature, but currently we don't offer it to our users at all.

Are there any plans to change?

@MatthiasVanMeerbeek
Copy link

MatthiasVanMeerbeek commented Nov 16, 2021

Compared to Vaadin 8 the sorting works different and i.m.o. also counter-intuitive but some might find the new way intuitive. Providing an option to choose how you want to have the sorting is probably the optimal way to make both parties happy.

In Vaadin 8 you just had sort by ascending or descending order but in Vaadin Flow there is also a "default" or "neutral" state/order. If you use MultiSort you can just click on the headers to add (ascending/descending) or remove (default/neutral) sorting on the specific column. In the past you had to shift-click the header of the columns to to be able to use the MultiSort. It would be nice if the MultiSort was again only available when using shift-click. When you f.e. have 5 columns sorted right now you need to go through all the columns to set them back to the "default"/"neutral" state instead of just left clicking on one column header. This is also not so user-friendly i.m.o.

In the new (counter-intuitive) way it is also a lot harder to f.e. change the sorting (f.e. change from ascending to descending) for a specific column.
Example: There are two columns, column A and column B. Column A is sorted in ascending order and is nr.1 and column B is currently also sorted in ascending order and is nr.2. I want to change column B to descending order but I want column A to stay nr.1. I can't just click on B because that would make B nr.1. I would have to click on B (and set it to descending) and then I would also have to click on A again (and set it back to ascending).
So basically clicking on a header currently not only changes the sorting but also changes its "priority" if I you could call it that way.

@neelu-sarkar
Copy link

Adding my +1 for this issue

@stefanuebe
Copy link
Contributor

+1

@web-padawan
Copy link
Member

Related issue in the web component: vaadin/web-components#1281

@rolfsmeds rolfsmeds changed the title Grid Multisort Grid Multisort behavior unintuitive Jun 10, 2022
@web-padawan web-padawan self-assigned this Jun 24, 2022
@quartelh
Copy link

+1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.