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 packaging: Add better rasterization of icon #10958

Merged
merged 2 commits into from
Oct 14, 2022

Conversation

fwcd
Copy link
Member

@fwcd fwcd commented Oct 12, 2022

As a small follow-up to #4545, this PR adds a better rasterization of the macOS icon.

The version added in the original PR used an ICNS that was generated by downscaling a 1024x1024 render of the icon. This works fine for most use-cases and with the widespread use of high dpi displays usually isn't really a big issue. However, on non-retina displays there are some noticeable artifacts with the smaller versions. For example, consider the Mixxx icon in this dialog:

image

The icon noticeably lacks its gray border. Contrast this with e.g. the icons for Preview or GIMP, which feature crisp borders that are optimized for the smaller icon size.

This PR addresses this by rendering every size individually from the source SVG into the ICNS (using this script). With the new ICNS, the icon renders properly at all sizes:

Screen Shot 2022-10-13 at 01 04 27

@daschuer
Copy link
Member

Cool, thank you for taking care.
Can you also put the script in the Mixxx's tool folder?

@ronso0
Copy link
Member

ronso0 commented Oct 13, 2022

Can you also put the script in the Mixxx's tool folder?

I couldn't find clear info on gist licenses.
Can someone clarify this before the script (based on another script) is included in Mixxx?

@daschuer
Copy link
Member

So we need to ask the original author. @fwcd can you do it?

https://zebracatzebra.com/oss/getting-the-gist-of-github-gist-licensing/
... explains it and has even a template message.

@fwcd
Copy link
Member Author

fwcd commented Oct 13, 2022

Not a lawyer, but isn't the snippet sufficiently trivial to not be copyright-protectable?

@fwcd
Copy link
Member Author

fwcd commented Oct 13, 2022

If you'd like, I could rewrite it from scratch for Mixxx (it would in any case probably require a few tweaks to the paths)

@fwcd
Copy link
Member Author

fwcd commented Oct 13, 2022

I have added a new version of the script in packaging/macos/regenerate_icns.sh that is

Thus there hopefully shouldn't be any licensing issues.

Copy link
Member

@daschuer daschuer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, Thank you.

@daschuer daschuer merged commit ab867e2 into mixxxdj:main Oct 14, 2022
@fwcd fwcd deleted the improve-icns branch October 14, 2022 21:56
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.

3 participants