Closed Bug 1448528 Opened 3 years ago Closed 3 years ago
Missing exit full-screen button for videos
59 bytes, text/x-review-board-request
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:61.0) Gecko/20100101 Firefox/61.0 Build ID: 20180323221725 Steps to reproduce: Open a video in Firefox Press the full-screen button Actual results: The exit full-screen button icon is missing. Tapping where it should be still works to exit full-screen. Expected results: Icon is present.
Possibly related to Bug 1444489?
Thank you for filing the bug. I most have broken my own patch while histedit the changesets ... This can be fixed by listing fullscreenExitButton.svg in the mobile jar.mn. I will submit a patch soon.
Assignee: nobody → timdream
Status: NEW → ASSIGNED
FWIW, I reported this bug on 1379389. No one could reproduce it. The latest version of Firefox Android where ALL fullscreen controls were drawn properly in desktop mode was 47. Starting with 48, a control in the upper right (info control?) was improperly drawn and the problem got progressively worse (no exit full screen, no settings icon).
(In reply to Ken Riley from comment #5) > FWIW, I reported this bug on 1379389. No one could reproduce it. The latest > version of Firefox Android where ALL fullscreen controls were drawn properly > in desktop mode was 47. Starting with 48, a control in the upper right (info > control?) was improperly drawn and the problem got progressively worse (no > exit full screen, no settings icon). Your bug is unrelated to this one because you are talking about controls implemented by YouTube themselves. We are working on controls offered in Firefox natively here.
Comment on attachment 8962121 [details] Bug 1448528 - Include fullscreenExitButton.svg in mobile package https://reviewboard.mozilla.org/r/230964/#review236584 D'oh. r=me. I wish we had a fix for bug 1422041, which I think might have caught this.
Attachment #8962121 - Flags: review?(gijskruitbosch+bugs) → review+
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/89a60060413f Include fullscreenExitButton.svg in mobile package r=Gijs
Verified as fixed on Nightly 61.0a1 (2018-04-01) using Huawei M3 Lite 10 Tablet (Android 7.0) and Google Pixel XL (Android 8.1.0)
You need to log in before you can comment on or make changes to this bug.