-
-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Upgrade closure library version #2647
Conversation
return [eventPosition.x, eventPosition.y]; | ||
} | ||
var eventPosition = goog.style.getRelativePosition(event, this.viewport_); | ||
return [eventPosition.x, eventPosition.y]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We use event.changedTouches
when it is defined. google/closure-library@116609a does not use changedTouches
so I'm a bit uncertain that this commit really fixes the problem for us.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
testing now
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You're right; the change in closure-library in wrong. It always return [0, 0]
for touchend
events.
c60a7d9
to
ade15d7
Compare
ade15d7
to
926c21e
Compare
Is this ready to be merged from your point of view? |
No because of google/closure-library#343 |
926c21e
to
20ce664
Compare
Small clarification: the closure-library issue mentioned above don't occur with the latest closure-compiler version (closure-util >= 0.20.0) |
So I think we can merge this. Thanks! |
ok. thanks for the review |
And remove a workaround fixed upstream.
work in progress