Skip to content
This repository has been archived by the owner on Feb 11, 2021. It is now read-only.

Commit

Permalink
Don't look for touch-action-delay support
Browse files Browse the repository at this point in the history
Chrome 33 has a bug where the CSS OM objects exist for touchActionDelay even though parsing will always fail for them, so we'd incorrectly believe support was present.  Let's wait until there's actually a Chrome build to test before trying to enable a touch-action-delay fast-path.
  • Loading branch information
RByers committed Dec 31, 2013
1 parent 732cb6a commit 3599adb
Showing 1 changed file with 7 additions and 2 deletions.
9 changes: 7 additions & 2 deletions src/touch.js
Original file line number Diff line number Diff line change
Expand Up @@ -15,8 +15,13 @@
var CLICK_COUNT_TIMEOUT = 200;
var ATTRIB = 'touch-action';
var INSTALLER;
var HAS_TOUCH_ACTION_DELAY = (typeof document.head.style.touchActionDelay) === 'string';

// The presence of touch event handlers blocks scrolling, and so we must be careful to
// avoid adding handlers unnecessarily. Chrome plans to add a touch-action-delay property
// (crbug.com/329559) to address this, and once we have that we can opt-in to a simpler
// handler registration mechanism. Rather than try to predict how exactly to opt-in to
// that we'll just leave this disabled until there is a build of Chrome to test.
var HAS_TOUCH_ACTION_DELAY = false;

// handler block for native touch events
var touchEvents = {
scrollType: new WeakMap(),
Expand Down

0 comments on commit 3599adb

Please sign in to comment.