Closed Bug 1501382 Opened 6 years ago Closed 5 years ago

touch action regions are ignored inside opacity:0 divs

Categories

(Core :: Panning and Zooming, defect, P3)

64 Branch
defect

Tracking

()

RESOLVED FIXED
mozilla66
Tracking Status
firefox-esr60 --- wontfix
firefox63 --- wontfix
firefox64 --- wontfix
firefox65 --- wontfix
firefox66 --- fixed

People

(Reporter: kats, Assigned: mattwoodrow)

References

()

Details

(Keywords: regression, Whiteboard: [gfx-noted])

Attachments

(3 files)

Spinoff from bug 1500864 comment 5.

See the URL for the testcase. The STR are to (1) Load this page in Fennec and (2) try to scroll by starting your finger inside the black box. If you can scroll, that's bad. If you can't scroll, that means the touch-action:none is successfully preventing the scroll, which is good.

I ran mozregression on Android and found that this was broken by bug 1434243.
Keywords: regression
Priority: -- → P3
Whiteboard: [gfx-noted]
Marking fix-optional for 64. We could still take a patch for 65, and if it's verified and doesn't seem risky, could still take fixes for 64 as well.
Assignee: nobody → matt.woodrow
Blocks: 1500864
Pushed by mwoodrow@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/c62d53bdab79
Build display items within opacity:0 if we need them for APZ hit testing. r=kats
https://hg.mozilla.org/integration/autoland/rev/0d3d0fa680d8
Cull items within an opacity:0 container when using WebRender. r=kats
Is this something you wanted to consider for Beta uplift or should it ride the trains?
Flags: needinfo?(matt.woodrow)
Flags: in-testsuite+
This has been broken for a while without anyone complaining, so I don't think there's a huge rush to uplift.
Flags: needinfo?(matt.woodrow)
Depends on: 1516702
Depends on: 1535585
No longer depends on: 1535585
Regressions: 1535585
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: