Skip to content

fix: refresh desktop app icons#291

Merged
mrcfps merged 1 commit intomainfrom
feat/update-icon
Mar 20, 2026
Merged

fix: refresh desktop app icons#291
mrcfps merged 1 commit intomainfrom
feat/update-icon

Conversation

@mrcfps
Copy link
Copy Markdown
Contributor

@mrcfps mrcfps commented Mar 20, 2026

Summary

  • replace the desktop app icon assets under apps/desktop/build/ with the new cross-platform icon set
  • update the generated platform-specific icon files under apps/desktop/build/icons/ so packaged assets stay in sync
  • ensure macOS packaging picks up the new branding by refreshing the top-level build/icon.icns, build/icon.ico, and build/icon.png files

Update the packaged desktop icon assets to use the new cross-platform icon set so macOS, Windows, and Linux builds ship the latest branding.
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review


P2 Badge Restore contrast in the regenerated desktop icon source

Because apps/desktop/build/icon.png now uses very light pastel fills on a transparent background, the generated 16×16/32×32 variants in build/icon.ico and build/icons/linux/... lose most of their contrast on light system surfaces. In Windows Explorer / Start search and light-theme Linux launchers the app icon blends into the background and is much harder to recognize than the previous export, so this branding refresh regresses basic discoverability unless you keep a darker outline/background or ship hand-tuned small-size variants.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@mrcfps mrcfps merged commit fb39f41 into main Mar 20, 2026
6 checks passed
@mrcfps mrcfps deleted the feat/update-icon branch March 20, 2026 04:10
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