Skip to content

fix(web): use marketing Antigravity icon in settings - #13236

Closed
voltcrash wants to merge 3 commits into
pingdotgg:mainfrom
voltcrash:t3code/fix-antigravity-settings-icon
Closed

voltcrash wants to merge 3 commits into
pingdotgg:mainfrom
voltcrash:t3code/fix-antigravity-settings-icon

Conversation

@voltcrash

@voltcrash voltcrash commented Sep 23, 2026 •

Copy link
Copy Markdown

Depends on #13231. This branch starts at the marketing icon fix. GitHub's upstream diff includes that parent change until it merges; review the Settings-only diff for this PR's change.

The web provider icon still showed the older PNG with its built-in dark tile. Downscale the transparent Antigravity artwork from the marketing site to 128 × 128 and embed the resulting PNG as base64 in AntigravityIcon. The marketing image stays unchanged. This updates Settings → Providers and other web views that use the icon.

Before and after

Before After
Settings Providers with the old Antigravity tile icon Settings Providers with the resized transparent Antigravity icon

Verification

  • vp run --filter @t3tools/web typecheck
  • vp lint apps/web/src/components/Icons.tsx
  • vp run --filter @t3tools/web build
  • Reviewed Settings → Providers in the T3 Code preview browser and confirmed the rendered icon uses a 128 × 128 base64 PNG.

Model: GPT-6 (Codex)

Summary by CodeRabbit

  • Style
    • Updated the Antigravity icon’s size and presentation across the marketing page and app. The icon appears larger in harness displays, while the floating hero card uses a simpler card appearance with a smaller image. These adjustments provide a more consistent visual treatment of the Antigravity icon across the affected screens.

@github-actions github-actions Bot added the vouch:unvouched PR author is not yet trusted in the VOUCHED list. label Sep 23, 2026
@macroscopeapp

macroscopeapp Bot commented Sep 23, 2026

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Approved at 3bb2332

Macroscope's review found this PR approvable — This is a narrowly scoped Antigravity icon replacement affecting only marketing presentation and web UI rendering. The web build continues to use established bundled-asset imports, with no changes to provider behavior or product defaults.

You can add or adjust custom eligibility rules. Learn more.

@coderabbitai

coderabbitai Bot commented Sep 23, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository: pingdotgg/t3code/.coderabbit.yaml

Review profile: CHILL

Plan: Advanced

Run ID: 99fb595a-9576-49fd-bd88-cb51970e01c8

📥 Commits

Reviewing files that changed from the base of the PR and between 3bb2332 and 61b1f62.

📒 Files selected for processing (1)
  • apps/web/src/components/Icons.tsx

Included review availability: Your plan provides up to 10 included reviews per hour; 7 remain after this review.


📝 Walkthrough

Walkthrough

The web icon component now embeds the Antigravity PNG data and uses 128×128 dimensions. The marketing page enlarges the harness icon and changes the floating card to default styling with a 74% image.

Changes

Antigravity icon presentation

Layer / File(s) Summary
Restore the embedded Antigravity icon asset
apps/web/src/components/Icons.tsx
AntigravityIcon uses the ANTIGRAVITY_ICON_DATA_URL constant instead of an external asset URL. Its viewBox and image dimensions use 128×128.
Adjust Antigravity marketing styles
apps/marketing/src/pages/index.astro
The harness icon uses a 28×28 size. The floating card uses default styling, and its image uses 74% of the card dimensions.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~5 minutes

Change: Bug fix

Suggested reviewers: juliusmarminge

Merge Risk: ⚪ Minimal · up to 61b1f

The marketing hero displays a smaller Antigravity mark, but this is a localized presentation change; the web icon retains effectively the same artwork. No material merge-blocking behavior is evident.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title clearly describes the main change: updating the Antigravity icon used in Settings.
Description check ✅ Passed The description explains what changed and why, includes before-and-after UI screenshots, and lists verification steps. It does not include the template’s checklist, but the description is otherwise co…
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create a new PR

Comment @coderabbitai help to get the list of available commands.

@github-actions github-actions Bot added the size:S 10-29 changed lines (additions + deletions). label Sep 23, 2026
@juliusmarminge

Copy link
Copy Markdown
Member

Superseded by #13373, which already landed the transparent Antigravity mark in apps/web/src/components/Icons.tsx (and mobile). Closing this as leftover hygiene — the marketing half stays on #13231 / #13365.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S 10-29 changed lines (additions + deletions). vouch:unvouched PR author is not yet trusted in the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants