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

IgxGrid: Multi-column sorting does not work if custom sortStrategy is used #12611

Closed
mkamiishi opened this issue Feb 6, 2023 · 1 comment · Fixed by #12682
Closed

IgxGrid: Multi-column sorting does not work if custom sortStrategy is used #12611

mkamiishi opened this issue Feb 6, 2023 · 1 comment · Fixed by #12682
Assignees
Labels
🐛 bug Any issue that describes a bug ✅ status: resolved Applies to issues that have pending PRs resolving them, or PRs that have already merged.

Comments

@mkamiishi
Copy link

Description

In this sample(https://stackblitz.com/edit/angular-pjwndf-2xqznm?file=src/app/grid/grid-sorting-sample/grid-sorting-sample.component.html) ‘Fruit’ and ‘Company Name’ columns use a custom sortStrategy so that ‘Fruit’ column uses ‘fruitIndex’ column data and ‘Company Name’ column uses ‘companyIndex’ column data respectively when they are sorted.
Now when multi-column sorting is applied, for example, 1st sort by ‘Group’ column, 2nd sort by ‘Fruit’ column, and 3rd sort by ‘Company Name’ column, the 3rd sort does not work as expected.
image

  • igniteui-angular version: 15.0.9
  • browser: Chrome

Steps to reproduce

Run the sample:https://stackblitz.com/edit/angular-pjwndf-2xqznm?file=src/app/grid/grid-sorting-sample/grid-sorting-sample.component.html

Result

Multi-column sorting does not work as expected.

Expected result

Multi-column sorting works with a custom sortStrategy.

@mkamiishi mkamiishi added the 🐛 bug Any issue that describes a bug label Feb 6, 2023
@hanastasov hanastasov added 👀 status: in-review Issue is currently being reviewed and removed 🆕 status: new labels Feb 20, 2023
@viktorkombov viktorkombov added 🛠️ status: in-development Issues and PRs with active development on them and removed 👀 status: in-review Issue is currently being reviewed labels Feb 21, 2023
@viktorkombov viktorkombov added ✅ status: resolved Applies to issues that have pending PRs resolving them, or PRs that have already merged. 🛠️ status: in-development Issues and PRs with active development on them and removed 🛠️ status: in-development Issues and PRs with active development on them ✅ status: resolved Applies to issues that have pending PRs resolving them, or PRs that have already merged. labels Feb 22, 2023
@hanastasov hanastasov reopened this Mar 7, 2023
@hanastasov
Copy link
Contributor

Please add Prs for 14.2.x, 15.0.x as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug Any issue that describes a bug ✅ status: resolved Applies to issues that have pending PRs resolving them, or PRs that have already merged.
Projects
None yet
4 participants