[colReorder] warning Added non-passive event listener to a scroll-blocking

[lenamtl]

I'm testing colReorder v 1.5.2

I'm getting this warning
Added non-passive event listener to a scroll-blocking 'touchstart' event. Consider marking event handler as 'passive' to make the page more responsive.

[colin0117]

also here: https://datatables.net/forums/discussion/63794/colreorder-warning-added-non-passive-event-listener-to-a-scroll-blocking#latest

[lenamtl]

Hi, to see the warning just go to https://datatables.net/extensions/colreorder/examples/initialisation/simple.html
and open Chrome dev tool right click inspect tehn check console tab.

This should be fixed as Chrome staring to turn these warning to error in some case.

[lenamtl]

This should be reopen to make sure this get fixed

[DataTables]

We've got it in our own tracker, and also it is in the forum, but yes, it can be open here as well.

[lenamtl]

This issue is still present

Here is my actual solution, I placed this code completey at the end of the page after Datatables

if (typeof EventTarget !== "undefined") {
	let func = EventTarget.prototype.addEventListener;
	EventTarget.prototype.addEventListener = function (type, fn, capture) {
		this.func = func;
		if(typeof capture !== "boolean"){
			capture = capture || {};
			capture.passive = false;
		}
		this.func(type, fn, capture);
	};
};  

[AllanJard]

That looks like it would make all non capture events passive on the page? I wouldn't feel comfortable putting that into the library.

Part of the issue is that there isn't yet a jQuery way of assigning a passive event listener. We'll probably just need to restrict our browser support and use the event natively.