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

Touch scrolling on Android triggers page refresh #471

Closed
Pakaa opened this issue Jun 25, 2019 · 3 comments
Closed

Touch scrolling on Android triggers page refresh #471

Pakaa opened this issue Jun 25, 2019 · 3 comments

Comments

@Pakaa
Copy link

Pakaa commented Jun 25, 2019

Expected Behavior

Scrolling upward should hide the elements on the bottom of the table while revealing elements on the top of the table

Current Behavior

Scrolling upward refreshes the page

Possible Solution

I think this one and the many other scrolling issues have to do with the passive event listener configurations, but I know that adding 'overscroll-behavior: none' to the document body does not fix it

Steps to Reproduce (for bugs)

Have a scrollable table
Scroll down
Scroll back up

Context

You can only scroll in one direction

Your Environment

  • Version used:
  • Browser Name and version: Chrome 75.0.3770.101
  • Operating System and version (desktop or mobile): Android Oreo
@pradeepnschrodinger
Copy link
Collaborator

@Pakaa , hey there,
Are you using the beta version? A lot of bug fixes and features are added into the v1.0-beta branch, which is our current active branch of development. Could you try it out and see if the issue still exists?

Also could you try using the stopScrollDefaultHandling prop turned on, since I suspect the issue here is related to #154 or #419.

And also make sure for touch devices, you have the prop touchScrollEnabled turned on (see #456).

@Pakaa
Copy link
Author

Pakaa commented Jun 26, 2019

Hello! I plugged in some beta versions and found that while beta 21 was unusable, beta 20 did fix this particular issue while I had the stopScrollDefaultHandling prop. There are other bugs in the beta but it sounds like you guys already caught this one. I'll close this ticket and wait for the next release. Thanks @pradeepnschrodinger

@Pakaa Pakaa closed this as completed Jun 26, 2019
@pradeepnschrodinger
Copy link
Collaborator

pradeepnschrodinger commented Jun 30, 2019

Hello! I plugged in some beta versions and found that while beta 21 was unusable, beta 20 did fix this particular issue while I had the stopScrollDefaultHandling prop.

Glad it solved the issue :)

@Paaka Could you give more info about why beta 21 is unusable? And is it in anyway related to this recent issue? This will help a lot in reproducing and fixing it.

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

2 participants