Skip to content

fix(icon): aplica o icone H do Hermes em todos os assets de build - #62

Merged
criptogus merged 1 commit into
mainfrom
fix/hermes-app-icon
Sep 3, 2026
Merged

fix(icon): aplica o icone H do Hermes em todos os assets de build#62
criptogus merged 1 commit into
mainfrom
fix/hermes-app-icon

Conversation

@criptogus

Copy link
Copy Markdown
Owner

O sync do upstream reintroduziu o icone do GenOffice (dois retangulos brancos diagonais) nos assets de build. Regenerados do icon.svg do fork (Hermes H, gradiente roxo->aqua + cometa) para shell e docs: icon.png, icon.icns, icon.ico, icon-mac.png e o set multiresolucao icons/.

…ell/docs)

O sync do upstream reintroduziu o icone do GenOffice (dois retangulos brancos
diagonais) nos assets de build. Regenerados a partir do icon.svg do fork
(Hermes H, gradiente roxo->aqua + cometa): icon.png, icon.icns (via iconutil),
icon.ico (magick), icon-mac.png e o set multi-resolucao icons/ (16-1024px).
Aplicado igualmente ao apps/docs/build. Nenhuma referencia a GenOffice resta
nos icones.
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 3, 2026

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review Completed 2026-09-03T11:38:39.651673Z a550020 PR opened
ℹ️ 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" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

icns��dic12���PNG

P2 Badge Restore the standard inset in the macOS icon

On macOS builds, the regenerated ic10 representation now has nontransparent content spanning the entire 1024×1024 canvas, whereas the previous icon used the repository's documented 824/1024 content grid (100 px inset) in tools/gen-file-association-icons.mjs:39-42. Neither app overrides the mac icon, and the installed electron-builder schema confirms that the application icon defaults to build/icon.icns, so both packaged apps will appear roughly 24% oversized relative to neighboring Dock/Finder icons. Regenerate the .icns files from an inset macOS source rather than the full-bleed raster.

ℹ️ 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".

@criptogus
criptogus merged commit c78da1d into main Sep 3, 2026
3 checks passed
@criptogus
criptogus deleted the fix/hermes-app-icon branch September 3, 2026 11:44
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