Skip to content

Add Defi app icon assets - #46

Merged
qeude merged 2 commits into
mainfrom
feat/app-icon
Sep 2, 2026
Merged

Add Defi app icon assets#46
qeude merged 2 commits into
mainfrom
feat/app-icon

Conversation

@qeude

@qeude qeude commented Sep 2, 2026

Copy link
Copy Markdown
Owner

Summary

  • Add the Defi Icon Composer package and window artwork.
  • Add light and dark icon previews to the README.
  • Center the project title and CI badge around the new branding.

Testing

  • Not run (asset and documentation-only change).

- Add light and dark Icon Composer assets
- Showcase the icon in the README
@greptile-apps

greptile-apps Bot commented Sep 2, 2026

Copy link
Copy Markdown

Greptile Summary

The PR adds Defi’s Icon Composer artwork, README previews, and icon compilation to local application and DMG packaging flows.

  • Adds the Icon Composer package and source SVG.
  • Compiles icon resources and adds generated icon metadata to staged application bundles.
  • Adds light and dark branding previews to the README.

Confidence Score: 5/5

The PR appears safe to merge because no blocking failure remains established on the current head.

No blocking failure remains.

Important Files Changed

Filename Overview
Support/Defi.icon/icon.json Defines the layered light and dark Icon Composer artwork used by the application bundle.
script/build_and_run.sh Compiles the icon package into application resources and inserts generated icon metadata before signing.
script/package_dmg.sh Adds the same icon compilation and metadata integration to release DMG packaging.
README.md Centers the project branding and adds theme-aware icon previews.

Reviews (2): Last reviewed commit: "feat(icon): add branded app icon to buil..." | Re-trigger Greptile

Comment thread Support/Defi.icon/icon.json
- Update Defi icon colors
- Compile and embed the icon in local and DMG app bundles
@qeude
qeude merged commit ab8e6d1 into main Sep 2, 2026
2 checks passed
@qeude
qeude deleted the feat/app-icon branch September 2, 2026 22:34
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.

1 participant