-
-
Notifications
You must be signed in to change notification settings - Fork 5.1k
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: Resolves #11119: Add iOS Dark Icon #11460
Conversation
Thanks for the PR. I haven't reviewed yet but would you mind sharing screenshots of how it would be with this change? |
packages/app-mobile/ios/ShareExtension/ShareExtension.entitlements
Outdated
Show resolved
Hide resolved
Thanks for the update. I see that you removed the icons of various sizes - those are no longer needed, even for older devices? |
From my research, the only devices that would be affected are iOS 12 and under, which the Joplin app doesn't support. Seems like Xcode added the white space changes, i'll revert. |
Thanks a lot for implementing this @itzTheMeow! |
This reverts commit dc44557. Reason: Trying to fix error "Asset validation failed"
@itzTheMeow, unfortunately I had to revert this change for now as it's not being accepted by Apple's servers. The errors I'm getting are these ones:
And it's been reverted in edbb613. Please let me know if you have any suggestions on how to fix this. |
Just saw this, sorry. I'll take a look at it when i have time. |
What is the build process used for submitting a new release? (xcode version, setup, etc) |
Resolves #11119.
Implements the dark icon for iOS 18, and converts the AppIcon asset to use the "Single Size" format rather than "All Sizes (Xcode 13)" format.
I tested the icon on the iPhone 16 simulator and it becomes dark and tinted properly.