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: Resolves #11119: Add iOS Dark Icon #11460

Merged
merged 16 commits into from
Dec 13, 2024
Merged

Conversation

itzTheMeow
Copy link
Contributor

@itzTheMeow itzTheMeow commented Dec 3, 2024

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.

@itzTheMeow itzTheMeow changed the title iOS: Resolves #11119: iOS: Resolves #11119: Add iOS Dark Icon Dec 3, 2024
@laurent22
Copy link
Owner

Thanks for the PR. I haven't reviewed yet but would you mind sharing screenshots of how it would be with this change?

@itzTheMeow
Copy link
Contributor Author

simulator_screenshot_2469D6C8-7FD7-42DD-93CA-E8F175C43183
simulator_screenshot_D171843C-FF05-4ED1-B08C-CF276E444C14
simulator_screenshot_CA60C8E5-C9C3-424E-8595-32CB0CD2DCD5

packages/app-mobile/ios/Joplin/Joplin.entitlements Outdated Show resolved Hide resolved
packages/app-mobile/ios/Podfile.lock Outdated Show resolved Hide resolved
packages/tools/generate-images.ts Outdated Show resolved Hide resolved
@laurent22
Copy link
Owner

Thanks for the update. I see that you removed the icons of various sizes - those are no longer needed, even for older devices?

@itzTheMeow
Copy link
Contributor Author

From my research, the only devices that would be affected are iOS 12 and under, which the Joplin app doesn't support.
Xcode 14 release notes: https://developer.apple.com/documentation/xcode-release-notes/xcode-14-release-notes#Asset-Catalogs
Documentation for app icons: https://developer.apple.com/documentation/xcode/configuring-your-app-icon#Specify-app-icon-variations

Seems like Xcode added the white space changes, i'll revert.

@laurent22 laurent22 merged commit dc44557 into laurent22:dev Dec 13, 2024
7 checks passed
@laurent22
Copy link
Owner

Thanks a lot for implementing this @itzTheMeow!

laurent22 added a commit that referenced this pull request Jan 8, 2025
This reverts commit dc44557.

Reason: Trying to fix error "Asset validation failed"
@laurent22
Copy link
Owner

@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:

Asset validation failed

Missing Info.plist value. A value for the Info.plist key 'CFBundleIconName' is missing in the bundle 'net.cozic.joplin'. Apps built with iOS 11 or later SDK must supply app icons in an asset catalog and must also provide a value for this Info.plist key. For more information see http://help.apple.com/xcode/mac/current/#/dev10510b1f7. (ID: 6ee761d1-9848-4fb4-b586-1a14eaf8c59c)

Asset validation failed

Missing required icon file. The bundle does not contain an app icon for iPhone / iPod Touch of exactly '120x120' pixels, in .png format for iOS versions >= 10.0. To support older versions of iOS, the icon may be required in the bundle outside of an asset catalog. Make sure the Info.plist file includes appropriate entries referencing the file. See https://developer.apple.com/documentation/bundleresources/information_property_list/user_interface (ID: f28130a8-d88a-439d-b631-9718e6535350)

Asset validation failed

Missing required icon file. The bundle does not contain an app icon for iPad of exactly '167x167' pixels, in .png format for iOS versions supporting iPad Pro. To support older operating systems, the icon may be required in the bundle outside of an asset catalog. Make sure the Info.plist file includes appropriate entries referencing the file. See https://developer.apple.com/documentation/bundleresources/information_property_list/user_interface (ID: 487a277c-a069-4f59-96f6-f3f3d83400e0)

Asset validation failed

Missing required icon file. The bundle does not contain an app icon for iPad of exactly '152x152' pixels, in .png format for iOS versions >= 10.0. To support older operating systems, the icon may be required in the bundle outside of an asset catalog. Make sure the Info.plist file includes appropriate entries referencing the file. See https://developer.apple.com/documentation/bundleresources/information_property_list/user_interface (ID: b534a218-a9d7-4330-bbe3-35f09a6a4fa7)

And it's been reverted in edbb613. Please let me know if you have any suggestions on how to fix this.

@itzTheMeow
Copy link
Contributor Author

Just saw this, sorry. I'll take a look at it when i have time.

@itzTheMeow
Copy link
Contributor Author

What is the build process used for submitting a new release? (xcode version, setup, etc)

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.

Create the iOS 18 Dark icon
2 participants