mirror of
https://github.com/mozilla/gecko-dev.git
synced 2024-12-13 18:27:35 +00:00
d94292fc70
The DoubleTapToZoom code starts with the rect for the whole element: let's say it runs the entire height of the (scrollable) page, and the entire width, and the width is enough to make it scrollable horizontally. Then it shrinks the height of the rect so the aspect ratio is the same as the composition bounds. Since the width of the element is wider than the composition bounds at minimum zoom, so too is the height. So it is impossible zoom out far enough to fit the whole target rect. We end up scrolling to the top left of this rect. We want to zoom out so that we center what was visible before zooming out. This will keep whatever was in the center of the screen stationary. One might think we would want to center the rect on the cursor position, but that leads to scrolling document and does not look/feel good. The test runs into bug 1710052, so that the layout scroll position never gets updated. I checked that the test fails locally and on try server without this patch. Differential Revision: https://phabricator.services.mozilla.com/D114588 |
||
|---|---|---|
| .. | ||
| 2d | ||
| angle | ||
| cairo | ||
| config | ||
| docs | ||
| gl | ||
| graphite2 | ||
| harfbuzz | ||
| ipc | ||
| layers | ||
| ots | ||
| qcms | ||
| sfntly | ||
| skia | ||
| src | ||
| tests | ||
| thebes | ||
| vr | ||
| webrender_bindings | ||
| wgpu | ||
| wgpu_bindings | ||
| wr | ||
| ycbcr | ||
| moz.build | ||