d3 / d3-zoom

Pan and zoom SVG, HTML or Canvas using mouse or touch input.
https://d3js.org/d3-zoom
ISC License
502 stars 143 forks source link

Remove touch-action:none? #217

Closed Fil closed 3 years ago

Fil commented 4 years ago

This appears to create various problems:

Why it was useful is not documented: https://github.com/d3/d3-zoom/issues/206#issuecomment-657566655 ; it appears that touch-action: none was added at about the same time as this conversation happened. In that case it might be related with enabling something (?) on Surface tablets.

I think I want to remove it from the two branch, and see if we can fix the situation that called for it, if someone finds what it was.

Fil commented 3 years ago

Here's another comment about touch-action:none https://github.com/d3/d3/pull/2403

mbostock commented 3 years ago

I don’t recall the context around this change, sorry.