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

feat: add item count to data change handlers #570

Merged
merged 3 commits into from Jan 22, 2021

Conversation

ghiscoding
Copy link
Collaborator

@ghiscoding ghiscoding commented Jan 20, 2021

  • add item count to onRowCountChanged
  • add item count to onRowsChanged
  • add item count to onRowsOrCountChanged
  • add item count to onSetItemsCalled
  • also renamed getItemsCount to singular getItemCount so that it's aligned with the other counter.
    • note: this was a new function to get merged just 2 days ago, so it's ok to rename it now
  • onSetItemsCalled event should be called before the refresh method since that can itself trigger onRowCountChanged but it has to be after the onSetItemsCalled event instead of before

- add item count to `onRowCountChanged`
- add item count to `onRowsOrCountChanged`
- add item count to `onSetItemsCalled`
- the onSetItemsCalled should be called first and then the refresh can itself trigger onRowCountChanged but it has to be after the onSetItemsCalled event
@ghiscoding ghiscoding merged commit 32e03d5 into master Jan 22, 2021
@ghiscoding ghiscoding deleted the feat/handler-item-count branch January 22, 2021 14:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant