Automattic / simplenote-ios

Simplenote for iOS
https://simplenote.com
GNU General Public License v2.0
2.05k stars 284 forks source link

Updated AppIcon assets to work with new ios 18 styles #1660

Closed charliescheer closed 2 months ago

charliescheer commented 2 months ago

Fix

In iOS 18 we are given the ability to customize the look of app icons on our Home Screen. The app icons should be able to respond to light mode, dark mode, and custom schemes. For some reason the AppIcon and the AppIcon-Beta icons were not compatible with this new feature, but the AppIcon-Sustainer was.

So in this PR I have updated the AppIcon and AppIcon-Beta to work with the new Home Screen customizations.

fixes #1658

Test

  1. Launch Simplenote from Xcode. Must be running the iOS 18 Beta, so you will need an Xcode beta to test this

  2. After the app launches go to the Home Screen, long press on the background and then tap Edit > customize.

  3. Try all of the following settings and make sure the app icon updates to the correct theme:

    • Small - Light
    • Small - Dark
    • Small -Automatic
    • Small - Tinted
    • Large - Light
    • Large - Dark
    • Large -Automatic
    • Large - Tinted
  4. In Xcode go to the project navigator and select the Simplenote target > General. Scroll down to App Icon and Home Screen.

  5. Next to App Icon tap on the three dots and find Debug. Change the value from AppIcon-Beta to AppIcon

  6. Repeat steps 1-3 for the release app icon

Review

(Required) Add instructions for reviewers. For example:

Only one developer and one designer are required to review these changes, but anyone can perform the review.

Release

(Required) Add a concise statement to RELEASE-NOTES.txt if the changes should be included in release notes. Include details about updating the notes in this section. For example:

RELEASE-NOTES.txt was updated in d3adb3ef with:

Added markdown support

If the changes should not be included in release notes, add a statement to this section. For example:

These changes do not require release notes.

dangermattic commented 2 months ago
1 Warning
:warning: This PR is assigned to the milestone 4.54. This milestone is due in less than 4 days.
Please make sure to get it merged by then or assign it to a milestone with a later deadline.

Generated by :no_entry_sign: Danger

wpmobilebot commented 2 months ago
Simplenote Prototype Build📲 You can test the changes from this Pull Request in Simplenote Prototype Build by scanning the QR code below to install the corresponding build.
App NameSimplenote Prototype Build Simplenote Prototype Build
Build Numberpr1660-d52df74-0191ee7a-a04a-426a-822b-683b3b57f59e
Version4.53
Bundle IDcom.codality.NotationalFlow.Alpha
Commitd52df74f223742334f0ec83ff0e8752d7607adfb
App Center BuildSimplenote - Installable Builds #378
Automatticians: You can use our internal self-serve MC tool to give yourself access to App Center if needed.