Full Screen Video on Firefox Nightly displays the Android notification area in grey.
Categories
(Fenix :: Onboarding, defect)
Tracking
(Not tracked)
People
(Reporter: raj.azharuddin, Unassigned)
References
Details
Attachments
(3 files)
+++ This bug was initially created as a clone of Bug #1843244 +++
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:109.0) Gecko/20100101 Firefox/115.0
Firefox for Android
Steps to reproduce:
I am using Firefox Nightly on Android.
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:109.0) Gecko/20100101 Firefox/115.0
This is rather simple. I was watching a video full screen on Nightly and the far left of the screen, the area which corresponds to the notification area in vertical mode, is grey. Not the expected black. It makes the front facing camera stand out and also gives me burn in concerns.
Installing the stable Firefox version this works as intended.
Actual results:
Area is grey.
Expected results:
Area should be black.
Reporter | ||
Comment 1•1 years ago
|
||
It was fixed few version back but the bug again came back 😞
Comment 2•1 years ago
|
||
Hey! Thanks for helping out with tracking the bugs :)
Could you please confirm that this bug is reproducible again on the latest nightly? Could you provide the exact build number from the "About Firefox Nightly" screen?
Reporter | ||
Comment 3•1 years ago
|
||
Yes bug is reproducible in latest nightly
118.0a1 (Build #2015968291)
Attaching a screenshot
Reporter | ||
Comment 4•1 years ago
|
||
Comment 5•1 years ago
|
||
doesn't the strip look black?
Reporter | ||
Comment 6•1 years ago
|
||
Yes status bar/notch area is black. The video is not filling the notch area
Comment 7•1 years ago
|
||
So, in the screen shot the notch area is black, not gray. There is no bug then, right? should I close this?
Reporter | ||
Comment 8•1 years ago
|
||
But shouldn't the status bar get hidden in full screen? If not then what was fixed in the bug 1836896?
Comment 9•1 years ago
|
||
Could you help providing 1) Device information (including Android Version) 2) Link to the video 3) Is browser in private mode or normal mode , if in normal mode , the theme is light or dark.
Comment 10•1 years ago
|
||
But in that image the area is black.(In reply to raj.azharuddin from comment #8)
But shouldn't the status bar get hidden in full screen? If not then what was fixed in the bug 1836896?
The notch/notification area should be black, not purple. Full screen video tends to be up to that notch. More than that, full screen video doesn't take up the "full screen". Even in your example, it is black all around the video. These ultratall phone resolutions just don't match up to a video resolution.
Reporter | ||
Comment 11•1 years ago
|
||
I guess this is expected and can be closed..sorry for the trouble
Thanks mozilla team 🙂
Reporter | ||
Comment 12•1 years ago
|
||
In YouTube app, when video is stretched it is filling the notch area too
Attaching image from YouTube app and fenix app
Reporter | ||
Comment 13•1 years ago
|
||
Reporter | ||
Comment 14•1 years ago
|
||
Reporter | ||
Updated•1 years ago
|
Comment 15•1 years ago
|
||
(In reply to raj.azharuddin from comment #8)
But shouldn't the status bar get hidden in full screen? If not then what was fixed in the bug 1836896?
It looks like we have two different issues that got mixed up. :)
- Notch area would get colored in horizontal mode (that's fixed in the release by this patch);
- Full screen video doesn't extend to the notch area.
I opened a bug to tackle the second problem. It looks like full screen support hasn't been broken recently, but has never made use of the notch area. I will mark this one as resolved, it looks like we don't have the notch being colored problem any more.
The bug 1836896 you mentioned made the full screen video cover the notch area. But it stopped working after the patch was landed.
(for anyone interested in tech details: 1836896 relied on the parameter <item name="android:windowLayoutInDisplayCutoutMode">shortEdges</item>
, that was inherited by the whole app after we started using SplashScreenTheme
. But we had to default it, and it broke the 1836896 solution.)
Description
•