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

macos_icon: Fix skewed logo for Spotlight/Finder #1386

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

Conversation

pournaki
Copy link

macOS icons are skewed for sizes 16x16, 32x32, 48x48 (see issue #1316 ).


What's this PR do?
As previously mentioned in Issue #1316, the app icon in macOS looks skewed in Spotlight, Finder. This PR fixes this look.

Any background context you want to provide?
I created a new icon.icns based on the icons in build/icons using iconutil -c icns.

Screenshots
Before:
Screenshot 2024-09-18 at 10 45 34
After:
image

You have tested this PR on:

  • Windows
  • Linux/Ubuntu
  • macOS (14.6.1)

@alya
Copy link
Collaborator

alya commented Sep 18, 2024

Thanks for the fix! Could you please update your commit message to match the commit style guidelines?

macOS icons were skewed for sizes 16x16, 32x32, 48x48 (see issue zulip#1316 ), affecting their display in Spotlight and Finder.

A fixed `icon.icns` was built based on the icons in `build/icons` using the macOS-internal command `iconutil -c icns`.
@pournaki
Copy link
Author

Sure, does it comply now?

@alya
Copy link
Collaborator

alya commented Sep 20, 2024

@andersk are you up for reviewing this one?

@pournaki
Copy link
Author

pournaki commented Sep 23, 2024

I made a mistake building the new logo. I took the files from build/icons, but they are missing a crucial margin. Without it, the logo looks too big in the app switcher. I am attaching a screenshot that compares a logo file extracted from icon.icns (left) to the same size one from build/icons (right).

Screenshot 2024-09-23 at 14 52 00

I have created a new icon.icns with the correct margins by resizing the 512px logo to the faulty sizes using

$ sips -Z <SIZE> icon_512x512.png --out icon_<SIZE>x<SIZE>.png

Is there a way to update this pull request with the new file? Or should I close it and make a new one?

@pournaki
Copy link
Author

There is still a (probably) unwanted faint white border around the icon, visible in the Finder (pic1) and Spotlight (pic2). I don't think this is an artifact of the resizing operation.

image
image

This line does not appear in the app switcher.

@alya
Copy link
Collaborator

alya commented Sep 24, 2024

Yes, you can push to this PR. https://zulip.readthedocs.io/en/latest/git/using.html might help.

The previously generated `icon.icns` that fixed the skewed logos for small sizes was missing the necessary margin.
@pournaki pournaki changed the title Fix macOS icon. macos_icon: Fix skewed logo for Spotlight/Finder Sep 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants