dy / pan-zoom

Pan / zoom for any element
MIT License
88 stars 15 forks source link

Send original event. #20

Closed kabudahab closed 4 years ago

dy commented 4 years ago

It works only for the wheel listener, but not for pinch/move - I was trying to do that not once. The impetus and touch-pinch don't expose event afaik, so it either needs to be fake or lobby PRs somehow.

kabudahab commented 4 years ago

This sends the original event of the wheel listener and can be used to detect whether Ctlr key is down when scrolling. At the moment, event is undefined for all events.

dy commented 4 years ago

This event property is inconsistent for pinch/move, needs to be added for them too.

kabudahab commented 4 years ago

Inconsistent event property is better than undefined. I am happy with how it works now, so I will close.

dy commented 4 years ago

Just for the reference https://github.com/dy/pan-zoom/pull/17, https://github.com/dy/pan-zoom/issues/18. I suppose it may be updated from impetus/touch-pinch to something new.