Skip to content

Commit bf8a592

Browse files
authored
Additional fixes for listener setup (#254)
1 parent 742bb4b commit bf8a592

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

src/plugin.js

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -42,12 +42,14 @@ function resolveOptions(chart, options) {
4242
// We also want to make sure listeners aren't always on. E.g. if you're scrolling down a page
4343
// and the mouse goes over a chart you don't want it intercepted unless the plugin is enabled
4444
var node = props._node;
45-
if (!(options.zoom && options.zoom.drag)) {
45+
var zoomEnabled = options.zoom && options.zoom.enabled;
46+
var dragEnabled = options.zoom.drag;
47+
if (zoomEnabled && !dragEnabled) {
4648
node.addEventListener('wheel', props._wheelHandler);
4749
} else {
4850
node.removeEventListener('wheel', props._wheelHandler);
4951
}
50-
if (options.zoom && options.zoom.drag) {
52+
if (zoomEnabled && dragEnabled) {
5153
node.addEventListener('mousedown', props._mouseDownHandler);
5254
node.ownerDocument.addEventListener('mouseup', props._mouseUpHandler);
5355
} else {

0 commit comments

Comments
 (0)