Not that you were asking, but I've repro-ed here. To do so, you seem to have to remove all app styling. Then the default background color is black not white (tests on Nexus4 running 4.4 as suggested by Joe).
You won't notice this with the default cordova project, nor mobile-spec, since they set background color in its default styling. That likely explains why it wasn't caught during the release testing. Perhaps we should add a (few?) pages to mobile-spec that we should confirm render correctly before a release? -Michal On Wed, Sep 3, 2014 at 6:38 PM, Joe Bowser <[email protected]> wrote: > At any rate, I'm going to re-tag the release on Android, since we can't > release the current tag. > > > On Wed, Sep 3, 2014 at 3:32 PM, Joe Bowser <[email protected]> wrote: > > > This is the offending commit: > > https://git-wip-us.apache.org/repos/asf?p=cordova-android.git;h=67f474e > > > > I knew that this was a bad idea, but I thought people would actually test > > it themselves since they were convinced that I was wrong. Sadly, we're > now > > dealing with their hubris. > > > > > > On Wed, Sep 3, 2014 at 3:23 PM, Jesse <[email protected]> wrote: > > > >> Can you post a link to the offending commit? > >> > >> @purplecabbage > >> risingj.com > >> > >> > >> On Wed, Sep 3, 2014 at 3:21 PM, Joe Bowser <[email protected]> wrote: > >> > >> > It turns out that the change proposed by a user was merged in without > >> any > >> > testing, or without any testing that would have caught this pretty > major > >> > bug. Of course, we didn't catch it because MobileSpec is black and > our > >> > default background colour is black. I'm going to land the commit in > >> 3.6.0 > >> > and master, but we really need to actually test when we merge things > in > >> and > >> > not just merge them in because the person doesn't like me, which I > think > >> > was the case here. You can't set the colour of an App View without it > >> > having negative consequences on stock Android, or non-Samsung devices. > >> > > >> > I can't believe people approved a release without doing a simple check > >> like > >> > this. This is insane! > >> > > >> > > >> > On Wed, Sep 3, 2014 at 2:58 PM, Joe Bowser <[email protected]> wrote: > >> > > >> > > Hey > >> > > > >> > > I just tested on KitKat the 3.6.0 release and I noticed the page > when > >> you > >> > > first create a Cordova application is Black. It's the correct > colour > >> on > >> > L, > >> > > but that's not super helpful since L isn't even on any devices at > the > >> > > moment AFAIK. > >> > > > >> > > Has there been any changes that would cause the background colour to > >> be > >> > > black? I know I could check the commit log, but I'm hoping this is > >> > faster. > >> > > I'm probably going to -1 the release because of this, since this > >> seems > >> > > like a pretty obvious problem. > >> > > > >> > > > >> > > > >> > > >> > > > > >
