Skip to content

Fix for getPointer on touchDevices + Fix for absolute postioned scrolled elements #407

Merged
merged 1 commit into from Feb 3, 2013

3 participants

@Kienz
Collaborator
Kienz commented Feb 1, 2013

Fix touch events if canvas is inside a scrolled container (see issue #123).

Fix bug if mouse moves over absolute scrolled element while object scaling (see issue #385). Also if mouse is moved over documentElement (fabric.document) while scaling object is fixed (same issue like #385).

Tested with Safari 5.1.7, 6.1, Firefox 18, 3.6.28, Chrome 24/25beta, Opera 12.12 and iPhone iOS 6.1, iPad iOS 6.1 in non-scrolled and scrolled containers.

@Kienz Kienz Fix touch events if canvas is inside a scrolled container (see issue …
…#123).

Fix bug if mouse moves over absolute scrolled element while object scaling (see issue #385). Also if mouse is moved over documentElement (fabric.document) while scaling object is fixed (same issue like #385).

Tested with Safari 5.1.7, 6.1, Firefox 18, 3.6.28, Chrome 24/25beta, Opera 12.12 and iPhone iOS 6.1, iPad iOS 6.1 in non-scrolled and scrolled containers.
ddaf8ba
@kangax kangax merged commit ac78df5 into kangax:master Feb 3, 2013

1 check passed

Details default The Travis build passed
@coulix
coulix commented Feb 7, 2013

Are we certain about:

"Also if mouse is moved over documentElement (fabric.document) while scaling object is fixed (same issue like #385)."

On chrome 24, with scaling or freehand drawing a jumpy behavior remains.

Freehand
http://jsbin.com/ibeseh/11/edit

Move / resize
http://jsbin.com/ibeseh/13

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.