-
Notifications
You must be signed in to change notification settings - Fork 275
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
[BUG] New Icon Doesn't Match Apple Design Guidelines #746
Comments
Thanks for the report, @Logicer16. @jackbenwillis Have I mangled your asset in the merge? |
Also, thank you @Logicer16 for taking the time to assemble a visual aide. This is greatly appreciated when reporting UI concerns in particular. |
Thanks for the catch, @Logicer16. We'll look to revise the asset, but this is unlikely to happen in 2023. |
If you'd like to use this as template, it should be substantially closer to the Apple guidelines. |
I made an improved icon using Apple's Design Resources - #949. |
Describe the bug
Following up on #695, the new icon is too large and has a smaller corner radius than what Apple's Human Interface Guidelines. This is contrary to the goal stated in #695 (comment):
To Reproduce
Include steps to allow us to reproduce the error. Include your host OS, and the version of Raspberry Pi Imager.
Expected behaviour
The new icon to meet the goal of conforming to the Apple Human Interface Guidelines.
Screenshots or video


Comparing it with other icons in the dock makes the discrepancy quite striking
Placing the icon against Apple's provided template shows exactly by how much the new icon is off.
The micro SD card is fine as is though as it is a tool external from the main icon, which is permitted by the guidelines.
Desktop (please complete the following information):
The name of the OS you are trying to write
N/A
Are you using OS Customisation?
N/A
Additional context
N/A
The text was updated successfully, but these errors were encountered: