Scrolling inside a .div using the touchpad causes the whole page to scroll on google graphs
Categories
(Core :: DOM: UI Events & Focus Handling, defect, P3)
Tracking
()
People
(Reporter: obotisan, Unassigned)
References
Details
(Keywords: webcompat:site-wait)
Attachments
(1 file)
4.29 MB,
video/quicktime
|
Details |
Updated•6 years ago
|
Reporter | ||
Comment 1•6 years ago
|
||
Comment 2•6 years ago
|
||
Comment 3•6 years ago
|
||
Assignee | ||
Updated•6 years ago
|
Comment 4•6 years ago
|
||
See bug 1547409. Moving webcompat whiteboard tags to project flags.
Comment 5•6 years ago
|
||
Ksenia, can you take a look at this please?
Comment 6•6 years ago
|
||
In this file:
The following event listener is added to a div with the graph:
this.$ = s_s(this.Ya, s_k.eh ? "DOMMouseScroll" : "mousewheel", this, b)
,
where s_k.eh
is a check for a User Agent, so DOMMouseScroll
is added in Firefox and mousewheel
in Chrome.
When the event is firing there is a call to preventDefault in this file:
s_Gd.prototype.preventDefault = function () {
s_Gd.Ua.preventDefault.call(this);
var a = this.zd;
if (a.preventDefault) a.preventDefault();
else if (a.returnValue = !1, s_wda) try {
if (a.ctrlKey || 112 <= a.keyCode && 123 >= a.keyCode) a.keyCode = - 1
} catch (b) {
}
While preventDefault works for mousewheel
, it doesn't work for DOMMouseScroll
(need to call preventDefault() of wheel events). So the page scrolls up and down because of that.
I think both of those legacy events listeners can be replaced by wheel
, though it will require some changes in the event handler code.
Comment 7•6 years ago
|
||
Here is a related bug https://bugzilla.mozilla.org/show_bug.cgi?id=1529953 about mousewheel
support
Comment 9•6 years ago
|
||
I sent an email to our partner list.
Updated•6 years ago
|
Reporter | ||
Updated•5 years ago
|
Comment 10•5 years ago
|
||
No response so far, reaching out to our Google search mailing list.
Updated•4 years ago
|
Updated•4 years ago
|
Comment 12•4 years ago
|
||
Updating flags based on duplicate Bug 1699400
Updated•3 years ago
|
Updated•3 years ago
|
Comment 13•3 years ago
|
||
Managed to reproduce the issue also on Ubuntu 20 X64. with build 96.0b2-snap.
In comparison with the Windows repro steps, on ubuntu the touchpad scroll must be longer in order for the page to scroll.
Updated•3 years ago
|
Comment 14•3 years ago
|
||
Also reproducible on Beta v105.0b8 on Windows 7.
Updated•2 years ago
|
Description
•