Closed
Bug 1501382
Opened 6 years ago
Closed 6 years ago
touch action regions are ignored inside opacity:0 divs
Categories
(Core :: Panning and Zooming, defect, P3)
Tracking
()
RESOLVED
FIXED
mozilla66
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.
Reporter | ||
Updated•6 years ago
|
status-firefox63:
--- → wontfix
status-firefox64:
--- → affected
status-firefox65:
--- → affected
status-firefox-esr60:
--- → affected
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 | ||
Updated•6 years ago
|
Assignee: nobody → matt.woodrow
Assignee | ||
Comment 2•6 years ago
|
||
Assignee | ||
Comment 3•6 years ago
|
||
Depends on D14230
Reporter | ||
Comment 4•6 years ago
|
||
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
Pushed by kgupta@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/8c491aa06a8e
Add a mochitest. r=botond
Comment 7•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/c62d53bdab79
https://hg.mozilla.org/mozilla-central/rev/0d3d0fa680d8
https://hg.mozilla.org/mozilla-central/rev/8c491aa06a8e
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox66:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla66
Comment 8•6 years ago
|
||
Is this something you wanted to consider for Beta uplift or should it ride the trains?
Flags: needinfo?(matt.woodrow)
Flags: in-testsuite+
Assignee | ||
Comment 9•6 years ago
|
||
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)
Updated•6 years ago
|
Updated•6 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•