Black theme doesn't set the android status bar to black

[obosob]

** Jerboa Version **
0.0.33

Describe the bug
When in Black theme the status bar on android, remains the same colour as the background colour in Dark mode, rather than matching the theme.

To Reproduce
Steps to reproduce the behavior:

1. Go to 'Settings > Look and feel'
2. Click on 'Theme'
3. Select 'Black'
4. Observe the status bar

[Schoolkid1]

I'm going to pick this issue up for my school project. Does any one knows if there is a coding standard available? If there is not i will of course do my best to work as neat as possible.

[Schoolkid1]

On my phone (Oppo A78) the toolbar does match. Could you tell me what device you are using?

[Schoolkid1]

@dessalines I am not getting response and after checking on my android phone I concluded the issue doesn't show up. Shall we close this issue?

[obosob]

This is on a Google Pixel 6, sorry for late reply.

[Schoolkid1]

For me this problem is not reproduce-able. In combination with the deadline for my school year I am going to focus on another issue. Maybe some one else wants to pick up this issue?

[MV-GH]

What android version?

[Schoolkid1]

It seems the Google Pixel uses a different buildup in the way it shows colors when changing theme, to resolve this issue i added in the Color.kt file an extra value to adjust specifically the toolbar color. I staged the change, can some one check if this value is sufficient to solve this issue?

[MV-GH]

@Schoolkid1 make a PR with your proposed changes

[Schoolkid1]

I made a PR. I wanted to split it between #682 and #661 but it combined the files. Can you check if you can acces the pull request. This is my first PR with GitHub so if there are any mistakes I apologise in advance and I am eager to hear feedback to how I can improve.

[MV-GH]

I do not see the PR. Also the reason your changes are combined, is that you did both changes on the same branch.

Take a look at this article: https://docs.github.com/en/get-started/quickstart/contributing-to-projects

And this one for setting upstream as the above does not seem to mention it
https://devopscube.com/set-git-upstream-respository-branch/

or this but it's pretty lengthy https://git-scm.com/book/en/v2/GitHub-Contributing-to-a-Project

[Schoolkid1]

I do have a link for the PR , it is MV-GH#1

But i will delete my fork and reinstall it and do it following the given steps.

[MV-GH]

Be carefully, with deleting your fork, it will also delete your changes. Keep the fork make a new one, until you have it setup as above and recreated your changes their in separate branch. I see that PR but that one is a PR to my fork. You need to do a PR to this repository (upstream). As you can see it has also many conflicts, this is because it is quite behind on changes from upstream. If you have it setup as above it will be very easy to pull the changes from upstream and then merge them into your branches with the changes.

[Schoolkid1]

New link to PR #1167

Is this better?

[dessalines]

Stale issue, and doesn't seem to be an issue.