Closed
Bug 823327
Opened 12 years ago
Closed 12 years ago
White flash when going into/out of fullscreen in gallery
Categories
(Firefox OS Graveyard :: Gaia::Gallery, defect)
Tracking
(blocking-basecamp:-, b2g18+)
RESOLVED
DUPLICATE
of bug 820571
| blocking-basecamp | - |
| Tracking | Status | |
|---|---|---|
| b2g18 | + | --- |
People
(Reporter: cjones, Unassigned)
Details
(Whiteboard: UX-P3, BerlinWW)
STR
(1) Open gallery
(2) Tap an image
We seem to go fullscreen when transitioning to detailed image view. There's a paint of only white screen when we make this transition. The flash also happens when transitioning out of fullscreen.
bb? to get on the radar.
|
||
Updated•12 years ago
|
blocking-basecamp: ? → -
tracking-b2g18:
--- → +
|
||
Comment 1•12 years ago
|
||
I think this was reported before and fixed, but is now back again.
I'm almost certain it is a gecko issue, not gaia.
I wonder if there is any gaia trick I can use to hide the white... I don't know if I've got a background set on <body> for example.
Is it worth abandoning fullscreen mode and always showing the status bar to avoid this graphical jank?
I wonder if the white rectangle on the fullscreen transition is related to the white rectangle during the orientationchange transition (see bug 825848)
If we could fix both of those bugs by not using fullscreen mode, would it be worth it?
I also see this issue when:
1. open gallery
2. tap on image
3. press home button.
You'll see it flash white before exiting to the home screen.
I would like to see if there is a gaia fix before we consider abandoning the full-screen mode.
|
|
||
Comment 3•12 years ago
|
||
The flash of white we see is white because of this code in apps/system/style/system/system.css:
#windows > iframe.appWindow:not(.homescreen) {
background-color: #fff;
}
But if I change this line to anything other than its current white, it changes the background color of other apps that do not set their own background. So I can fix it there.
Setting the background color in the gallery app itself does not help.
Does the patch in bug 820571 fix this?
|
|
||
Comment 5•12 years ago
|
||
I can't test the patch right now. And, I've got a patch pending in 829362 to stop using mozRequestFullscreen() in the gallery app and just set the fullscreen flag in the manifest.
I'll get that patch landed anyway.
|
||
Comment 7•12 years ago
|
||
So, the suggested fix by David in https://bugzilla.mozilla.org/show_bug.cgi?id=823327#c3 definitely fixes the problem as far as I can see testing on the Unagi. Any reason why this has not been implemented?
|
|
||
Comment 8•12 years ago
|
||
(In reply to Schalk Neethling [:espressive] from comment #7)
> So, the suggested fix by David in
> https://bugzilla.mozilla.org/show_bug.cgi?id=823327#c3 definitely fixes the
> problem as far as I can see testing on the Unagi. Any reason why this has
> not been implemented?
There is a typo in comment 3: "so I can fix it there" should be "so I CANT fix it there". Changing that setting to black would break any app that does not set its own background color, and there are a number of them.
Meanwhile, I've fixed the gallery app so that it now just uses homescreen for everything, and doesn't have those flashy transitions anymore.
|
||
Updated•12 years ago
|
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → DUPLICATE
You need to log in
before you can comment on or make changes to this bug.
Description
•