-
Notifications
You must be signed in to change notification settings - Fork 17
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
Sort #45
Comments
Hello @papusa, Sort works on my Chrome (same version). You have to do the sort on your backend or with a specific function (see https://github.com/RoXuS/paper-datatable-api/blob/master/demo/advanced-pagination-front-side-demo.html). |
Hi, im very happy that i've found a working paper-datable component now after days of searching for some that is still developed. Nonetheless, i think this is quiet the right place for this issue. If i click on sort, whether with my implementation of your sort algorithm or yours on your demo here: It gets sorted, fine. But if you continue spamming the sort button, you have to click twice when the arrow points downwards to get it sorted the other way. What causes that? |
Hi @drdreo, Yeah it is normal you return to the "normal" state. The steps are : normal state (without specific order) -> order by desc -> order by asc -> normal state...etc |
Would be nice to have sort icon indicate that normal state. Currently, the icon by the column shows it is either ascending or descending only. |
The other issue with sorting behavior I encountered was that Here is a stack trace from a call when the data is set and it goes up to _handleSort.
I am running Chromium 70.0.3538.77 and recent paper-datatable-api v2.0.18. |
Hi,
How does sorting works? Does it require only sortable attribute to be set to true or needs some additional handlers?
In demos here: https://roxus.github.io/paper-datatable-api/components/paper-datatable-api/#paper-datatable-api sorting doesn't work on chrome (Version 58.0.3029.110 (64-bit)), but works on firefox.
Locally sort is not working on any browser.
Thanks
The text was updated successfully, but these errors were encountered: