Open
Bug 1516471
Opened 6 years ago
Updated 2 years ago
Zoom location isn't restored after exiting fullscreen
Categories
(Core :: Panning and Zooming, defect, P3)
Tracking
()
NEW
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
Comment 2•6 years ago
|
||
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
Updated•6 years ago
|
Priority: -- → P3
Whiteboard: [geckoview]
Updated•6 years ago
|
Keywords: regression
Comment 3•6 years ago
|
||
(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.
Updated•6 years ago
|
Blocks: fullscreen-api
Comment 4•6 years ago
|
||
[geckoview:fenix:p3] because we should check whether this works in Fenix.
Whiteboard: [geckoview] → [geckoview:fenix:p3]
Updated•6 years ago
|
status-firefox67:
--- → affected
Comment 6•6 years ago
•
|
||
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.
Flags: needinfo?(botond)
FWIW, since Bug 1423013 the page is technically zoomed in upon loading.
Comment 9•6 years ago
|
||
Bulk change for all regression bugs with status-firefox67 as 'fix-optional' to be marked 'affected' for status-firefox68.
status-firefox68:
--- → affected
Updated•6 years ago
|
Updated•5 years ago
|
status-firefox69:
--- → fix-optional
status-firefox70:
--- → affected
Comment 11•5 years ago
|
||
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.
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•