Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed the statusBar color #16

Merged
merged 6 commits into from Sep 16, 2019
Merged

Fixed the statusBar color #16

merged 6 commits into from Sep 16, 2019

Conversation

yiss
Copy link

@yiss yiss commented Sep 9, 2019

Before the plugins set the status bar color to white. As a consequence, it overrides the status bar color defined by the material swatches.

I looked in the Flutter source code to look for how they generate the material color swatch used the same principles to generate the material color dark for every color

Copy link
Collaborator

@henriquearthur henriquearthur left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

First of all, thank you for your contribution.

I added a few review comments that must be addressed before we can merge this PR.

There is also a unexpected behaviour happening after splash screen disappears: the status bar color remains the splash color. This is happening because color is changed in MainActivity and is not changed back.

This is not caused specifically by your PR but using a custom color on status bar makes it more visible, so we also need to address that.

Thanks again for your contribution. If you can make changes in order to fix those issues I would really appreciate that, otherwise I will look into this in a few days (as soon as I have time!)

Screenshot_1568237843

lib/android.dart Show resolved Hide resolved
lib/templates.dart Show resolved Hide resolved
@henriquearthur henriquearthur changed the base branch from master to dev September 16, 2019 22:58
@henriquearthur henriquearthur merged commit f4ed19b into jonbhanson:dev Sep 16, 2019
henriquearthur added a commit that referenced this pull request Sep 17, 2019
@henriquearthur
Copy link
Collaborator

Issues from the review are fixed and this is now available in v0.1.8+1.

SDK version check is now on Java/Kotlin code as proposed by @yaymalaga.

Thanks guys!

@yiss
Copy link
Author

yiss commented Sep 17, 2019

@henriquearthur thank you for fixing the issue sorry didn't had the time to fix it had a hectic week at work :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants