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

Some fixes for components. #295

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Some fixes for components. #295

wants to merge 4 commits into from

Conversation

Sellec
Copy link

@Sellec Sellec commented Jan 20, 2020

1. Added ability to freeze dialog.
When freezed, all buttons and header icons are disabled

2. Update paginator.js
Changed setTotalRecords behaviour:

  1. If new page count is smaller than current page number, then new max page is set and paginate event triggered.
  2. If new page count is higher or equal current page number, then doing nothing.
    Related to puidatatable, old behaviour works badly with lazy loading and filtering. When filtered data is received and applied, paginator is need to be updated.

3. Update datatable.js
Fixed initial lazy datasource call error when "rows" parameter was not filled from paginator options becouse in the initial call paginator is not yet created.

When freezed, all buttons and header icons are disabled.
Changed setTotalRecords behaviour:
1) If new page count is smaller than current page number, then new max page is set and paginate event triggered.
2) If new page count is higher or equal current page number, then doing nothing.
Related to puidatatable, old behaviour works badly with lazy loading and filtering. When filtered data is received and applied, paginator is need to be updated.
Fixed initial lazy datasource call error when "rows" parameter was not filled from paginator options becouse in the initial call paginator is not yet created.
@Sellec Sellec changed the title Added ability to freeze dialog. Some fixes for components. Jan 21, 2020
Added "renderData" event to catch table contents update if you want to bind any trigger handlers.
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.

1 participant