Skip to content

Crop non-square icons to squares - #87

Merged
aphansal123 merged 2 commits into
mainfrom
resize-non-square-icons
Oct 20, 2026
Merged

aphansal123 merged 2 commits into
mainfrom
resize-non-square-icons

Conversation

@aphansal123

@aphansal123 aphansal123 commented Oct 20, 2026 •

Copy link
Copy Markdown
Contributor

Figma, Microsoft Foundry and Impeccable icons were 16:9 banners (512x288 / 384x216) with the logo small in the center. Letterboxing left transparent bars (white in many UIs), so they are now center-cropped to squares (256x256) with the original background filling the frame.

Two commits, per CONTRIBUTING.md (a declaration cannot pin its own commit):

  1. Asset-only commit with the cropped PNGs.
  2. Declaration commit updating catalog-icons.json URLs, sourceUrl and sha256, plus the matching three icon URLs in ai-catalog.json.

Merge with a merge commit or rebase, not squash, so the pinned asset commit stays reachable on main. ai-catalog.json was edited only for the icon URLs; the Catalog workflow's regeneration (unrelated registry drift) is intentionally not included.

Copilot AI max review requested due to automatic review settings October 20, 2026 16:53

This comment was marked as outdated.

@aphansal123
aphansal123 force-pushed the resize-non-square-icons branch from 2bd18fb to bbebb76 Compare October 20, 2026 17:12
@aphansal123
aphansal123 requested review from gokhanarkan and a balanced review from Copilot October 20, 2026 17:14
aphansal123 and others added 2 commits October 20, 2026 13:16
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 11982459-b47d-4559-afe0-ab750c80fc09
…ets [skip ci]

Pins the URLs to the asset commit and updates the SHA-256 digests.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 11982459-b47d-4559-afe0-ab750c80fc09
@aphansal123
aphansal123 force-pushed the resize-non-square-icons branch from bbebb76 to 5aa1382 Compare October 20, 2026 17:16
@aphansal123 aphansal123 changed the title Resize non-square icons to 512x512 Crop non-square icons to squares Oct 20, 2026
@aphansal123
aphansal123 merged commit 53be719 into main Oct 20, 2026
2 checks passed

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟢 Approval recommended

Asset dimensions, pinned bytes, checksums, and matching catalog references were verified, with no unresolved findings.

No open issues 🎉

Review effort: Balanced

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