Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Use has_other_pages() to check if the pagination bar should be shown #1446

Merged
merged 1 commit into from
Jun 13, 2020

Conversation

Ninjaclasher
Copy link
Member

...Rather than the number of pages since the infinite paginator
returns a constant 1e3000 even if there is only 1 page.

@codecov-commenter
Copy link

codecov-commenter commented Jun 12, 2020

Codecov Report

Merging #1446 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1446   +/-   ##
=======================================
  Coverage   38.42%   38.42%           
=======================================
  Files         201      201           
  Lines       11410    11410           
=======================================
  Hits         4384     4384           
  Misses       7026     7026           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1d27c23...a9278e2. Read the comment docs.

...rather than the number of pages since the infinite paginator
returns a constant 1e3000 even if there is only 1 page.
@Ninjaclasher Ninjaclasher changed the title Use has_other_pages() to check if the pagination bar should be shown. Use has_other_pages() to check if the pagination bar should be shown Jun 12, 2020
@Ninjaclasher Ninjaclasher marked this pull request as ready for review June 12, 2020 14:47
@quantum5
Copy link
Member

I mean, the infinite paginator probably shouldn't return infinity if it knows the number of pages, but this change will make it look more elegant than > 1.

@quantum5 quantum5 merged commit 2149155 into DMOJ:master Jun 13, 2020
@Ninjaclasher Ninjaclasher deleted the pagination-bar branch June 13, 2020 16:10
outloudvi pushed a commit to SchOJ/site that referenced this pull request Oct 27, 2020
DMOJ#1446)

...rather than the number of pages since the infinite paginator
returns a constant 1e3000 even if there is only 1 page.
malbareda pushed a commit to malbareda/JOEL-web that referenced this pull request Jan 4, 2024
DMOJ#1446)

...rather than the number of pages since the infinite paginator
returns a constant 1e3000 even if there is only 1 page.
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants