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

Scrolling is broken on Android #7036

Closed
larsbergstrom opened this issue Aug 6, 2015 · 4 comments
Closed

Scrolling is broken on Android #7036

larsbergstrom opened this issue Aug 6, 2015 · 4 comments
Assignees

Comments

@larsbergstrom
Copy link
Contributor

@larsbergstrom larsbergstrom commented Aug 6, 2015

I saw clicking on a link work once, but scrolling and pinch-zoom appear to be non-functional.

@larsbergstrom larsbergstrom changed the title Touch events no longer work on Android Scrolling is broken on Android Aug 10, 2015
@metajack metajack added the I-bustage label Aug 10, 2015
@metajack metajack mentioned this issue Aug 10, 2015
28 of 40 tasks complete
@mbrubeck
Copy link
Contributor

@mbrubeck mbrubeck commented Aug 10, 2015

Glutin translates touch input on Android into mouse events. Upstream Glutin master has a touch event API, but this isn't yet used in the Android port.

We could easily change Servo's android port to make "mouse" movement scroll the page instead of sending DOM mouse events, but this would break the demos that use mousemove events for other purposes.

To make touch input usable for both scrolling and DOM events, we need to implement the Touch Events DOM API, and cancel scrolling if the web page calls preventDefault on the "touchstart" or "touchmove" event.

@mbrubeck
Copy link
Contributor

@mbrubeck mbrubeck commented Aug 10, 2015

If a temporary hack is needed for demo purposes, we could also provide a way for a web page to opt out of touch scrolling, for example by implementing a tiny subset of the 'touch-action' CSS property.

@mbrubeck mbrubeck self-assigned this Aug 12, 2015
@mbrubeck
Copy link
Contributor

@mbrubeck mbrubeck commented Aug 12, 2015

I started working on basic touch event support at https://github.com/mbrubeck/servo/commits/touchevent

@waddlesplash
Copy link

@waddlesplash waddlesplash commented Oct 23, 2015

#7204 was merged.

@mbrubeck mbrubeck closed this Oct 23, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
4 participants
You can’t perform that action at this time.