Chrome 74: Unable to preventDefault inside passive event listener due to target being treated as passive.

[SvetlozarValchev]

Bug report

Expected behaviour

No errors in Dev Console

Actual behaviour

When Zooming in/out, the console repeatedly logs the following error: Unable to preventDefault inside passive event listener due to target being treated as passive.

Steps to reproduce the behaviour

1. Zoom In/Out
2. Observe Dev Console

Configuration

• svg-pan-zoom version: 3.6.0
• Browser(s): Chrome 74
• Operating system(s): Windows 10 Pro

[Vktun]

I have same problem about this

[TrishE]

I have the same problem.

[azhengyongqin]

I have the same problem.

[ethubert]

You can get rid of the message by commenting line 845 in the the library ( i included the file in a browser app at the moment so I'll just drop the code you need to find.
Probably not the best way but it works and didn't break anything for now.

`
/**
* Handle mouse wheel event
*
* @param {Event} evt
*/

SvgPanZoom.prototype.handleMouseWheel = function(evt) {
  if (!this.options.zoomEnabled || this.state !== "none") {
    return;
  }

  if (this.options.preventMouseEventsDefault) {
    if (evt.preventDefault) {
      // evt.preventDefault(); COMMENT THIS LINE
    } else {
      evt.returnValue = false;
    }
  }

[ariutta]

Hi @SvetlozarValchev, could you review this pull request to verify it solves this issue:
#355
Thanks!

[SvetlozarValchev]

I can confirm I no longer can reproduce the issue.

[gingerr]

Doesnt this change zoom the svg and scrools the browser window at the same time then (if the page that hosts the svg is scrollable cause of long content? I think the purpose of this preventDefault() here is to prevent that scrolling.

[ariutta]

@SvetlozarValchev thanks for checking. Just to clarify, which code did you test for reproducing the issue: current master or pull request #355?

[ariutta]

@gingerr you're talking about pull request #355, right?

[BodoMinea]

For now people who encounter this issue may try cloning and building from https://github.com/Bigfellahull/svg-pan-zoom, the repo from where the PR came. That´s what I did and it worked.

[wassfila]

@BodoMinea you did a great job, but I assume that you intentionally did not want to push the built dist result ?

I created a fork including the file ready to be used, I have to admit that this great library is unusable without this fix.

https://github.com/WebSVG/svg-pan-zoom/tree/master/dist

[connerkennedy32]

Still experiencing this issue with version 3.12.1. Any file I can change now that will fix this with the newest chrome version?