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

Vertical scroll: unable to scroll on touch device #10768

Closed
KacperMadej opened this issue May 20, 2019 · 3 comments · Fixed by #10786
Closed

Vertical scroll: unable to scroll on touch device #10768

KacperMadej opened this issue May 20, 2019 · 3 comments · Fixed by #10786

Comments

@KacperMadej
Copy link

Expected behaviour

Default 2 finger scroll to scroll the chart and a 1 finger scroll to scroll the web site.

Actual behaviour

Only the arrow buttons on the ends of the scrollbar are working on a touch device.

Live demo with steps to reproduce

https://jsfiddle.net/BlackLabel/hsm957qg/3/show

Open on a mobile device and try to scroll page and the chart.

Affected devices

Tested on Android 7.0 and iPad

about scrollbar fyi: @pawelfus

@TorsteinHonsi
Copy link
Collaborator

TorsteinHonsi commented May 21, 2019

An alternative approach to fixing could be to enable vertical scrolling in scrollablePlotArea. It works in a different way, by rendering the plot area in a specific div, and will provide a much smoother scrolling experience for both mobile and desktop.

It is already a popular request at #9378.

PS: I'll look into that one.

@sebastianbochan
Copy link
Contributor

The idea with scrollablePlotArea sounds good, but im afraid that in combination with this bug #8607 will be really limited.

@TorsteinHonsi
Copy link
Collaborator

Work in progress on scrollablePlotArea: https://highcharts.github.io/highcharts-utils/samples/#gh/f67b942449/sample/highcharts/chart/scrollable-plotarea-vertical

The idea with scrollablePlotArea sounds good, but im afraid that in combination with this bug #8607 will be really limited.

That shouldn't be too hard to fix.

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

Successfully merging a pull request may close this issue.

3 participants