-
Notifications
You must be signed in to change notification settings - Fork 790
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
Fix status bar #4
Comments
@housseindjirdeh I can take a peek at this this afternoon! |
Nothing but love for you <3 |
@housseindjirdeh on Android, a simple change works.
But I do not understand why you need a dark status bar, can by default make it light? |
@lex111 On almost every screen a dark status bar should be used because a light one would be impossible to see. However, on profiles, orgs and repos, a light one should be used. |
@andrewda mmm, yeah, thanks, I did not understand at first. But however, a simple property change seems to work. |
I noticed that when the Tab Bar is initiated it loads and renders all its' tab views one by one. The final result is the status bar color is the one defined in AuthProfile, as it is the last screen loaded. It seems that rendering a StatusBar component overrides any previously StatusBar rendered. You can even do things like:
which will render a view with only one status bar styled as "dark-content" (the last one). Maybe this things are obvious, but it was quite surprising to me, so I'm pointing that out. Now, my idea of resolving the issue is to re-render the corresponding screen when you change a tab . That would render only the relevant StatusBar. Unfortunately, I have no idea how to force such re-rendering. Can you help me here? |
Hey, Also, exposing a option in app settings to change status bar color would be great :) |
Assigned @jouderianjr as he's working on it, using |
🎉 |
Screens that use a profile component (repository screen/user profile screen) should have white status bar text. Currently, the text is dark.
I've already exposed a
barColor
prop to modify this attribute in the ViewContainer component which is the wrapper for all screens in the application. However simply changing the value doesn't correctly change the status bar colour as it should, and this probably has to do with how we have nested stack navigations within tabs so it's most likely being overwritten.The text was updated successfully, but these errors were encountered: