Hi,
I've noticed that mouse wheel scroll events tend to result in excessive
scrolling. Has anyone else noticed similar behavior?
Looking at guacamole-common-js/Mouse.js it looks like the wheel handler is
being attached in different ways to support old browsers, however modern
browsers support the old event names ("mousewheel" and "DOMMouseScroll") for
backwards compatibility in addition to the new ("wheel"). I think the result is
that Firefox (and likely Chrome) end up triggering the wheel handler multiple
times per actual use.
Should I create a JIRA issue for this? In basic testing a change like this
seems appropriate:
if (WheelEvent) {
// all modern browsers
addEventListener('wheel', ...
} else {
// firefox
addEventListener('DOMMouseScroll', ...
// chrome
addEventListener('mousewheel', ...
}
https://developer.mozilla.org/en-US/docs/Web/API/Element/DOMMouseScroll_event
https://developer.mozilla.org/en-US/docs/Web/API/Element/mousewheel_event
Christopher
