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

0.8 gestures in core #1486

Merged
merged 1 commit into from
May 6, 2015
Merged

0.8 gestures in core #1486

merged 1 commit into from
May 6, 2015

Conversation

sjmiles
Copy link
Contributor

@sjmiles sjmiles commented May 5, 2015

dfreedman's latest gesture code tweaked to work in core

Allow behaviors to easily override event listener creation

Expose a Behavior

Use a coarse mouse on/off feature for handling synthetic mouse events and ghost clicks

Implement all in the burndown list

Add a smoke test

only handle one finger with touchevents

Make sure track state can be "start" for mouse

Minor fix for touchevent single-finger check

bolt basic gestures onto core

add tests for gesture private data tracking
// but this breaks `<input>` focus and link clicks
// disable mouse handlers for MOUSE_TIMEOUT ms after
// a touchend to ignore synthetic mouse events
var MOUSE_CANCELLER = function(mouseEvent) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

suggest camelCasing function names

@sorvell
Copy link
Contributor

sorvell commented May 6, 2015

LGTM (refinements to follow)

sorvell pushed a commit that referenced this pull request May 6, 2015
@sorvell sorvell merged commit 9126b3c into 0.8-preview May 6, 2015
@sorvell sorvell deleted the 0.8-gestures-in-core branch May 6, 2015 01:12
dfreedm added a commit that referenced this pull request May 6, 2015
@dfreedm dfreedm mentioned this pull request May 6, 2015
sorvell pushed a commit that referenced this pull request May 6, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants