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

Rework splash screen #224

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Rework splash screen #224

wants to merge 5 commits into from

Conversation

rileychh
Copy link
Member

@rileychh rileychh commented Feb 13, 2024

Description

This PR replaces splash screens across all platforms. NPC branding has been added to the bottom of the screens. Details about the changes can be found in the commit messages.

Implementation

  • Add splash-android_12.png and its dark variant.
  • Add branding.png and its dark variant.
  • Remove unused images from the assets directory.

Testing Instructions

Launch the app on each platform in both light mode and dark mode to check that the images are correctly displayed.

Additional Notes

  1. Due to the requirements of the new SplashScreen API in Android 12, the text "我的北科生活 TAT" has been removed from the splash screen.
  2. Once this upstream issue is resolved, we will be able to add splash screen animations.
  3. For easier review, consider reviewing each commit separately and skip the generated files.

@rileychh rileychh requested a review from ntut-xuan March 24, 2024 14:10
@rileychh rileychh force-pushed the rework-splash-screen branch from 276a30f to daf3a8c Compare December 4, 2024 05:50
@rileychh rileychh added this to the 1.7 milestone Dec 4, 2024
@rileychh rileychh force-pushed the rework-splash-screen branch 2 times, most recently from 563a5ab to 0229daf Compare December 7, 2024 16:18
@ntut-xuan
Copy link
Collaborator

ntut-xuan commented Dec 8, 2024

Hey, could you update the branch please?

@rileychh rileychh force-pushed the rework-splash-screen branch from 0229daf to 771557c Compare December 8, 2024 13:37
@rileychh
Copy link
Member Author

rileychh commented Dec 8, 2024

Hey, could you update the branch please?

Sure, rebased onto main 😀

rileychh and others added 5 commits January 31, 2025 16:01
- Remove unused tat-round.png and tat-transparent-white.png
- Remove explicit default configuration in flutter_native_splash section of pubspec.yaml
- Add splash-android_12.png and its dark variant. The image is 1152x1152, and the icon fits within a circle 768 pixels in diameter.
The branding image is based on `NPC text.svg` found in the club Drive. The text itself is half of the image size. The colors have contrast ratio of about 2 to the background.
Run `dart run flutter_native_splash:create`.
@rileychh rileychh force-pushed the rework-splash-screen branch from 771557c to a2a88b9 Compare February 2, 2025 04:36
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