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

Add new 'setPage' and 'drawPage' functions. #76

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

Add new 'setPage' and 'drawPage' functions. #76

wants to merge 3 commits into from

Conversation

timgws
Copy link
Contributor

@timgws timgws commented Jul 4, 2014

As an alternative to @michael-misshore's patch for drawPage, this also adds a setPage function that will respect selectOnClick. I moved all of the duplicated code from _selectPage into setPage so that drawPage and _selectPage calls the code that is provided in setPage.

This allows you to update the page that is selected without triggering the onPageClick callback.

Also, if you still want selectOnClick to be respected, you can use setPage instead of drawPage. setPage will not redraw the pagination.

Note: This patch has been rebased, and @michael-misshore's original drawPage function removed.

timgws and others added 3 commits August 22, 2013 15:57
As an alternative to @michael-misshore's patch for drawPage, this also adds a setPage function that will respect selectOnClick. I moved all of the duplicated code from _selectPage into setPage so that drawPage and _selectPage calls the code that is provided in setPage.

This allows you to update the page that is selected without triggering the onPageClick callback.

Also, if you still want selectOnClick to be respected, you can use setPage instead of drawPage. setPage will not redraw the pagination.
the onPageClick callback.

Also, if you still want selectOnClick to be respected, you can use
setPage instead of drawPage. setPage will not redraw the pagination.
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