Enable Visual Viewport API by default on Android
Categories
(Core :: Panning and Zooming, enhancement, P3)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox68 | --- | fixed |
People
(Reporter: JanH, Assigned: botond)
References
(Blocks 1 open bug)
Details
(Keywords: dev-doc-complete)
Attachments
(1 file)
|
(deleted),
text/x-phabricator-request
|
Details |
|
||
Updated•6 years ago
|
|
||
Updated•6 years ago
|
|
Assignee | |
Updated•6 years ago
|
|
|
Assignee | |
Comment 1•6 years ago
|
||
Kats, what do you think about making dom.visualviewport.enabled conditional on apz.allow_zooming (and, until bug 1459260 is resolved, dom.meta-viewport.enabled)?
The motivation for this is that correct updating of the visual viewport properties and dispatching of events and such currently relies on MobileViewportManager. We'd have to add new codepaths if we wanted e.g. visual viewport resize events to be fired without apz.allow_zooming=true.
Seeing as the motivating uses of the Visual Viewport API involve zooming, and even if we added such non-zooming codepaths now, they'd go way when we implemented desktop zooming, it seems to me that doing so would be rather pointless.
Thoughts?
|
|
Assignee | |
Comment 3•6 years ago
|
||
Intent to ship email: https://groups.google.com/forum/#!topic/mozilla.dev.platform/Mkw_Toh9Siw
|
|
Assignee | |
Updated•6 years ago
|
|
|
Assignee | |
Updated•6 years ago
|
|
|
Assignee | |
Comment 4•6 years ago
|
||
|
||
Comment 6•6 years ago
|
||
| bugherder | ||
|
|
Assignee | |
Comment 7•6 years ago
|
||
Updating bug title to reflect that the enablement by default happened on Android only for now.
Enablement by default on desktop is tracked in bug 1551302 and depends on desktop zooming.
|
||
Comment 8•5 years ago
|
||
Documentation updates:
- Submitted [BCD PR 4347)(https://github.com/mdn/browser-compat-data/pull/4347) noting that
VisualViewportis now enabled by default on Android Firefox. - Listed on Firefox 68 for developers
Description
•