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

Crash on range bounds if refresh view height is greater than 1000 #129

Open
twronka opened this issue Feb 14, 2022 · 0 comments
Open

Crash on range bounds if refresh view height is greater than 1000 #129

twronka opened this issue Feb 14, 2022 · 0 comments

Comments

@twronka
Copy link

twronka commented Feb 14, 2022

Report a bug

What did you do?

Refresh view with height greater than 1000

What did you expect to happen?

Refresh should be performed without any issues.

What happened instead?

Code crashed with error: "Fatal error: Range requires lowerBound <= upperBound"

Crash occured in file PullToRefresh line 178

Proposed solution

As a quick fix I have replaced line 177 that was:
let refreshViewHeight = refreshView.frame.size.height
with
let refreshViewHeight = refreshView.frame.size.height < 1000 ? refreshView.frame.size.height : 999

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

No branches or pull requests

1 participant