Open Bug 1516471 Opened 6 years ago Updated 2 months ago

Zoom location isn't restored after exiting fullscreen

Categories

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

Product:
Core
Component:
Panning and Zooming
Version:
66 Branch
Platform:
ARM
Android
Type:
defect

Tracking

()

Tracking Flags:
Tracking Status
firefox64 --- wontfix
firefox65 --- wontfix
firefox66 --- wontfix
firefox67 --- wontfix
firefox68 --- wontfix
firefox69 --- fix-optional
firefox70 --- fix-optional

People

(Reporter: csheany, Unassigned)

References

(Blocks 1 open bug)

Details

(Keywords: regression, Whiteboard: [geckoview:fenix:p3])

User Agent: Mozilla/5.0 (Android 7.1.1; Tablet; rv:66.0) Gecko/66.0 Firefox/66.0 Steps to reproduce: 1. Open https://www.mozilla.org/en-US/firefox/ 2. Zoom in on the right side 3. Enter fullscreen of a video 4. Exit Actual results: The page is zoomed in on the left Expected results: The page should still be zoomed in on right
Is this caused by Bug 1498812?
Hi, thanks for your report. I can confirm this issue on all branches (Nightly 66.0a1 2018-12-26, Beta 65.0b5 and Release 64.0.1) using Samsung Galaxy Tab S3 (Android 8.0), Huawei P9 Lite (Android 6.0) and OnePlus 5T (Android 8.1.0). Also, I found that the scroll position is changed after exiting full-screen mode. This issue occurs on multiple sites not only on the link from the description.
Status: UNCONFIRMED → NEW
status-firefox64: --- → affected
status-firefox65: --- → affected
status-firefox66: --- → affected
Ever confirmed: true
OS: Unspecified → Android
Hardware: Unspecified → ARM
See Also: → 1498812
(In reply to csheany from comment #1) > Is this caused by Bug 1498812? Good question! I believe that, unlike page navigation (bug 1509575), entering/exiting fullscreen mode does not involve the session store, and therefore, this is unrelated to bug 1498812. Rather, this is a regression from bug 1493976. In that bug we changed the behaviour of entering/exiting fullscreen to zoom out when entering, and zoom back in when exiting. Zooming out and then back in can naturally lead to losing the visual scroll position (since the original scroll position may no longer be valid at the lower zoom level). To fix this, we'll need to extend the approach taken in bug 1493976, to also save and restore the visual scroll position. As the relevant code runs on the main thread, restoring the visual scroll position depends on bug 1507279.
Blocks: 1493976
Component: General → Panning and Zooming
Depends on: 1507279
Product: Firefox for Android → Core
See Also: 1498812 →
Version: Firefox 66 → 66 Branch
Blocks: 1520079

[geckoview:fenix:p3] because we should check whether this works in Fenix.

status-firefox65: fix-optional → wontfix
status-firefox66: affected → fix-optional
Whiteboard: [geckoview] → [geckoview:fenix:p3]

Botond, is this something we can fix in 67?

Flags: needinfo?(botond)

Given the other things I have in the queue for 67, I think 68 is more realistic. The regression seems to be fairly low impact, so I'd say shipping it in one more release is fine.

status-firefox66: fix-optional → wontfix
status-firefox67: affected → fix-optional
Flags: needinfo?(botond)

FWIW, since Bug 1423013 the page is technically zoomed in upon loading.

Bulk change for all regression bugs with status-firefox67 as 'fix-optional' to be marked 'affected' for status-firefox68.

status-firefox68: --- → affected

This is also a good test case

https://downloadfiletest-eab8a.firebaseapp.com/

Happy to take a patch for 70 but since this is triaged and set to P3 priority I'm setting it as fix-optional.
That will remove the bug from weekly regression triage.

status-firefox70: affected → fix-optional
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.