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

Don't get stuck in overscroll #264

Merged
merged 2 commits into from Apr 25, 2016
Merged

Don't get stuck in overscroll #264

merged 2 commits into from Apr 25, 2016

Conversation

@mbrubeck
Copy link
Contributor

mbrubeck commented Apr 19, 2016

  • Never trigger overscroll bounce when overscroll is disabled. This fixes a bug where rounding could cause a layer to enter overscroll even if CAN_OVERFLOW is false, and then get stuck.
  • Run the overscroll animation all the way to the destination, then stop animating. This fixes a bug where a layer could get "stuck" in overscroll because it gets within 0.1 of the edge and then stops.
mbrubeck added 2 commits Apr 19, 2016
This fixes a bug where a layer could get "stuck" in overscroll because it gets
within 0.1 of the edge and then stops.
This fixes a bug where rounding could cause a layer to enter overscroll even
if CAN_OVERFLOW is false, and then get stuck in overscroll.
@mbrubeck
Copy link
Contributor Author

mbrubeck commented Apr 25, 2016

@pcwalton
Copy link
Collaborator

pcwalton commented Apr 25, 2016

@bors-servo
Copy link
Contributor

bors-servo commented Apr 25, 2016

📌 Commit 5dcd4d3 has been approved by pcwalton

@bors-servo
Copy link
Contributor

bors-servo commented Apr 25, 2016

Testing commit 5dcd4d3 with merge a06f12a...

bors-servo added a commit that referenced this pull request Apr 25, 2016
Don't get stuck in overscroll

* Never trigger overscroll bounce when overscroll is disabled. This fixes a bug where rounding could cause a layer to enter overscroll even if CAN_OVERFLOW is false, and then get stuck.

* Run the overscroll animation all the way to the destination, then stop animating.  This fixes a bug where a layer could get "stuck" in overscroll because it gets within 0.1 of the edge and then stops.
@bors-servo
Copy link
Contributor

bors-servo commented Apr 25, 2016

💔 Test failed - travis

@mbrubeck
Copy link
Contributor Author

mbrubeck commented Apr 25, 2016

@bors-servo retry

@bors-servo
Copy link
Contributor

bors-servo commented Apr 25, 2016

Testing commit 5dcd4d3 with merge 496513b...

bors-servo added a commit that referenced this pull request Apr 25, 2016
Don't get stuck in overscroll

* Never trigger overscroll bounce when overscroll is disabled. This fixes a bug where rounding could cause a layer to enter overscroll even if CAN_OVERFLOW is false, and then get stuck.

* Run the overscroll animation all the way to the destination, then stop animating.  This fixes a bug where a layer could get "stuck" in overscroll because it gets within 0.1 of the edge and then stops.
@bors-servo
Copy link
Contributor

bors-servo commented Apr 25, 2016

☀️ Test successful - travis

@bors-servo bors-servo merged commit 5dcd4d3 into servo:master Apr 25, 2016
3 checks passed
3 checks passed
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.