-
Notifications
You must be signed in to change notification settings - Fork 46
server side sorting #4055
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
base: develop
Are you sure you want to change the base?
server side sorting #4055
Conversation
99966e5 to
76c3b41
Compare
|
I just noticed that this doesn't work well together with our search form. I'm working on a solution |
MizukiTemma
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good 😸
I have just a small suggestion to add " ▲▼" to columns that can be used for sorting but not yet clicked, to singnalise users.
|
I think arrows are also good - just to make sure: there is either an up arrow (ascending), a down arrow (descending) or no arrow (unsorted)? @hauf-toni |
We could show it as not active (grey color) until clicked, then as active (black). |
Screen.Recording.2025-12-19.at.16.58.46.movHere's what I planned: Initially, all columns look the same. If a sortable column header is clicked, an arrow will appear next to the title. The sort can be reversed by clicking the same column again. The sort for that column is reset by double-clicking the column header. |
Just to make sure - what you are showing in the video is obviously what works already but then there's a reset for the search that does not work yet? Can you explain to me the difference between sort and search in this case? @jonbulz |
Sorry, I meant sort both times. What works is to sort by a column by clicking on the header, and then reversing the sort by clicking again. The only way to reset the sort in the current implementation is to refresh the whole page. That's what I meant. |
|
What is the current state? Should I review or is this blocked by #4009? |
Pardon my late reply, I just wanted to add that from a usability perspective I would not recommend to hide the functionality. As the sorting is not visible before clicking the first time the the user needs insider knowledge to access it. I would strongly recommend (with my ui/ux hat on, not the PO hat 🧢 ) that we adjust it to be visible before it clicked, eg by displaying the arrow as disabled first. |
|
I'd love to give you the opportunity to test this yourself @hauf-toni because I think it's hard to capture the user experience in a video. However, changing the icons and the way they are displayed is not a big code change. We could either:
|
We could have a look today during our meeting :) |
|
In our discussion today, we decided to abolish multi-column sorting and reset the sort by clicking the header and cycling through: |
Review SummaryWhat it does
Key observations
Note: There's an ongoing design discussion in comments about the arrow styling. Waiting for design input. |

Short description
Add server-side sorting (for contacts) in list views. Clicking a table header in the list view will toggle the sorting by that column. Clicking the header again will reverse the sort. Multiple columns can be used for sorting simultaneously.
Proposed changes
table_fieldsin list view (only for contacts thus far)_sortable_table_headertemplate to generate table header from table fieldssort_tagsto make table header with sort label clickableFilterSortMixinto handle sort fields correctlySide effects
table_fieldsmatches the rows of the list view, otherwise there could be table header/column mismatchesFaithfulness to issue description and design
There are no intended deviations from the issue and design.
How to test
Go to Contact list view, test sorting by clicking on column headers
Resolved issues
Fixes: #3347 #3880
Pull Request Review Guidelines