Refactor code to fit style rules

This commit is contained in:
sasensi 2018-10-02 19:20:12 +02:00
parent 54e1a0c1ef
commit c034ebf554
2 changed files with 19 additions and 13 deletions

View file

@ -24,15 +24,17 @@ var DomEvent = /** @lends DomEvent */{
var func = events[type],
parts = type.split(/[\s,]+/g);
for (var i = 0, l = parts.length; i < l; i++) {
var eventName = parts[i];
// For touchstart/touchmove events on document, we need to explicitely
// declare that event is not passive (can be prevented).
// Otherwise chrome browser would ignore event.preventDefault() calls.
// See #1501 and https://www.chromestatus.com/features/5093566007214080
var options = el === document && (eventName === 'touchstart'|| eventName === 'touchmove')
? {passive: false}
: false;
el.addEventListener(eventName, func, options);
var name = parts[i];
// For touchstart/touchmove events on document, we need to
// explicitely declare that event is not passive (can be
// prevented). Otherwise chrome browser would ignore
// event.preventDefault() calls. See #1501 and
// https://www.chromestatus.com/features/5093566007214080
var options = (
el === document
&& (name === 'touchstart' || name === 'touchmove')
) ? { passive: false } : false;
el.addEventListener(name, func, options);
}
}
}

View file

@ -1441,11 +1441,15 @@ new function() { // Injection scope for event handling on the browser
// - If this is a unhandled mousedown event, but the view or tools
// respond to mouseup.
//
// Some events are not cancelable anyway (like during a scroll inertia
// on mobile) so trying to prevent default in those case would result
// in no effect and an error.
if (event.cancelable !== false && (called && !mouse.move || mouse.down && responds('mouseup')))
// Some events are not cancelable anyway (like during a scroll
// inertia on mobile) so trying to prevent default in those case
// would result in no effect and an error.
if (
event.cancelable !== false
&& (called && !mouse.move || mouse.down && responds('mouseup'))
) {
event.preventDefault();
}
},
/**