Highcharts: Multi-[Violation] Added non-passive event listener to a scroll-blocking 'touchstart' event ++

Thanks for reporting!

@sebastianbochan - could you take a look?

Internal note
In the stockTools we add events (click / touch) by default H.addEvent. In the function we should add support of {passive: true} for touch events.

• https://developer.mozilla.org/en-US/docs/Web/API/EventTarget/addEventListener

Is there any quick workaround till you add passive support?

Hi @slashvortal,
At this moment you can overwrite the H.addEvent function, but, what is important, it requires to be in correct place (before loading modules).

Demo:

• https://jsfiddle.net/BlackLabel/x3c7w2mv/

Thanks @sebastianbochan, I would wait for the fix so I can use it with highcharts-react wrapper.

Any plans to fix this yet? The workaround demo linked by @sebastianbochan still shows 21 event-related violations to me.

Hi @pjanaya

Unfortunately we haven't had time to prioritize this yet.

@pawelfus Ok, thanks. Maybe you should consider removing the "Has workaround" label? Since, at least for me, the workaround doesn't seem to be enough to eliminate the errors.

Internal note:
It's a general issue, not Highstock only. We need to verify if all warnings are correct, sometimes we do need preventDefault() and sometimes we can not promise that a developer won't call preventDefault() through one of the callbacks.

Hi All, is there any update on this issue?

Hi @dave-brown755,
At this moment we are not working on this case, so we do not have new information about. Have you tried to use our workaround?

Hi, I didn't try any workaround as there appeared to be a subsequent post asking for the workaround label to be removed. Are you able to outline the exact steps that need to be followed to implement the workaround to allow the error to be removed. This is significantly impacting browser performance for ourselves I don't want to move away from high charts but we will not be able to live with the amount of errors being generated. Hope you can help. Thanks Dave.

Thank you for the feedback. I added the inbox label which means that the ticket is prioritized.

Hi, any update on the priority of the ticket?

Hi @dave-brown755 - this ticket is already prioritized.

I am waiting for this to be fixed too
https://piyasa.paratic.com/

I could see the ticket had been prioritized but was asking what this means, what is the typical lead time for a ticket that has been given a priority?

That means ticket will be picked up once current assignments are finished. I expect this issue to be fixed in v8.0.1 or v8.0.2.

I don't see any warnings in the console anymore (newest Chrome 80). Tested on 3 machines (Mac and Windows 10). Can someone confirm whether the issue still occurs and provide the exact steps to reproduce it?

I'm using v8.0.2 but I still see this message. :(

Just a clarification note: v8.0.1 was released (and v8.0.2) and v8.0.3 will be released shortly because of other critical bugs. Not related to the releasing schedule I expected when I said that I expect issue to be fixed in v8.0.2.

@raf18seb tried to debug the issue a few days ago, but could't recreate this. Any more details (os + chrome version) or new live demo - highly appreciated.

@pawelfus we're also seeing this violation, with the latest version 8.0.4. We're rolling back to 5.0.11 where the issue doesn't exist but are looking to update this as soon as we can.

@onur-celik posted a link above where the issue is present for debugging. https://piyasa.paratic.com/

I'm using a Mac with Chrome 80.

Thanks! I can see these two errors. Both events can be blocking - e.g. when you move your finger around the chart to see the tooltip, then you don't want to scroll your page. Chrome shouldn't show this violation (according to spec, passive: true is set by default..).

@raf18seb - could you check once again? Maybe it's OS-dependant issue?

I confirm, I am able to reproduce it now - I have the same errors in the console on Windows.

hi, is this issue solved? any solution for this issue? I am facing this same issue when using highchart to get complex data from API

The issue has not been solved yet. At this moment we do not have ETA for this ticket, but is in our high priority group of tickets.

okay, thanks for the reply

I'd also really appreciate a fix for this or just a note of when it might be released.

I am glad to see this issue "in progress" :)

You have mentioned fixed here. When can these changes be applied in CDN versions.

Hi @sharmankita - issue still has in progress label. That means it's not fixed, we are still working on it.

Hi, any update on this? Is there an ETA for the fix?

Hi @violetVo - no ETA, we are still working on this.

My personal estimate: within next two versions should be ready.

What happens when the event is marked as "passive"?

elm.on("touchstart", function(ev) {
  // stuff
}, {
  passive: true // does this option help?
});

I'm not sure what do you mean @richardeschloss - could you elaborate? I will try to answer your questions one by one:

What happens when the event is marked as "passive"?

See description here: https://developer.mozilla.org/en-US/docs/Web/API/EventTarget/addEventListener

passive: true // does this option help?

This won't work in IE and Safari on iOS.

Pawel, it seems like Rafal has got this one fixed. I saw the PR which helps answer the question I had. thanks.

finally 🥰

Hi,
Thanks for the fix! Is it available in version 8.2.2?

Hi @violetVo v8.2.2 was released ~2 weeks ago. Fix will be available in the next release.

@pawelfus do you have a timeframe for the next release? Just wondering.
Hope you and the team are safe (re: covid)!

Thank you @forgivegod, we are good 👍 I hope you are good too!

The next release should be ready within a few weeks