cordova-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Joe Bowser <bows...@gmail.com>
Subject Re: CB-12099 (android) SplashScreen Screen Flicker
Date Sat, 17 Dec 2016 07:34:05 GMT
OK, I read the PR, and I'll admit that I misunderstood part of the
problem.  I have a few concerns about this change:

1. Why does Android need to have a PR at all? The splashscreen plugin can
use edit-config tag to change the manifest to add the theme, which it
really should anyway and the XML files can be copied by the plugin.xml.
2. The bug refers to the flicker, which I haven't seen in quite a while,
but can be attributed to the splashscreen disappearing at the wrong time
and the WebView not being made visible or not rendering properly.  CB-12099
explicitly refers to the weird behaviour of the SplashScreenDelay not
actually being followed, which is why I made my initial comment. It's not
its own bug, but a part of CB-12099
3. I'm not sure why we need the prepare step and why the drawable's colour
can't be changed programmatically in Java instead of pulling it from the
config.xml in prepare.js, which isn't guaranteed to work in earlier
versions of Cordova.

I know this seems like more work, but I think it's possible to do this
without having to change the Plugin API for Splashscreen, also doing so
will also work on existing versions of Cordova, and this fix would probably
consist of a major version bump, because the new splashscreen wouldn't work
on the old versions of Cordova-Android.

On Fri, Dec 16, 2016 at 10:34 PM, Sergey Shakhnazarov <daserge@apache.org>
wrote:

> Hello Joe,
>
> Could you please elaborate - is there a Jira item corresponding to that
> duration issue?
>
> Thanks,
> Sergey.
>
> 16 Дек 2016 г. 23:38 пользователь "Joe Bowser" <bowserj@gmail.com>
> написал:
>
> I think we should figure out why the duration of the Splashscreen is messed
> up before we start messing with the background colour of the application,
> especially since the last time we did that, we broke Hello World!
>
> On Fri, Dec 16, 2016 at 11:32 AM, Sergey Shakhnazarov <daserge@apache.org>
> wrote:
>
> > Hi guys!
> >
> >
> >
> > There’s an issue with Android splashscreen that every app has a black
> flash
> > on start [1].
> >
> > I propose to fix this using the android:windowBackground composed as
> splash
> > image (which we use in splashscreen plugin) laid on top of
> > SplashScreenBackgroundColor
> > [2] (this preference is supported on Windows only as of now and will be
> > particularly useful for transparent images).
> >
> > I would appreciate any feedback on the proposal and prototype
> > implementation [3, 4].
> >
> >
> >
> > [1]: https://issues.apache.org/jira/browse/CB-12099
> >
> > [2]: https://cordova.apache.org/docs/en/dev/config_ref/index.html
> >
> > [3]: https://github.com/daserge/cordova-android/tree/CB-12099
> >
> > [4]: https://github.com/daserge/cordova-plugin-splashscreen/
> tree/CB-12099
> >
> >
> >
> > Please let me know if you have any questions or considerations.
> >
> >
> > Best regards,
> >
> > Sergey Shakhnazarov,
> >
> > Akvelon developer.
> >
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message