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

iOS: Re-Add iOS Dark Icon #11943

Open
wants to merge 22 commits into
base: dev
Choose a base branch
from
Open

iOS: Re-Add iOS Dark Icon #11943

wants to merge 22 commits into from

Conversation

itzTheMeow
Copy link
Contributor

Follow-Up to #11460 (comment)

Same as that PR:
Implements the dark icon for iOS 18, and converts the AppIcon asset to use the "All Sizes" format rather than "All Sizes (Xcode 13)" format.

I tested the icon on the iPhone 16 simulator and it becomes dark and tinted properly.

I may have misunderstood what the Single Size format was for, still not entirely sure since I can't find more documentation.
Also some changes were made to the icon generation script to avoid having to keep copying each asset.
These changes can be applied to the other icon sets in a future PR if necessary.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
itzTheMeow and others added 2 commits March 8, 2025 12:16
@itzTheMeow
Copy link
Contributor Author

itzTheMeow commented Mar 10, 2025

Error: 36:11 error Interface name iOSAppIconContents must match one of the following formats: StrictPascalCase @typescript-eslint/naming-convention

Should this be IosAppIconContents then?

@laurent22
Copy link
Owner

Should this be IosAppIconContents then?

What we do usually is to add an exception here:

'regex': '^(GET|POST|PUT|DELETE|PATCH|HEAD|SQLite|PostgreSQL|ASC|DESC|E2EE|OR|AND|UNION|INTERSECT|EXCLUSION|INCLUSION|EUR|GBP|USD|SJCL.*|iOS)$',

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.

None yet

2 participants