Closed Bug 1093671 Opened 10 years ago Closed 8 years ago

Black background while webapp is still loading

Categories

(Firefox OS Graveyard :: Gaia::System, defect)

x86
macOS
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: martijn.martijn, Unassigned)

Details

(Keywords: testcase, Whiteboard: [systemsfe])

This looks similar to bug 1052497. I can reproduce this on b2g desktop and on the Flame device, trunk. Steps to reproduce: - Open the browser and go to http://moz.kantjils.nl/webapps/slow_webapp/appinstall.html - Tap on the Install App button - In the dialog, tap on Install - Open the installed webapp Expected result: - White background, so you can see the text that is counting the page Actual result: - Black background, which makes the majority of the page unreadable while it is loading When the page has finally finished loading, the background goes finally white.
Ben, do you know who should own this? It seems a similar fix as in bug 1052497 is needed.
Component: General → Gaia::System
Flags: needinfo?(bfrancis)
Alive, it looks like this intentionally set to transparent so that the black background shows through? https://github.com/mozilla-b2g/gaia/blob/master/apps/system/style/window.css#L20 Eric, if browser windows are supposed to have a white background while loading, are app windows meant to be black or white?
Flags: needinfo?(epang)
Flags: needinfo?(bfrancis)
Flags: needinfo?(alive)
(In reply to Ben Francis [:benfrancis] from comment #2) > Alive, it looks like this intentionally set to transparent so that the black > background shows through? > https://github.com/mozilla-b2g/gaia/blob/master/apps/system/style/window. > css#L20 > > Eric, if browser windows are supposed to have a white background while > loading, are app windows meant to be black or white? Yes, it's intentional to be black "before" the page loaded. I remembered there was an argue about to be black or to be white in the past...
Flags: needinfo?(alive)
Thank you Alive, that's what I needed to know. Marking invalid as this is expected behaviour.
Status: NEW → RESOLVED
Closed: 10 years ago
Flags: needinfo?(epang)
Resolution: --- → INVALID
Why is this expected behavior? It doesn't make sense to me. Webpages expect the background to be white by default.
Flags: needinfo?(bfrancis)
(In reply to Martijn Wargers [:mwargers] (QA) from comment #5) > Why is this expected behavior? It doesn't make sense to me. > Webpages expect the background to be white by default. If you load the web page in a browser window, the default background colour will be white. But web apps are a brave new world! Lots of people filed bugs about a "flash of white" when certain apps loaded which I'm guessing led to the decision for the background to be black. Apparently black is the new white! But seriously, is there actually a web standard specification somewhere which says the default background colour of a web page should be white? I've never seen one.
Flags: needinfo?(bfrancis)
(In reply to Ben Francis [:benfrancis] from comment #6) > (In reply to Martijn Wargers [:mwargers] (QA) from comment #5) > > Why is this expected behavior? It doesn't make sense to me. > > Webpages expect the background to be white by default. > > If you load the web page in a browser window, the default background colour > will be white. But web apps are a brave new world! Lots of people filed bugs > about a "flash of white" when certain apps loaded which I'm guessing led to > the decision for the background to be black. Apparently black is the new > white! A web app is just a web page without the normal brower UI. Now, people see "flash of black" when apps are loaded (which I've seen too). Is that really an improvement at all? I think it's just as bad. > But seriously, is there actually a web standard specification somewhere > which says the default background colour of a web page should be white? I've > never seen one. I don't know. Boris, do you know? But every browser has a default white background for its viewport, that makes the text readable (because the text-color is black), for pages that don't have any styling.
Flags: needinfo?(bzbarsky)
And note that we're changing the background here. While loading, the viewport background is black, then when finished loading, the viewport is turned white.
> is there actually a web standard specification somewhere which says the default > background colour of a web page should be white? https://html.spec.whatwg.org/multipage/rendering.html#rendering says: The canvas' background is expected to be white. Like most of that section, it's a "if you want to be interoperable with real-world content" kind of thing, of course.
Flags: needinfo?(bzbarsky)
In the light of comment 9, can this be reconsidered? Please? Webapps are supposed to be interoperable with real-world content, right?
Flags: needinfo?(bfrancis)
Also, this is incompatible with how webapps are working on Firefox desktop and Fennec.
Alive, whose decision was it to make this black, and what was the rationale?
Flags: needinfo?(bfrancis) → needinfo?(alive)
(In reply to Ben Francis [:benfrancis] from comment #12) > Alive, whose decision was it to make this black, and what was the rationale? Sorry I didn't remember, but maybe Josh(!). We could change for sure! There was also a proposal to make app be able to specify the default color of the splash background, but didn't be realized for some reason.
Flags: needinfo?(alive)
It's not a splash background, fwiw. It's the website being displayed, while it's getting loaded. So the website can already specify the background color, but this bug happens when the website doesn't have a background-color specified.
(In reply to Alive Kuo [:alive][NEEDINFO!] from comment #13) > (In reply to Ben Francis [:benfrancis] from comment #12) > > Alive, whose decision was it to make this black, and what was the rationale? > > Sorry I didn't remember, but maybe Josh(!). We could change for sure! > There was also a proposal to make app be able to specify the default color > of the splash background, but didn't be realized for some reason. You mean Josh Cheng? I'm reopening this, because this is clearly a bug, imo.
Status: RESOLVED → REOPENED
Flags: needinfo?(alive)
Resolution: INVALID → ---
Flags: needinfo?(alive)
Whiteboard: [systemsfe]
Marking WONFTIX, sorry for the bug spam. If somebody still wants to work on this, please open a new bug.
Status: REOPENED → RESOLVED
Closed: 10 years ago8 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.