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

Grid - selectRows does not emit RowSelectionChanged #8352

Closed
Swiftpaws opened this issue Oct 13, 2020 · 3 comments
Closed

Grid - selectRows does not emit RowSelectionChanged #8352

Swiftpaws opened this issue Oct 13, 2020 · 3 comments

Comments

@Swiftpaws
Copy link

Description

The method selectRows does not seem to emit RowSelectionChanged when executed

  • igniteui-angular version: 10.1.0
  • browser: Chrome dev

Steps to reproduce

Grid - code behind call grid.selectRows
https://stackblitz.com/edit/github-ub6tvd?file=src/app/grid/grid-sample-selection/grid-selection.component.ts

(Set a timer to select the first row every second - row selection changed should be sent at least once - click to actually emit the event)

Result

No row selection changed event is emitted, even though the selection changed on the grid

Expected result

Row selection changed event should be emitted regardless of the origin (code behind or user interaction)

Attachments

@Swiftpaws Swiftpaws added the 🐛 bug Any issue that describes a bug label Oct 13, 2020
@ddincheva ddincheva self-assigned this Oct 13, 2020
@ddincheva
Copy link
Contributor

Hello @Swiftpaws,
The behavior that you describe above is by design. When you as a developer perform an action through the public grid API, you know that the action is performed and you have all the necessary information to apply any additional custom logic based on that.

@Swiftpaws
Copy link
Author

Swiftpaws commented Oct 14, 2020

@ddincheva ok I see, thanks. This info should be added to the method description though - the current description makes it look like it emits on every selection change.

@ddincheva
Copy link
Contributor

@Swiftpaws , thank you so much for the feedback. We will update our documentation ASAP.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants