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

Fix default page size for Pagination plugin. #247

Closed
wants to merge 1 commit into from

Conversation

@versusvoid
Copy link
Contributor

versusvoid commented Nov 30, 2016

When non-default pageSizes like [25, 'All'] are passed to Pagination plugin, defaultPageSize still becomes 10, because global pageSize array is used during initialization.

When non-default pageSizes like [25, 'All'] are passed to Pagination plugin, defaultPageSize still becomes 10, because global pageSize array is used during initialization.
@jsf-clabot

This comment has been minimized.

Copy link

jsf-clabot commented Nov 30, 2016

CLA assistant check
All committers have signed the CLA.

@dylans dylans added this to the 1.13 milestone Dec 5, 2016
@dylans

This comment has been minimized.

Copy link
Member

dylans commented Jan 11, 2017

Thanks. Closed as fd12797. Backported as 476f95d (1.12), c6d3716 (1.11), db284ba (1.10), 7a65cfe (1.9), 1b9fdf7 (1.8), and 0ea916b (1.7).

@dylans dylans closed this Jan 11, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.