Improved browser compatibility for table scroll

[ManelBH]

Fixes #5052.
Currently the table scroll uses the non-standard 'mousewheel' event which is not supported by firefox (and maybe other browsers). Changed it to the standard 'wheel' event, supported by most browsers, with a fallback to 'mousewheel' just in case.

[archmoj]

Could you please investigate why these two tests are failing?

plotly.js/test/jasmine/tests/table_test.js

Line 446 in a54c502

it('bubbles scroll events iff they did not scroll a table', function() {

plotly.js/test/jasmine/tests/table_test.js

Line 472 in a54c502

it('does not scroll any tables with staticPlot', function(done) {

[ManelBH]

Yeah, apparently the tests check that the 'mousewheel' event is raised so they fail when 'wheel' is used instead.

[archmoj]

Those tests were added in #3327 so I'll request a review from @alexcjohnson.

[ManelBH]

Sorry it's my first PR and I'm not sure what's the expected thing to do in such cases, do I add a change to the test to handle the 'wheel' event?

[alexcjohnson]

Nice fix @ManelBH ! In fact looking around our code, we have a number of places that only 'wheel' is used, and at least one place we have a similar fallback to this:

plotly.js/src/plots/cartesian/dragbox.js

Lines 1228 to 1232 in a54c502

	// still seems to be some confusion about onwheel vs onmousewheel...
	function attachWheelEventHandler(element, handler) {
	if(!supportsPassive) {
	if(element.onwheel !== undefined) element.onwheel = handler;
	else if(element.onmousewheel !== undefined) element.onmousewheel = handler;

dragbox is about our oldest interactive component, and looking at the compatibility table in https://developer.mozilla.org/en-US/docs/Web/API/Element/wheel_event it seems Chrome didn't get 'wheel' until v61 (late 2017) so until then it was crucial to support both flavors. However at this point we could in principle 🔪 'mousewheel' entirely.

But we don't need to clean all of that up now. It should be fine to change the test to use 'wheel'.

[archmoj]

@ManelBH after 7f983c5 there is still one failing test:

plotly.js/test/jasmine/tests/table_test.js

Line 446 in a54c502

it('bubbles scroll events iff they did not scroll a table', function() {

[archmoj]

The no_ci tests were also passed on this branch.
💃 💃