search

palantir / plottable

:bar_chart: A library of modular chart components built on D3
http://plottablejs.org/
MIT License
2.97k stars 223 forks source link

Mouse event handling broken by new Chrome Release #3491

Open JohanZackrisson opened 5 years ago

JohanZackrisson commented 5 years ago

Bug report

As described in https://www.chromestatus.com/features/6662647093133312 chrome has started handling events differently. This is affecting, for example, zoom behaviour in plottable, even in the examples on the page.

Quick fix is to set { passive: false } when setting up the listeners.

Live example URL: http://plottablejs.org/examples/finance/ Steps to repro:

  1. Install chrome 73
  2. Go to url
  3. Zoom and check the console output
  4. "[Intervention] Unable to preventDefault inside passive event listener due to target being treated as passive. See "

Expected behavior

Scroll behaviour is leaking from the component to the rest of the page. The component should handle this as zoom.

Actual behavior

The component is both zoomed and the rest of the page is scrolling.

Environment