Selected rows are not numerically sorted.

[muendlein]

Describe the bug

Rows returned from onSelectedRowsChanged events are not numerically sorted but rather lexicographically, e.g. [0, 1, 10, 11, .. ].

Reproduction

SlickGrid/src/slick.grid.ts

Line 4011 in 2722c60

if (!this.arrayEquals(previousSelectedRows.sort(), this.selectedRows.sort())) {

Expectation

Arrays that are considered numerical should also be sorted numerically.

Which Framework are you using?

Other

Environment Info

Binaries:
    Node: 24.14.0
    npm: 11.12.0

Validations

• Follow our Code of Conduct
• Read the Wikis.
• Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
• Check that this is a concrete bug. For Q&A open a GitHub Discussion.
• The provided reproduction is a minimal reproducible example of the bug.

[ghiscoding]

I think you misunderstood that the only reason it does a sort internally is for the array comparison arrayEquals(). Since I think that SlickGrid also accepts basic string arrays as dataset, adding a sort by id would probably fail for that kind of basic dataset. So again I don't think the sort was put in place for the end user but probably just for array comparison checks internally. You should sort it yourself however you want

[muendlein]

I think you misunderstood that the only reason it does a sort internally is for the array comparison arrayEquals(). Since I think that SlickGrid also accepts basic string arrays as dataset, adding a sort by id would probably fail for that kind of basic dataset. So again I don't think the sort was put in place for the end user but probably just for array comparison checks internally. You should sort it yourself however you want

Thank you for your reply. The line I'm referring to is however about the (internal) row selection and not the dataset itself. According to the types used, it is strictly numerically.
Even if row selection is generally assumed to be unsorted and therefore a user responsibility to deal with, I think it still make sense to avoid the default sort() which treats numbers as strings.
Although the sorting is just in place for internally reasons, it leaks through and can cause confusion when querying the selected rows and receiving them in string order.

SlickGrid/src/slick.grid.ts

Line 471 in 2722c60

protected selectedRows: number[] = [];

[ghiscoding]

you can open a Pull Request if you want