Skip to content

Bugfix: Status bar does not update when the appearance changes on Android#50

Merged
marcprux merged 3 commits intoskiptools:mainfrom
fhasse95:main
Dec 15, 2025
Merged

Bugfix: Status bar does not update when the appearance changes on Android#50
marcprux merged 3 commits intoskiptools:mainfrom
fhasse95:main

Conversation

@fhasse95
Copy link
Contributor

Currently, when the system appearance changes from light to dark mode (or vice versa) on Android, the status bar icons do not update accordingly. As a result, the status bar icons and text may have insufficient contrast (e.g., light icons on a light background or vice versa). This happens because the system bar style is not re-applied when the appearance changes.

Steps to Reproduce:

  • Run the app on Android
  • Switch the system appearance from light to dark mode (or vice versa)
  • Observe the status bar icons
    => The status bar icons keep their previous appearance and do not adapt to the new system theme ⚡

This PR solves the issue by re-applying the system bar style when the system appearance changes, ensuring that the status bar icons always match the current light / dark mode.

Thank you for developing Skip! 😊


Thank you for contributing to the Skip project! Please use this space to describe your change and add any labels (bug, enhancement, documentation, etc.) to help categorize your contribution.

Skip Pull Request Checklist:

  • REQUIRED: I have signed the Contributor Agreement
  • REQUIRED: I have tested my change locally with swift test
  • OPTIONAL: I have tested my change on an iOS simulator or device
  • OPTIONAL: I have tested my change on an Android emulator or device

@cla-bot
Copy link

cla-bot bot commented Dec 13, 2025

Thank you for your pull request and welcome to the Skip community. We require contributors to sign our contributor license agreement (CLA), and we don't seem to have the user(s) @fhasse95 on file. In order for us to review and merge your code, for each noted user please add your GitHub username to Skip's .clabot file

@fhasse95 fhasse95 marked this pull request as draft December 13, 2025 21:00
@fhasse95 fhasse95 marked this pull request as ready for review December 13, 2025 21:00
@fhasse95
Copy link
Contributor Author

@cla-bot check

@cla-bot cla-bot bot added the cla-signed label Dec 13, 2025
@cla-bot
Copy link

cla-bot bot commented Dec 13, 2025

recheck

@marcprux
Copy link
Member

This will be a great improvement! Unfortunately, it isn't working for me. Was there anything else that needs to be done to get it working (like a change to AndroidManifest.xml)?

Screen.Recording.2025-12-13.at.16.29.33.mov

@fhasse95
Copy link
Contributor Author

Hi @marcprux,
thanks for the quick feedback. You’re right. The first change I made only ensured that the status bar was updated when the system appearance mode changed, but not when toggling via the preferred color scheme setting.

I’ve now made a few additional adjustments so that it works correctly in both cases.
Please let me know if it’s working for you now or if any further adjustments are needed.

@marcprux
Copy link
Member

It works great, thanks!

We will also update the default app template (created with skip init/skip create) to also contains this logic.

@marcprux marcprux merged commit 70bec9b into skiptools:main Dec 15, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants