Skip to content

Floor the value returned by viewPortToCount() #772

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

Merged
merged 1 commit into from
May 15, 2022
Merged

Conversation

juuz0
Copy link
Collaborator

@juuz0 juuz0 commented May 14, 2022

Fixes #766

@juuz0 juuz0 requested a review from kelson42 May 14, 2022 10:26
@codecov
Copy link

codecov bot commented May 14, 2022

Codecov Report

Merging #772 (2771a95) into master (8dbf015) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #772   +/-   ##
=======================================
  Coverage   61.45%   61.45%           
=======================================
  Files          58       58           
  Lines        3860     3860           
  Branches     2092     2092           
=======================================
  Hits         2372     2372           
  Misses       1487     1487           
  Partials        1        1           

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 8dbf015...2771a95. Read the comment docs.

@kelson42 kelson42 requested review from veloman-yunkan and removed request for kelson42 May 14, 2022 10:51
Previously, the value returned by viewPortToCount() could be a decimal number, this floors its value.
Helps in clean requests and caching.
Fix #766
@kelson42 kelson42 merged commit 5fb919e into master May 15, 2022
@kelson42 kelson42 deleted the roundHomepage branch May 15, 2022 08:02
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.

Round count in homepage request
3 participants