While working on stylus support I've noticed that events from stylus are being sent twice. This is because pen also sends TouchEvents. Our code has the following check:
if (event.changedTouches[i].touchType === 'stylus') {
continue;
}
Given that TouchEvents that we send to handlers are constructed manually from data taken from PointerTracker (see this function) I think it is better to remove TouchEventManager than trying to find some weird ways to block it from sending stylus events.
Description
While working on
stylussupport I've noticed that events fromstylusare being sent twice. This is becausepenalso sendsTouchEvents. Our code has the following check:The problem with this approach is that
touchTypeis only supported on mobile Safari.Given that
TouchEventsthat we send to handlers are constructed manually from data taken fromPointerTracker(see this function) I think it is better to removeTouchEventManagerthan trying to find some weird ways to block it from sendingstylusevents.Test plan
Verified that our examples work as they should