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

Make sure that pagination block is available in template regardless of pagination status #622

Merged
merged 2 commits into from
Sep 21, 2018

Conversation

apocalyptech
Copy link
Contributor

Templates which extend tables.html don't have access to the pagination block when there's only one page, which can pose a problem for templates which attempt to do some custom pagination coding.

This fixes #621

Verified that tests still pass, and verified that this does the trick on my own app which was using a custom template to do just that, though I've not done further testing than that.

…f pagination status, so custom templates can make use of the block in any situation
Repository owner deleted a comment from coveralls Sep 21, 2018
@jieter
Copy link
Owner

jieter commented Sep 21, 2018

@apocalyptech looks good, thanks!

@jieter jieter merged commit 9cd2116 into jieter:master Sep 21, 2018
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.

Include pagination block in table.html regardless of table.paginator.num_pages?
2 participants