Skip to content

fix(shell+apps): Genspark invisivel na UI, logo HermesOffice correto e nav sem deslocamento - #63

Merged
criptogus merged 2 commits into
mainfrom
fix/genspark-invisivel+logo
Sep 3, 2026
Merged

fix(shell+apps): Genspark invisivel na UI, logo HermesOffice correto e nav sem deslocamento#63
criptogus merged 2 commits into
mainfrom
fix/genspark-invisivel+logo

Conversation

@criptogus

Copy link
Copy Markdown
Owner

Fecha a tarefa de tornar Genspark 100% invisivel na interface (só creditos):

  1. LOGO: hermesoffice-logo.svg tinha o wordmark 'GenOffice' desenhado em paths (rebrand so renomeou o arquivo). Regenerado com icone Hermes H + wordmark 'HermesOffice'.
  2. NAV: corrige deslocamento i18n (navCloud/cloudSubtitle/cloudLoginHint estavam 1 idioma a frente - ex: pt com texto arabe). Reconstruidos 19 blocos.
  3. STRINGS: neutraliza marca Genspark em 19 linguas (setAiGskTools, hints, login, onboarding, aiCreditsExhausted). Provider genspark filtrado do SettingsModal (back end preservado).
  4. AiPanel/ribbon das 5 apps: titulo 'Hermes AI' + HermesMark.

Validado via CDP no app: logo HermesOffice, nav 100% pt sem arabe, zero 'Genspark'/'GenOffice' na tela, ribbon 'Hermes AI' com icone H.

…+ deslocamento i18n

- Logo sidebar: reconstrói hermesoffice-logo.svg com icone Hermes H + wordmark
  'HermesOffice' (o SVG antigo soletrava 'GenOffice' por dentro - rebrand so
  renomeou o arquivo, nao o conteudo vetorial).
- Neutraliza strings de UI com marca Genspark em 19 linguas: setAiGskTools,
  setAiGskToolsDesc, setAiGensparkHint, setAiByokNote, onbNote3, loginGenspark,
  loggedInGenspark, loginNetworkError, accountGenspark. Provider Genspark nao
  aparece mais no dropdown (filtrado no SettingsModal); backend preservado.
- Corrige deslocamento i18n: navCloud/cloudSubtitle/cloudLoginHint estavam com
  valores de um idioma a frente (ex: pt com texto arabe). Reconstruidos 19 blocos.
- AiPanel/ribbon (docs/sheets/slides/pdf/markdown): titulo Hermes AI + HermesMark.

@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

Here are some automated review suggestions for this pull request.

Reviewed commit: 95599616b6

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

'Problema de red: no se pudo conectar con el servicio de IA. Comprueba tu conexión e inténtalo de nuevo',
aiCreditsExhausted:
'Tus créditos de Genspark se han agotado. Recarga en genspark.ai/pricing e inténtalo de nuevo',
aiCreditsExhausted: 'Tus créditos de se han agotado. Recarga en e inténtalo de nuevo',

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Restore complete credit-exhaustion translations

When a user in Spanish or one of the similarly affected Thai, Indonesian, Russian, Arabic, Italian, Polish, Dutch, Malay, Hebrew, Hindi, or Portuguese locales exhausts their credits, this message contains blank nouns and URLs while still instructing them to recharge. The same destructive replacement appears across the Docs, Sheets, Slides, PDF, and Markdown strings; replace these fragments with complete provider-neutral messages like those added for English and French.

Useful? React with 👍 / 👎.

Comment on lines +4 to +6
<stop stopColor="#6E4FF6"/>
<stop offset="0.55" stopColor="#5A3DE0"/>
<stop offset="1" stopColor="#14B8A6"/>

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Use valid SVG stop-color attributes

Because this file is loaded as an external SVG through an <img>, it is parsed as SVG/XML rather than JSX, where the presentation attribute is stop-color, not stopColor. These camel-cased attributes are ignored and the stops default to black, so the new icon loses its intended purple-to-teal gradient.

Useful? React with 👍 / 👎.

Comment on lines +10 to +13
<rect x="1" y="1" width="238" height="238" rx="54" fill="url(#hermes-office-logo-grad)"/>
<path d="M72 72v96M168 72v96M72 120h96" stroke="#ffffff" stroke-width="34" stroke-linecap="round"/>
<circle cx="196" cy="50" r="10" fill="#ffffff" opacity="0.9"/>
<circle cx="214" cy="34" r="6" fill="#ffffff" opacity="0.6"/>

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Preserve the colored logo in dark mode

When the shell is in explicit or system dark mode, home.css still sets --logo-invert: 1 and applies filter: invert(...) to the entire lockup because the previous asset was black-and-white. With this new colored icon, that filter inverts the gradient and turns the white H and comet marks black; update the dark-mode treatment rather than applying whole-image inversion to the new artwork.

Useful? React with 👍 / 👎.

@chatgpt-codex-connector

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-03T13:29:04.657166Z 9559961 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.

@criptogus
criptogus merged commit 05c5512 into main Sep 3, 2026
2 checks passed
@criptogus
criptogus deleted the fix/genspark-invisivel+logo branch September 3, 2026 13: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