Skip to content
This repository has been archived by the owner on May 4, 2022. It is now read-only.

ion-refresher on Android with native scrolling - scrolling inverted #186

Open
jgw96 opened this issue Jan 17, 2017 · 4 comments
Open

ion-refresher on Android with native scrolling - scrolling inverted #186

jgw96 opened this issue Jan 17, 2017 · 4 comments

Comments

@jgw96
Copy link
Contributor

jgw96 commented Jan 17, 2017

From @rolandoldengarm on January 25, 2016 23:36

When using the ion-refresher, with native scrolling enabled (which is default), in a specific usecase the scrolling is inverted.
How to reproduce: Pull down to refresh until the text/icon appears, but don't release. Scroll back up until the text/icon has disappeared. Now the scrolling of the list is inverted, i.e. if you scroll down with your finger, the items scroll in the opposite direction.

This cannot be reproduced on iOS.

Problem also does not occur with js scrolling.

Copied from original issue: ionic-team/ionic-framework#5194

@jgw96
Copy link
Contributor Author

jgw96 commented Jan 17, 2017

From @PerfectionCSGO on January 26, 2016 9:56

I can reproduce this. You get rid of the inversion effect when let go of it though.
Tested on an Android 5.x device.

@jgw96 jgw96 added the v1 label Jan 17, 2017
@jgw96
Copy link
Contributor Author

jgw96 commented Jan 17, 2017

From @crossdot on January 29, 2016 12:44

The same behavior on my device. Tested on Android 5.x

@jgw96
Copy link
Contributor Author

jgw96 commented Jan 17, 2017

From @ramsaylee on March 7, 2016 13:4

+1 for this.
Tested on Android 4.4

@theartur
Copy link

theartur commented Feb 2, 2017

+1 for this
Tested on Android 6.0.1

This is a bug in <ion-refresher>, controller $ionicRefresher, function handleTouchmove()

Change this snippet:
if (isDragging) { nativescroll(scrollParent, deltaY - dragOffset * -1); }

To this:
if (isDragging && isOverscrolling) { nativescroll(scrollParent, deltaY - dragOffset * -1); }

I made a pull request #203

#203

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants