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)
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.
Reporter | ||
Comment 1•10 years ago
|
||
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)
Comment 2•10 years ago
|
||
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)
Comment 3•10 years ago
|
||
(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)
Comment 4•10 years ago
|
||
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
Reporter | ||
Comment 5•10 years ago
|
||
Why is this expected behavior? It doesn't make sense to me.
Webpages expect the background to be white by default.
Flags: needinfo?(bfrancis)
Comment 6•10 years ago
|
||
(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)
Reporter | ||
Comment 7•10 years ago
|
||
(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)
Reporter | ||
Comment 8•10 years ago
|
||
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.
Comment 9•10 years ago
|
||
> 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)
Reporter | ||
Comment 10•10 years ago
|
||
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)
Reporter | ||
Comment 11•10 years ago
|
||
Also, this is incompatible with how webapps are working on Firefox desktop and Fennec.
Comment 12•10 years ago
|
||
Alive, whose decision was it to make this black, and what was the rationale?
Flags: needinfo?(bfrancis) → needinfo?(alive)
Comment 13•10 years ago
|
||
(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)
Reporter | ||
Comment 14•10 years ago
|
||
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.
Reporter | ||
Comment 15•10 years ago
|
||
(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 → ---
Updated•10 years ago
|
Flags: needinfo?(alive)
Whiteboard: [systemsfe]
Reporter | ||
Comment 16•8 years ago
|
||
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 ago → 8 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•