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

Add new styleDarkContent with possibility of selection of default style that is very important for iOS (#164) #181

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

exxbrain
Copy link

@exxbrain exxbrain commented Mar 20, 2020

Platforms affected

All platforms, especially iOS.

Motivation and Context

Fixes #164. Fixes #162.
It has been already fixed but not completely correctly. iOS statusbar has different behaviour for Default and Dark styles.

Description

Added styleDarkContent similar to styleLightContent

Testing

Tested on iPhone SE with iOS 13, Android emulator SDK 29.

Checklist

  • I've run the tests to see all new and existing tests pass
  • I added automated test coverage as appropriate for this change
  • Commit is prefixed with (platform) if this change only applies to one platform (e.g. (android))
  • If this Pull Request resolves an issue, I linked to the issue in the text above (and used the correct keyword to close issues using keywords)
  • I've updated the documentation if necessary

@exxbrain exxbrain changed the title Add new styleDarkContent (#164) Add new styleDarkContent with possibility of selection of default style that is very important on iOS (#164) Mar 20, 2020
@exxbrain exxbrain changed the title Add new styleDarkContent with possibility of selection of default style that is very important on iOS (#164) Add new styleDarkContent with possibility of selection of default style that is very important for iOS (#164) Mar 20, 2020
@jcesarmobile
Copy link
Member

I proposed this in #165, but looks like people wanted "default" to be the dark as it used to be, that's why it was implemented like that.

@exxbrain
Copy link
Author

exxbrain commented Mar 22, 2020

I proposed this in #165, but looks like people wanted "default" to be the dark as it used to be, that's why it was implemented like that.

Unfortunately such solution may be confusing in the future because of difference between plugin behaviour and platform documentation for default style. The behaviour was changed in iOS. I think it will be reasonable to do the same for this plugin.

Also the default statusbar behaviour has became unavailable for iOS13 in master branch. This pull request fixes it.

I think it Fixes #162 correctly as well.

@NiklasMerz NiklasMerz added this to the 3.0 milestone Mar 23, 2020
@NiklasMerz
Copy link
Member

To me this makes more sense. Otherwise I have to implement app logic to set the right statusbar color and what makes this harder that there is no explicit dark style.

@svzi
Copy link

svzi commented Jun 5, 2020

Any update on this topic? When to expect a new official release with this PR merged? 🤔

@omargon
Copy link

omargon commented Jul 23, 2020

Anyway we can help push this out? Really needed as it is probably the only pending use case when dealing with light and dark mode settings right now.

@aml25
Copy link

aml25 commented Aug 31, 2020

I'm using @exxbrain PR master and it's working great on iOS.

The behavior is opposite on Android when setting style using config.xml - so <preference name="StatusBarStyle" value="darkcontent" /> yields white text in the status bar on Android, but black on iOS.

If I set the style using JS StatusBar.styleDarkContent(); then it works as expected on Android.

@SunboX
Copy link

SunboX commented Oct 7, 2020

What's the current state on this issue? It's really important on iOS now.

@timbru31
Copy link
Member

timbru31 commented Jan 8, 2021

I'd like to get this PR merged, requesting some more eyes. I feel this is the cleaner approach, too.
I will take care of resolving the conflicts if you are fine, too.

@jcesarmobile
Copy link
Member

I think we should get a release as is and then do another release with this changes and also removing all the styles that don't work since iOS 7 (that has been deprecated for over a year in the plugin but not officially since it wasn't released)

Not to mention that the PR has conflicts and the documentation is misleading since this is iOS 13+ only and that is not mentioned on the doc changes

@nosTa1337
Copy link

Any updates here? It pretty annoying that the statusbar text is white on a bright background in iOS :)

@Maks113
Copy link

Maks113 commented Aug 5, 2021

I also have a problem with this. Can anyone complete the merge?

@tobeagram
Copy link

I've also been having this issue, would love this to be merged asap! In the meantime @exxbrain's PR master works great but is a bit of out date, so I've merged it with the most up-to-date version of the current master.

I've made my repo public here for anyone who needs it to save some time until this PR is merged or an alternative solution is made available.

@simpian
Copy link

simpian commented May 4, 2022

can this be merged to enable black text on light theme on dark mode?

@nvahalik
Copy link

Another 👍🏻 for this patch. We have come across this as well...

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.

Add new styleDarkContent Styles documentation is confusing