Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unable to preventDefault inside passive event listener invocation #939

Closed
maayanl600 opened this Issue Feb 20, 2019 · 6 comments

Comments

Projects
None yet
2 participants
@maayanl600
Copy link

maayanl600 commented Feb 20, 2019

Describe the Bug

The event “mouse-down” is throwing exception in the method preventDefault in bpmn-modeler.development.js
The exception is:
“Unable to preventDefault inside passive event listener invocation”

I looked for this error online and found the following:
“When using third party libraries (e.g. jQuery or lory.js) this error rises often - it’s due to fact that their listeners arent using passive event flag as default and still use preventDefault which in case of our default-passive property raises an error.”

Steps to Reproduce

  1. drag two elements to the canvas
  2. click on the arrow in the context pad on one element
  3. connect this element to the other
  4. in Dev-Tool you can see the following error:
    image

If you report a modeling related issue, ensure you can reproduce it on demo.bpmn.io

When reporting a library error, try to build an example that reproduces your problem. You can use our playgrounds for viewer or modeler as a starting point or put a demo up on GitHub for inspection.

Expected Behavior

No error

Environment

  • Browser: Chrome 72
  • OS: Windows 10
  • Library version: I have tried on 1.1.1 and also on last version 3.2.1. In both cases the error is reproduced
@nikku

This comment has been minimized.

@nikku

This comment has been minimized.

Copy link
Member

nikku commented Feb 20, 2019

We accept contributions that improve the situation.

@nikku nikku added the pr-welcome label Feb 20, 2019

@nikku

This comment has been minimized.

Copy link
Member

nikku commented Feb 20, 2019

Were you able to reproduce this issue in on our demo?

@maayanl600

This comment has been minimized.

Copy link
Author

maayanl600 commented Feb 20, 2019

I couldn't reproduce in the demo.
I new here so I posted a topic in addition. one of the posts can be removed if not necessary .

@nikku

This comment has been minimized.

Copy link
Member

nikku commented Feb 21, 2019

It is reladed to zone.js and the fact that you use BpmnJS in an Angular.io context.

@nikku

This comment has been minimized.

Copy link
Member

nikku commented Feb 21, 2019

I'll close this issue as we cannot reproduce it outside an Angular context.

We are open to contributions that get rid of the warning for Angular developers.

@nikku nikku closed this Feb 21, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.