cordova-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Joe Bowser <bows...@gmail.com>
Subject Re: Change background color of Android Cordova app
Date Fri, 18 Jul 2014 17:32:48 GMT
You're more than welcome to submit a patch, since you clearly know
what the problem is.

On Fri, Jul 18, 2014 at 10:30 AM, Jan Velecký <VVelda@seznam.cz> wrote:
>
>
>
> Your effort to make Android Cordova looks without bug...
>
>
> Cannon reproduce, not a problem, won't fix...
>
> -----------------------------------------
>
>
> What makes you think I'm interested in this bug?
>
> On Fri, Jul 18, 2014 at 9:52 AM, Jan Velecký <VVe...@seznam.cz> wrote:
>
>
> From when is Android emulator Sony device?!?
>
> You downplay every bug, you are interested in...
>
> -----------------------------------------------------
>
> So, you only tested on Sony devices.
>
> That's a way smaller percentage than even ICS. Last time I checked, it was
> around 1%. You should test on more than one manufacturer's device. While you
> may not like Samsung, the fact is that they are ridiculously common devices
> and should be tested before others. On Jul 18, 2014 9:31 AM, "Jan Velecký" <
> VVe...@seznam.cz> wrote:
>
> Okay, okay, I just have done more testing. It occurs on Xperia X10 mini pro
> running Android 2.3 and Xperia mini pro running 4.0 (both better devices
> than Samsung and HTC you mentioned). But it don't relate with fact, that
> these are qwerty devices – I tested it alson on emulators and 4.0 emulator
> have same problem, whereas 4.1 and up does not have this problem.
>
> So it is only Android 2.3 – 4.0 problem (probably also 2.2), but I cannot
> agree, that this is edge problem. Problem of fourth Android users isn't
> edge...
>
> ----------------------------------------------------------------------------
>
>
> ---------
>
> BTW: I don't know exactly which device you have, since all I really have is
> a photo of it. I don't have any modern Android phones with keyboards, nor am
> I even aware of a modern Android phone with a keyboard other than the Desire
> Z and some weird Samsung Galaxy model on T-Mobile. These usually tend to be
> extremely underpowered, because they are so far from the flagship lines. If
> we knew which device you had in the issue, or anywhere, that would have
> allowed us to be able to reproduce this without doing weird things like set
> the opacity of the HTML tag to 0.
>
> On Fri, Jul 18, 2014 at 6:31 AM, Joe Bowser <bows...@gmail.com> wrote:
>
> So, where's your repro steps? Also, where's the basic info, like which
> device you used to reproduce it. Which version of Android? https://issues.
> (https://issues.) (https://issues.(https://issues.)) apache.org/jira/browse/
> CB-7159 (https://issues.apache.org/(https://issues.apache.org/) jira/browse/
> CB-7159(https://issues.apache.org/jira/browse/CB-7159
> (https://issues.apache.org/jira/browse/CB-7159)))
>
> Also, if this is really just a few hundreds of a millisecond, and this is
> only on your device, which we don't have, this will probably be lowered in
> priority since it's an edge case. Patches are welcome.
>
> On Fri, Jul 18, 2014 at 2:19 AM, Jan Velecký <VVe...@seznam.cz> wrote:
>
> It's so simple reproduce this even without opacity. Simply make new Android
> Cordova project.
>
> Add this to css to make clear that css doesn't play any role here:
>
> * { background-color: black !important }
>
> Then infest <body> with HTML code from http://html-ipsum.com/
> (http://html-ipsum.com/) (http://html-ipsum.com/(http://html-ipsum.com/)) (
> http://html-ipsum.com/(http://html-ipsum.com/)(http://html-ipsum.com/
> (http://html-ipsum.com/))) and ideally form code. Copy it and hold paste for
> 10 seconds for example at <body> to make DOM very big, then this problem
> will be well visible, when you do rotate to landscape mode.
>
> And finally, add <preference name="BackgroundColor" value="0xff000000"/> to
> config.xml which does nothing...
>
> That is it. Stupid simple to reproduce this.
>
> ----------------------------------------------------------------------------
>
>
> ------------------
>
> Delay the loading of the URL without a screenshot? Also, you could run the
> manual test application in test that's marked "Test Background Color". This
> isn't automated because it would require the inclusion of a third party
> library like Robotium, and there's no way to programmatically test the
> colour blue that's worth writing. (Seriously, it'd be yet another library
> that'd take the screenshot within the 2 second delay, and then process the
> image to see if the pixels are blue, a non-trivial amount of work for
> something that someone can do when they need to.)
>
> On Thu, Jul 17, 2014 at 8:23 AM, Andrew Grieve <agri...@chromium.org> wrote:
>
>
> I did it to test whether the backgroundColor preference was working. Do you
> know another way to test that preference?
>
> On Thu, Jul 17, 2014 at 11:12 AM, Joe Bowser <bows...@gmail.com> wrote:
>
> So, this is only an issue if you set the html tag opacity to 0? Why would
> anyone do this?
>
> On Thu, Jul 17, 2014 at 7:56 AM, Andrew Grieve <agri...@chromium.org> wrote:
>
>
> Try setting opacity:0 on the <html> tag and you can reproduce white showing
> instead of your backgroundColor.
>
> On Thu, Jul 17, 2014 at 10:54 AM, Joe Bowser <bows...@gmail.com> wrote:
>
> The WebView element isn't supposed to have the backgroundColor, it's the
> view behind the WebView that's supposed to have the backgroundColor.
>
> Also, I haven't seen this behaviour on any of the tests that we done. Unless
> we can reproduce the issue, we can't fix it, and shouldn't just throw things
> over the wall.
>
> On Thu, Jul 17, 2014 at 7:46 AM, Andrew Grieve <agri...@chromium.org> wrote:
>
>
> I was actually looking at this recently and found that the backgroundColor
> preference is not being applied to the actual WebView element. Didn't want
> to change it because I was changing other things at the time, but likely
> this is just a bug that we should fix.
>
> Filed an issue: https://issues.apache.org/jira/browse/CB-7159
> (https://issues.apache.org/jira/browse/CB-7159) (https://issues.apache.org/
> jira/browse/CB-7159(https://issues.apache.org/jira/browse/CB-7159)) (
> https://issues.apache.org/(https://issues.apache.org/) jira/browse/CB-7159(
> https://issues.apache.org/jira/browse/CB-7159
> (https://issues.apache.org/jira/browse/CB-7159))) ( https://issues.apache.
> org/(https://issues.apache.org/)(https://issues.apache.org/
> (https://issues.apache.org/)) jira/browse/CB-7159( https://issues.apache.
> org/jira/browse/CB-7159(https://issues.apache.org/jira/browse/CB-7159) (
> https://issues.apache.org/jira/browse/CB-7159
> (https://issues.apache.org/jira/browse/CB-7159))))
>
> On Thu, Jul 17, 2014 at 9:56 AM, Sergey Grebnov (Akvelon) < v-se...@
> microsoft.com> wrote:
>
> I suppose this could be done by backgroundColor preference (config.xml)
>
> <preference name="backgroundColor" value="0xFFF" />
>
> Thx! Sergey
>
> -----Original Message----- From: Jan Velecký [mailto:VVe...@seznam.cz] Sent:
> Thursday, July 17, 2014 4:18 PM To: de....@cordova.apache.org Subject:
> Change background color of Android Cordova app
>
> Hello, how can I change background color, which is visible during
> orientation change until WebView is correctly resized. I specified it more
> at http://
>
> stackoverflow.com/questions/24601226/change-background-color-of-app-visible-
>
>
> during-resizing-webview-due-to-device-ori
>
> But, it look stackoverflowers don't know solution to this problem, so I post
> it here. :-)
>
> I hope, someone have had different problem ;-)
>
>
>

Mime
View raw message