Skip to content

fix(web): render Gmail logo in worker flow#2027

Merged
federicodeponte merged 1 commit into
mainfrom
codex/fix-gmail-flow-logo
Jun 26, 2026
Merged

fix(web): render Gmail logo in worker flow#2027
federicodeponte merged 1 commit into
mainfrom
codex/fix-gmail-flow-logo

Conversation

@federicodeponte

@federicodeponte federicodeponte commented Jun 26, 2026

Copy link
Copy Markdown
Member

Closes #2026.

Summary

  • adds DOM regression coverage for WorkerFlow connection chips
  • locks that Gmail renders through the shared BrandLogo sprite (#brand-gmail)
  • locks that unknown/unregistered connection slugs still render as text

Note: current main already contains the component-side ToolChip helper that renders known brand logos; this PR preserves that behavior with focused regression coverage.

Verification

  • npm test -- --project dom tests/worker-flow-brand-logo.dom.test.tsx PASS
  • npm run lint PASS with existing warnings only
  • npx tsc --noEmit FAILS on existing tests/collection-view.dom.test.tsx Element typing errors, unrelated to this change
  • Local browser proof passed: #brand-gmail present, visible lowercase gmail text absent
  • Refreshed proof gist: https://gist.github.com/federicodeponte/29260fea7db1ac29f132b993dee3fe26

@federicodeponte federicodeponte force-pushed the codex/fix-gmail-flow-logo branch from 87554c0 to fdb2f8c Compare June 26, 2026 02:41
@federicodeponte federicodeponte merged commit b4a7931 into main Jun 26, 2026
7 checks passed
@federicodeponte federicodeponte deleted the codex/fix-gmail-flow-logo branch June 26, 2026 02:41
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.

WorkerFlow should render Gmail logo instead of gmail text chip

1 participant