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

🚀 - Emit column names for server-sorting Table #528

Closed
biased-badger opened this issue Jul 2, 2021 · 1 comment · Fixed by #1659
Closed

🚀 - Emit column names for server-sorting Table #528

biased-badger opened this issue Jul 2, 2021 · 1 comment · Fixed by #1659
Assignees
Labels
P2 This issue has medium priority

Comments

@biased-badger
Copy link
Contributor

🚀 Feature request

Is your feature request related to a problem?

Existing sorting comparators are handy, when working with all rows loaded in memory. However, they are not very useful when working with server side sorting. You have to write extra code to map comparators to column names.

Describe the solution you'd like

It would be more convenient to have some directive made especially for server-side sorting, that would emit column name instead of comparator function.

Describe alternatives you've considered

An alternative is to create your own function that would convert comparator to column name like it's done in the docs: https://taiga-ui.dev/components/table#server

@waterplea waterplea added the P2 This issue has medium priority label Jan 20, 2022
@waterplea
Copy link
Collaborator

waterplea commented Jan 20, 2022

I believe we can solve this by a directive to encapsulate boilerplate of mapping sorters to column names.

@waterplea waterplea changed the title [FEATURE] Emit column names for server-sorting TuiTable 🚀 - Emit column names for server-sorting TuiTable Feb 1, 2022
@waterplea waterplea changed the title 🚀 - Emit column names for server-sorting TuiTable 🚀 - Emit column names for server-sorting Table Feb 1, 2022
@splincode splincode moved this to ✅ Done in Taiga-family Sep 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P2 This issue has medium priority
Development

Successfully merging a pull request may close this issue.

4 participants