Closed Bug 1730856 Opened 3 years ago Closed 3 years ago

Mousewheel/touchpad scrolling stopped working on fixed background images with body height 100vh

Categories

(Core :: Web Painting, defect, P3)

Firefox 87
All
Unspecified
defect

Tracking

()

VERIFIED FIXED
94 Branch
Tracking Status
firefox-esr78 --- unaffected
firefox-esr91 --- wontfix
firefox92 --- wontfix
firefox93 --- wontfix
firefox94 --- verified

People

(Reporter: ke5trel, Assigned: mikokm)

References

(Regression)

Details

(Keywords: regression)

Attachments

(3 files)

STR:

  1. Visit https://mcibb.github.io/mcibb/.
  2. Make sure the window width is larger than 1024px.
  3. Move the cursor over the background image.
  4. Try to scroll the page with the mousewheel or touchpad.

Mousewheel/touchpad scrolling doesn't work when initiated over the background image, only over text. Other page scrolling methods work normally, including keyboard, scrollbar and autoscroll. Works on Chrome.

Disabling background-attachment: fixed from the images or removing height: 100vh or overflow-x: hidden from the body allows mousewheel scrolling to work.

Regression window:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=576f61f7bb0f92b5e73dea66232912df2dd7d7c0&tochange=08c8d6946406b113c3725c999479c9dd67d2abef

Regressed by Bug 1689207

Hm, that patch likely just enabled HW-WR by default. Could you try if older version are also affected when you pass in --pref gfx.x11-gfx.webrender.all:true to mozregression?

And mind sharing your about:support?

Flags: needinfo?(ke5trel)

Can reproduce.

Flags: needinfo?(ke5trel)

Debian Testing, Gnome Xwayland, Intel, 2560x1440
mozregression --good 2020-09-10 --bad 2021-09-10 --pref gfx.webrender.all:true -a https://mcibb.github.io/mcibb/

7:37.59 INFO: Last good revision: cc4889143d653957d83a6e2ca9ea89691d39b435
7:37.59 INFO: First bad revision: f72030ee528939167d88804b61faa200246e037c
7:37.59 INFO: Pushlog:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=cc4889143d653957d83a6e2ca9ea89691d39b435&tochange=f72030ee528939167d88804b61faa200246e037c

f72030ee528939167d88804b61faa200246e037c Miko Mynttinen — Bug 1534549 - Part 4: Shrink nsDisplayCompositorHitTestInfo r=mstange
1907d855cb60b8e1eccd92513441a9a0f4aa5e27 Miko Mynttinen — Bug 1534549 - Part 3: Add tests r=mstange
fe81233c13cf45a57b6f422c5d05ac2a50e22071 Miko Mynttinen — Bug 1534549 - Part 2: Allow all display items to carry hit testing information r=mstange
56d9644b5a41b253a398157dd812d4cf7b12e30a Miko Mynttinen — Bug 1534549 - Part 1: Remove old FLB hit test info optimization r=mstange

Basic was not affected:
mozregression --repo autoland --launch f72030ee528939167d88804b61faa200246e037c --pref gfx.webrender.force-disabled:true -a https://mcibb.github.io/mcibb/

Has Regression Range: --- → yes
Has STR: --- → yes
Component: Graphics → Web Painting
Flags: needinfo?(mikokm)
Regressed by: 1534549
No longer regressed by: 1689207

The other WR-only scrolling bug is bug 1695822.

Can reproduce on Linux. Looking into this.

Assignee: nobody → mikokm
Status: NEW → ASSIGNED
Flags: needinfo?(mikokm)

Changing severity to S2 because of regression.

Severity: -- → S2
Priority: -- → P3
Attached file scroll.html (deleted) —

Simplified testcase.

Depends on D127120

Pushed by mikokm@gmail.com: https://hg.mozilla.org/integration/autoland/rev/8216b43fec63 Part 1: Use the correct scroll target for fixed backgrounds. r=mstange https://hg.mozilla.org/integration/autoland/rev/23f67b8ae851 Part 2: Add test. r=mstange
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 94 Branch
Flags: qe-verify+

Reproduced this issue with the test case from comment 7, on an affected Nightly build, running macOS 11.

The issue is verified as fixed on latest Beta 94.0b6, under macOS 11, Win 11, and Ubuntu 18.04 x64.

Status: RESOLVED → VERIFIED
Flags: qe-verify+
Blocks: 1737005
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: