shlomiassaf/ngrid

MatSort wont trigger Sort.change = true, when using Custom Triggers.

Mikeyeli opened this issue · 2 comments

Mikeyeli commented

What is the expected behavior?

MatSort should change the event.sort.change flag to true, when sorting.

What is the current behavior?

MatSort currently fires the event, but the flag remains false.

What are the steps to reproduce?

Add MatSort to the current Infinite Scroll, Custom Triggers example

Which versions of Angular, CDK, Material, NGrid, OS, TypeScript, browsers are affected?

  • Angular, CDK & Material all using @latest 11.x.x
  • NGrid 3.x.x
  • Windows 10
  • Chrome

Is there anything else we should know?

I wasn't sure if this was more of a feature request since I'm not sure if this is actually supported.

shlomiassaf commented

Hi

I've just checked it and it does fire the event with changed: true

image

This is when I click on the header to activate sorting

image

Mikeyeli commented

This was my mistake, I was accidentally resetting the source of the grid everytime.
This is not an issue!