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

Update Status Bar to Match Theme Colors Dynamically #2711

Open
wants to merge 4 commits into
base: development
Choose a base branch
from

Conversation

Saifuddin53
Copy link
Contributor

@Saifuddin53 Saifuddin53 commented Sep 20, 2024

Fixes #2710

Please Add Screenshots If there are any UI changes.

Light theme:

Before After
light light

Dark theme:

Before After
dark dark

Please make sure these boxes are checked before submitting your pull request - thanks!

  • Apply the AndroidStyle.xml style template to your code in Android Studio.

  • Run the unit tests with ./gradlew check to make sure you didn't break anything

  • If you have multiple commits please combine them into one commit by squashing them.

window.statusBarColor = colors.primary.toArgb()
WindowCompat.getInsetsController(window, view).isAppearanceLightStatusBars = !useDarkTheme
}
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Use enableEdgeToEdge() API, and set status bar color to Transperant in both light and dark theme.

@therajanmaurya
Copy link
Member

is it ready for final review @niyajali

@niyajali
Copy link
Collaborator

@therajanmaurya looks good to me..and it's ready for the final review

@Saifuddin53
Copy link
Contributor Author

@therajanmaurya can we merge this PR?

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.

Bug: Status Bar Background Not Adhering to Theme
3 participants