Skip to content

Commit

Permalink
fix(pagination): throw events only on page & total-items changes (#326)
Browse files Browse the repository at this point in the history
  • Loading branch information
olkeoguz authored Nov 24, 2022
1 parent ac3fdc8 commit 5386778
Showing 1 changed file with 8 additions and 1 deletion.
9 changes: 8 additions & 1 deletion src/components/pagination/bl-pagination.ts
Original file line number Diff line number Diff line change
Expand Up @@ -89,7 +89,11 @@ export default class BlPagination extends LitElement {
/**
* Fires when the current page changes
*/
@event('bl-change') private onChange: EventDispatcher<{ selectedPage: number; prevPage: number; itemsPerPage: number }>;
@event('bl-change') private onChange: EventDispatcher<{
selectedPage: number;
prevPage: number;
itemsPerPage: number;
}>;

connectedCallback() {
super.connectedCallback();
Expand All @@ -111,6 +115,9 @@ export default class BlPagination extends LitElement {
changedProperties.has('totalItems')
) {
this._paginate();
}

if (changedProperties.get('currentPage') || changedProperties.get('itemsPerPage')) {
this.onChange({
selectedPage: this.currentPage,
prevPage: changedProperties.get('currentPage'),
Expand Down

0 comments on commit 5386778

Please sign in to comment.