[meta] Enable desktop zooming on the nightly channel
Categories
(Core :: Panning and Zooming, task, P3)
Tracking
()
People
(Reporter: botond, Assigned: kats)
References
(Blocks 1 open bug, Regressed 2 open bugs)
Details
(Keywords: dev-doc-complete, meta)
Attachments
(1 file)
This meta bug tracks issues blocking the enablement on desktop zooming by default on the nightly channel.
Reporter | ||
Updated•5 years ago
|
Reporter | ||
Updated•5 years ago
|
Reporter | ||
Updated•5 years ago
|
Comment 1•4 years ago
|
||
This should probably be documented at https://developer.mozilla.org/en-US/docs/Mozilla/Firefox/Experimental_features once implemented.
Sebastian
Comment 2•4 years ago
|
||
It's already implemented, so we should document it now I think.
Comment 3•4 years ago
|
||
(In reply to Timothy Nikkel (:tnikkel) from comment #2)
It's already implemented, so we should document it now I think.
Then I guess, I misunderstand the feature. What always worked for me is zooming in and out pinching two fingers on the touchpad of my Windows 10 laptop. Switching apz.allow_zooming
to true
doesn't change that behavior for me.
So could you please shortly explain what the desktop zooming does differently and how it works? That would help a lot to properly describe it on MDN.
Sebastian
Updated•4 years ago
|
Comment 4•4 years ago
|
||
Currently pinching on a desktop or laptop on a touchpad or touchscreen with do a "reflow zoom" where it zooms in discrete steps and things jump around. This is different from "apz" or "pinch" zoom which currently happens on mobile if you do the pinch gestures on the screen and it is a smooth operation where everything gets bigger without reflowing.
Desktop zooming is to bring this smooth type of zoom to desktop.
If you enable apz.allow_zooming on Windows you will be able to do this smooth pinch zooming if you do the pinch gestures on the touchscreen (if it exists) of your laptop. If you have a laptop with a precision touchpad then you can further enable the pref apz.windows.use_direct_manipulation and that should allow you to do the pinch gesture on your touchpad and have it do the smooth pinch zooming.
Does that all make sense?
Comment 5•4 years ago
|
||
Thank you very much for the clarification, Timothy! Unfortunately, my laptop neither has a touch screen nor a precision touchpad (but Dell Touchpad driver), so I can't test it myself right now.
I have added it under https://developer.mozilla.org/en-US/docs/Mozilla/Firefox/Experimental_features#desktop-zooming now (taking Firefox 42 as the earliest version, as that was the version that introduced the preference according to bug 1186004).
Sebastian
Comment 6•4 years ago
|
||
I don't see it on that page now.
Assignee | ||
Comment 7•4 years ago
|
||
I wasn't seeing it either so I made a no-op change to the page to rebuild it and it shows up now.
Assignee | ||
Comment 8•4 years ago
|
||
Depends on D85431
Updated•4 years ago
|
Comment 9•4 years ago
|
||
Is it okay to land this with Bug 1654933 unfixed?
Comment 10•4 years ago
•
|
||
(In reply to Kagami :saschanaz from comment #9)
Is it okay to land this with Bug 1654933 unfixed?
Are you still seeing bug 1654933? That bug should be "fixed" by bug 1655160 (ie flipping a pref so the code that regressed it is no longer active). I'm leaving the bug open to track a proper fix.
Comment 11•4 years ago
|
||
Ah cool, I'm indeed not seeing the issue anymore. Thanks!
Comment 12•4 years ago
|
||
Comment 13•4 years ago
•
|
||
Backed out 3 changesets (Bug 1620055, Bug 1621740) for causing leaks (DManipEventHandler)
Failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=311943853&repo=autoland&lineNumber=14316
https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=311952775&repo=autoland&lineNumber=1777
Backout: https://hg.mozilla.org/integration/autoland/rev/135a148fcdb41e67d97abdafbf11d16cd28f9205
Comment 14•4 years ago
|
||
Dang, the ownership of DManipEventHandler shouldn't be too complicated. Hopefully not hard to figure out.
Comment 15•4 years ago
|
||
From the log:
WARNING: mDmManager->Deactivate() failed: file /builds/worker/checkouts/gecko/widget/windows/DirectManipulationOwner.cpp, line 659
That is why it's leaking. Not sure why it's failing though.
Comment 16•4 years ago
|
||
Moved the dmanip leak to bug 1657073 with fix.
Assignee | ||
Comment 17•4 years ago
|
||
Thanks! I looked at the autoland push and there as another failure as well, which I've filed bug 1657102 for.
Assignee | ||
Updated•4 years ago
|
Comment 18•4 years ago
|
||
Assignee | ||
Comment 19•4 years ago
|
||
Posted to dev-platform and firefox-dev: https://lists.mozilla.org/pipermail/dev-platform/2020-August/025710.html
Comment 20•4 years ago
|
||
bugherder |
Updated•4 years ago
|
Assignee | ||
Comment 21•4 years ago
|
||
Incidentally, metrics showing that now there's a lot of zooming activity on Nightly: https://mzl.la/3iB9rCz
Comment 22•4 years ago
|
||
Looks like we have covered this appropriately on https://wiki.developer.mozilla.org/en-US/docs/Mozilla/Firefox/Experimental_features#UI
So I think the docs for this are completed for now.
Description
•