Skip to content

Fix: Update System Bar Theme Handling#725

Merged
yusufnasserdev merged 1 commit intodevelopfrom
fix/sys-bars-color
Aug 27, 2025
Merged

Fix: Update System Bar Theme Handling#725
yusufnasserdev merged 1 commit intodevelopfrom
fix/sys-bars-color

Conversation

@yusufnasserdev
Copy link
Collaborator

Key changes:

  • Enabled edge-to-edge display by calling WindowCompat.setDecorFitsSystemWindows(window, false) in onCreate.
  • Removed deprecated code related to setting the navigation bar color to transparent, as this is handled by the edge-to-edge configuration.
  • Simplified the ApplySystemBarTheme composable by removing redundant conditional logic for API levels below Q regarding navigation bar color.
  • Removed unused imports for Color and toArgb.

This commit refactors how the system bar theme is handled in `MainActivity.kt`.

Key changes:
- Enabled edge-to-edge display by calling `WindowCompat.setDecorFitsSystemWindows(window, false)` in `onCreate`.
- Removed deprecated code related to setting the navigation bar color to transparent, as this is handled by the edge-to-edge configuration.
- Simplified the `ApplySystemBarTheme` composable by removing redundant conditional logic for API levels below Q regarding navigation bar color.
- Removed unused imports for `Color` and `toArgb`.
@yusufnasserdev yusufnasserdev requested review from a team August 27, 2025 15:05
@yusufnasserdev yusufnasserdev self-assigned this Aug 27, 2025
@london-bot
Copy link
Contributor

london-bot bot commented Aug 27, 2025

[London Bot] Metrics Update Status

📊 PR metrics successfully updated for event: closed.

@london-bot
Copy link
Contributor

london-bot bot commented Aug 27, 2025

[London Bot] Auto-update Status

✅ Branch successfully updated from develop.

@github-actions
Copy link

Code Coverage Summary

Total Project Coverage 60.68%

@yusufnasserdev yusufnasserdev merged commit 283843c into develop Aug 27, 2025
12 checks passed
@yusufnasserdev yusufnasserdev deleted the fix/sys-bars-color branch August 27, 2025 15:15
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.

2 participants