Skip to content

Fix macOS Dock/Launchpad icon padding for Fritzing#4313

Open
jaganganesh wants to merge 1 commit into
fritzing:developfrom
jaganganesh:hotfix/macos-tweaks
Open

Fix macOS Dock/Launchpad icon padding for Fritzing#4313
jaganganesh wants to merge 1 commit into
fritzing:developfrom
jaganganesh:hotfix/macos-tweaks

Conversation

@jaganganesh
Copy link
Copy Markdown

This PR updates the macOS application icon asset so the Fritzing icon displays consistently with the system icon set in Dock and Launchpad.

What changed

  • Updated fritzing_icon.icns
  • Adjusted icon padding and sizing for macOS presentation

Why

  • The previous icon padding caused the Fritzing Dock/Launchpad icon to appear misaligned compared to native macOS icons
  • This is a visual polish fix that improves the macOS user experience

Notes

  • No source code changes were made
  • This is an asset-only update
  • Target branch: hotfix/macos-tweaks

…con size/spacing matches the system icon set.
@sonarqubecloud
Copy link
Copy Markdown

@KjellMorgenstern
Copy link
Copy Markdown
Member

This PR is missing a github issue, e.g. it would be good to have a screenshot of the problem. Also, some crucial info is missing

  • screenshot of the problem
  • fritzing version the issue was observed with
  • macos version the issue was observed with
  • fritzing version the fix tested
  • macos versions the fix was tested on
  • screenshot of the fix

@KjellMorgenstern

This comment was marked as outdated.

@jaganganesh
Copy link
Copy Markdown
Author

Hi @KjellMorgenstern, Thanks for the review. I've added the requested information below.

Issue

The Fritzing application icon appears noticeably larger than surrounding applications in Launchpad on macOS.

Observed with

  • Fritzing version: 1.0.7
  • macOS version: 15.7.7

Before (issue)

Screenshot attached showing the oversized icon in Launchpad.
before

After (fixed)

Screenshot attached showing the corrected icon size matching neighboring applications.
after

Please let me know if you need any additional information or further testing.

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.

2 participants