Skip to content

Add dark and tinted icons#522

Open
codebymini wants to merge 1 commit intodevfrom
dark-and-tinted-icons
Open

Add dark and tinted icons#522
codebymini wants to merge 1 commit intodevfrom
dark-and-tinted-icons

Conversation

@codebymini
Copy link
Collaborator

  • Adds dedicated iOS app icon variants for Any, Dark, and Tinted appearances (1024.png, 1024-dark.png, 1024-tinted.png).
  • Updates the AppIcon asset configuration to use the single-size iOS format.
  • Removes legacy per-size PNG icon files and updates convert_icon.sh to generate the new Any/Dark/Tinted outputs plus Contents.json.

Apple now supports generating iOS/iPadOS icon variations from a single 1024×1024 asset (and explicitly supports Single Size with Any/Dark/Tinted variants), so the old manually maintained small-size PNG set is no longer required:
https://developer.apple.com/documentation/xcode/configuring-your-app-icon

@marionbarker
Copy link
Collaborator

Test

This works. I had to learn how to configure dark icons.

To change your iPhone to show dark icons (requires iOS 18+), long-press an empty space on your home screen until icons jiggle, tap Edit in the top-left, then select Customize. Choose Dark for dark icons, Light for light, or Auto to switch based on time. You can also choose Tinted for custom colors.

Copy link
Collaborator

@marionbarker marionbarker left a comment

Choose a reason for hiding this comment

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

Approve by test. I would like a developer to give thumbs up for the code review.

@MikePlante1
Copy link
Contributor

If I were to be super nitpicky, I'd say the black is too black compared to the rest of the dark icons... but I'm perfectly okay with merging this as is.

Nothing in the code jumped out at me as incorrect, but I'm not super familiar with a lot of that.

Default Dark
IMG_9187 IMG_9186
Clear Tinted
IMG_9185 IMG_9184

@MikePlante1
Copy link
Contributor

I did get linting/fixems in 68 files when I built with Xcode 26.2, though.

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.

3 participants