macOS packaging: Add better rasterization of icon #10958
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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:
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: