palantir / plottable

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

Regression in drag behavior with PanZoom after 3.4.1 #3395

Closed aleburato closed 7 years ago

aleburato commented 7 years ago

Bug report

Live example URL:

WORKING (3.4.1) https://jsfiddle.net/aleburato/cdtarz12/2/

BROKEN (3.5.1) https://jsfiddle.net/aleburato/0h93pbbt/8/

Steps to repro:

Expected behavior

The drag box is resized correctly and the main chart is updated accordingly

Actual behavior

The main chart just pans as if you were dragging on it directly

hellochar commented 7 years ago

Thanks for the bug report @aleburato - this has been fixed in 3.5.2. I've updated your example by simply upgrading to 3.5.2: https://jsfiddle.net/0h93pbbt/9/ Let us know if that works, cheers!

aleburato commented 7 years ago

Definitely. Loving Plottable AND Blueprint AND TSLint. You guys are a constant stream of awesomeness.

On ven 1 set 2017, 01:34 Xiaohan Zhang notifications@github.com wrote:

Thanks for the bug report @aleburato https://github.com/aleburato - this has been fixed in 3.5.2. I've updated your example by simply upgrading to 3.5.2: https://jsfiddle.net/0h93pbbt/9/ Let us know if that works, cheers!

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/palantir/plottable/issues/3395#issuecomment-326447767, or mute the thread https://github.com/notifications/unsubscribe-auth/AQPZGCPKJU8WxKbG0yNZlIumfD9sTitnks5sd0MVgaJpZM4PD27b .