Skip to content

Conversation

@rileychh
Copy link
Member

@rileychh rileychh commented Feb 8, 2024

Description

This PR adds support for Themed Icons on Android. This feature allows users running Android 13 or later theme our app icon based on the system theme.

Implementation

  • Re-enable flutter_launcher_icons with the version added support for themed icons.
  • Add android.png with the background removed as the foreground for the adaptive icon.
  • Rename tat.png to original.png, and use this image for iOS and older Android versions.

Testing Instructions

Install the app on Android, drag a shortcut onto the home screen, and enable "Themed Icons" in the launcher settings. Verify that the TAT logo correctly follows the system theme.

Additional Notes

N/A.

c1ydehhx
c1ydehhx previously approved these changes Feb 12, 2024
Copy link
Collaborator

@c1ydehhx c1ydehhx left a comment

Choose a reason for hiding this comment

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

LGTM.

@c1ydehhx c1ydehhx added this to the 1.6 milestone Feb 12, 2024
@rileychh
Copy link
Member Author

rileychh commented Feb 13, 2024

Just found out that the flutter_launcher_icons we use has added support for themed icons in a merged PR. Should we use this tool to generate these icons instead?

edit: to use this unreleased feature, we need to switch to the latest master version.

@rileychh rileychh changed the title feat: add "Themed Icons" for Android dd "Themed Icons" for Android Feb 14, 2024
@rileychh rileychh changed the title dd "Themed Icons" for Android Add "Themed Icons" for Android Feb 14, 2024
@rileychh rileychh force-pushed the android-themed-icons branch from 5be76a7 to 7046e7b Compare February 17, 2024 18:04
@rileychh
Copy link
Member Author

Re-implemented with flutter_launcher_icons. Please review again.

@rileychh rileychh force-pushed the android-themed-icons branch from 7046e7b to 59a00dd Compare February 17, 2024 18:15
@rileychh rileychh requested a review from c1ydehhx February 17, 2024 18:21
@rileychh
Copy link
Member Author

Please note that this PR is blocked by #224. Without a proper splash screen for Android 12 and later, the app will launch with 2 splash screens.

Demonstration:
screen-20240219-164753

Details about double splash screen: https://stackoverflow.com/questions/77098464/play-console-pre-launch-warning-double-splash-screen

Copy link
Collaborator

@c1ydehhx c1ydehhx left a comment

Choose a reason for hiding this comment

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

Since it's blocked by #224, I suggest to change the PR into draft.

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.

3 participants