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

Updated MacOS icon to more closely fit Apple's design standards #852

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

rhysbeynon
Copy link

This is a duplicate of my mis-appropriated fork of bitcoin/bitcoin

Mac OS Minor icon change

This is a fix to an outdated format of app icon. I am not trying to change re-invent the wheel with this, I am simply fixing a minor perceived issue. This should not be seen as a subjective design change as it does not change anything substantially.

Apple's design guidelines have changed since the previous iteration of the Bitcoin-Core app icon. As it stands, the current icon is too large, and juts out when viewed alongside other program's icons- especially when viewed in the dock where is sits semi permanently as the node is running on the system.
My solution was to decrease the overall size to more closely fit Apple's current design documentation, keeping the main body of the icon in the body section of MacOS's icon template, while maintaining the same shape, color, and design.
My motives for this method were caused by Spotify's app icon. Fundamentally, it is very similar to Bitcoin-Core (Circle, plain color, iconography inside circe) yet Spotify's icon is sightly smaller and in-bound. This allows their icon to live amongst the other squircles without seeming too out of place.
Another option i considered was making the icon into a squircle. This, i would personally prefer but I didn't opt for that for familiarity.
I consider my small change positive to everyone including people who generally dislike change.
The .icns file was compiled using Apple's own "iconutil" cli program.

I have successfully built this fork and i am using it without issue.

Replaces the outdated .icns icons for the Bitcoin-Qt gui client for MacOS to align more closely to Apple's design standards and documentation.
@DrahtBot
Copy link
Contributor

DrahtBot commented Feb 3, 2025

The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

Reviews

See the guideline for information on the review process.
A summary of reviews will appear here.

@katesalazar
Copy link
Contributor

As an ex macOS user, all I see here now is bytes.

@rhysbeynon
Copy link
Author

rhysbeynon commented Feb 10, 2025

Screenshot 2025-02-10 at 5 50 14 AM
Screenshot 2025-02-10 at 5 50 02 AM

Some screenshots to show size difference in context of other apps in a populated dock.

Top - before
Bottom - after

Smaller icon may appear smaller than the other icons, this may be because the dot underneath is missing, the circle is less area - and looks smaller (because it technically is) or because of the side-by-side comparison. The circle is at the furthest possible safe area for an app icon to be in according to apple's guidelines. It is a lot more visually appealing to use in practice.

@hebasto
Copy link
Member

hebasto commented Feb 10, 2025

The .icns file was compiled using Apple's own "iconutil" cli program.

What are steps to reproduce the new src/qt/res/icons/bitcoin.icns file?

@rhysbeynon
Copy link
Author

The .icns file was compiled using Apple's own "iconutil" cli program.

What are steps to reproduce the new src/qt/res/icons/bitcoin.icns file?

I cloned bitcoin.icns, used the command iconutil -c iconset bitcoin.icns to convert the assembled icon file into a png IconSet. I then used Affinity Photo to make programmable size changes to each png identically. With the changed icon set, I then used iconutil -c icns bitcoin.iconset to assemble the modified .icns file. This process is used for display scaling reasons.

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.

4 participants