Skip to content

fix: update OpenAI logo to current brand asset - #13611

Merged
shivamhwp merged 2 commits into
pingdotgg:mainfrom
aaditagrawal:fix/openai-logo-upstream
Sep 25, 2026
Merged

shivamhwp merged 2 commits into
pingdotgg:mainfrom
aaditagrawal:fix/openai-logo-upstream

Conversation

@aaditagrawal

@aaditagrawal aaditagrawal commented Sep 25, 2026 •

Copy link
Copy Markdown
Contributor

What Changed

  • Updates the OpenAI logo across web/desktop, mobile, and marketing using the current Blossom artwork from SVGL #1039.
  • Preserves the new SVG path exactly, adjusting the viewBox to maintain its visible size.
  • Retains existing theme colors and icon sizing behavior.

Why

T3 Code uses the previous OpenAI logo. This updates it to the current artwork, consistent with OpenAI’s brand guidelines.

UI Changes

Previous and updated OpenAI logos, side by side

Validation

  • Full application build and in-app verification completed by the contributor.
  • Verified all three copies match the updated upstream path.
  • git diff --check passes.

Checklist

  • This PR is small and focused
  • I explained what changed and why
  • I included before/after images for the UI change
  • Animation/interaction video: not applicable

Prepared with GPT-6 Astra in Codex.

Summary by CodeRabbit

  • Style
    • Updated the default provider icon shown for Codex and unrecognized providers in the mobile app. Provider-specific icons remain unchanged. The OpenAI icon in the web app has no visible change.

@github-actions github-actions Bot added vouch:unvouched PR author is not yet trusted in the VOUCHED list. size:S 10-29 changed lines (additions + deletions). labels Sep 25, 2026
@macroscopeapp

macroscopeapp Bot commented Sep 25, 2026

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Approved at 66dc956

Macroscope's review found this PR approvable — This PR replaces the existing OpenAI artwork across the mobile icon, web icon, and marketing asset while preserving sizing, theming, and component behavior. Its impact is limited to a small, self-contained visual branding update with no product-default or static-analysis changes.

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

@coderabbitai

coderabbitai Bot commented Sep 25, 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: 7f9c82ca-264a-43fa-bfac-23964194925f

📥 Commits

Reviewing files that changed from the base of the PR and between 66dc956 and 16cbd6c.

📒 Files selected for processing (1)
  • apps/web/src/components/Icons.tsx
🚧 Files skipped from review as they are similar to previous changes (1)
  • apps/web/src/components/Icons.tsx

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


📝 Walkthrough

Walkthrough

The mobile Codex and unknown-provider fallback uses a different SVG viewBox and path, with even-odd fill and clip rules. The web OpenAI path declaration is reformatted; its path data and fill and clip rules are unchanged.

Changes

Provider icon artwork

Layer / File(s) Summary
Update SVG icon geometry
apps/mobile/src/components/ProviderIcon.tsx, apps/web/src/components/Icons.tsx
The mobile Codex and unknown-provider fallback uses a different viewBox and path, with even-odd fill and clip rules. The web OpenAI path declaration is split across multiple lines; its path data and rules are unchanged.

Priority: ⬇️ Low

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

Change: Bug fix

Merge Risk: ⚪ Minimal · up to 16cbd

The updated OpenAI artwork is consistent across the three surfaces. No issue identified here needs to be resolved before merge.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 1 functions across 2 files. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly identifies the main change: updating the OpenAI logo to the current brand asset.
Description check ✅ Passed The description includes the required What Changed, Why, UI Changes, and Checklist sections. It also provides validation details and a before/after image for the UI change.
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.
  • Fix all pre-merge checks with AI
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create a new PR

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

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@shivamhwp
shivamhwp merged commit e55e731 into pingdotgg:main Sep 25, 2026
19 of 20 checks passed
github-actions Bot added a commit to omarcresp/t3code-flake that referenced this pull request Sep 25, 2026
## What's Changed
* feat(web): control Android foldables in the Device panel by @juliusmarminge in pingdotgg/t3code#13534
* fix(mcp): preview snapshots fit in the agent's tool output again by @t3dotgg in pingdotgg/t3code#13558
* fix(web): paste after clicking away from the composer lands in it again by @t3dotgg in pingdotgg/t3code#13553
* feat(desktop): keep running threads synced in the background by @t3dotgg in pingdotgg/t3code#13554
* fix(mcp): preview errors tell agents what to do instead by @t3dotgg in pingdotgg/t3code#13559
* feat(web): agents working banner links to the Agents panel by @t3dotgg in pingdotgg/t3code#13572
* fix(web): size the Android fold model from the inner display by @juliusmarminge in pingdotgg/t3code#13574
* fix(clients): a preview app no longer knocks the desktop's own server offline by @t3dotgg in pingdotgg/t3code#13577
* fix(web): keep nested task states out of parent bullets by @dominic-r in pingdotgg/t3code#11477
* feat(release): ship a Linux .deb that updates itself by @t3dotgg in pingdotgg/t3code#13575
* perf(desktop): cache compiled JavaScript between launches by @t3dotgg in pingdotgg/t3code#13501
* fix(dev): one t3.json setup action that works on every OS by @t3dotgg in pingdotgg/t3code#13589
* fix(web): new worktree threads no longer say "checkout" during setup by @t3dotgg in pingdotgg/t3code#13590
* fix(desktop): `t3 app` keeps working after a second desktop app quits by @t3dotgg in pingdotgg/t3code#13585
* fix(usage): price Claude fast-mode requests at the fast rate by @t3dotgg in pingdotgg/t3code#13599
* fix: update OpenAI logo to current brand asset by @aaditagrawal in pingdotgg/t3code#13611
* fix(mobile): render assigned project icons in chat list by @SunkenInTime in pingdotgg/t3code#12810

## New Contributors
* @aaditagrawal made their first contribution in pingdotgg/t3code#13611

**Full Changelog**: pingdotgg/t3code@v0.0.43-nightly.20260925.2237...v0.0.43-nightly.20260925.2251

Upstream release: https://github.com/pingdotgg/t3code/releases/tag/v0.0.43-nightly.20260925.2251
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