Skip to content

fix(header): drop OSS wordmark from Vercel logo - #72

Merged
atinux merged 1 commit into
mainfrom
fix/header-vercel-logo
Oct 27, 2026
Merged

atinux merged 1 commit into
mainfrom
fix/header-vercel-logo

Conversation

@atinux

@atinux atinux commented Oct 27, 2026 •

Copy link
Copy Markdown
Contributor

Removes the "OSS" wordmark next to the Vercel triangle in the header, matching chat-sdk.dev. The triangle now shows at every width, and the link still points to https://vercel.com/oss.

Checked in the dev server at 1280px and 390px: the header shows only the triangle, then the slash and the Comark logo. ESLint passes on the changed file.

Written by an AI agent (Amp) on behalf of @atinux.


Play the walkthrough

Summary by CodeRabbit

  • Style
    • The Vercel link in the app header now displays the triangle logo without the larger wordmark. The small-screen logo remains visible, so the brand link has a simpler appearance across screen sizes.

Show only the Vercel triangle at every width, matching chat-sdk.dev. The link still points to vercel.com/oss.

Amp-Thread-ID: https://ampcode.com/threads/T-01a17c7f-c299-72d4-b5ee-6d78ecf40c6e
Co-authored-by: Sébastien Chopin <atinux@gmail.com>
@vercel

vercel Bot commented Oct 27, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
comark-docs-layer Ready Ready Preview Oct 27, 2026 8:40am UTC

@coldtea-pr-lens

coldtea-pr-lens Bot commented Oct 27, 2026 •

Copy link
Copy Markdown

Architecture

Architecture diagram for comarkdown/comark-docs at e7f16e6

Play the walkthrough


Data flow

No data-flow sequence changed in this PR.


Review

✨ Nothing flagged · reviewed e7f16e6


Tip

Open a diagram on the canvas, then press W or click play to walk through the change one step at a time

🪧 More tips
  • Run npx skills add coldteadotai/pr-lens, then tell your coding agent: "Diagram the change you just made with PR Lens and attach it to the pull request."
  • Run npx @coldtea/pr-lens-cli analyze --base origin/main on a branch, then npx @coldtea/pr-lens-cli render .pr-lens/graph.json. Same lenses, your own model key, before the pull request exists
  • Click the link under each diagram to open it on a canvas you can zoom, pan and step through
  • The diagrams are links. Click one to open it on the canvas, then press W or click play to walk through the change
  • The CLI's render reads .github/pr-lens.yml and applies your renames, exclusions and lane pins at draw time
  • Set github.comment.collapsed: true in .github/pr-lens.yml to fold the comment behind one View architecture and data flow row. Drawing still runs as before
  • Set github.draw: on-demand in .github/pr-lens.yml and PR Lens stops drawing pull requests on its own. Comment @pr-lens draw on a pull request when you want that one drawn
  • Add .github/workflows/pr-lens.yml with coldteadotai/pr-lens/packages/action@v0 and your model provider's key as its api-key to run PR Lens from your own CI. Any /chat/completions endpoint works
  • After a push, tick Redraw to draw the latest commit
  • Switch GitHub to dark mode and the diagrams follow. The moving dots are this pull request's data in motion
  • Set github.description: false in .github/pr-lens.yml to stop PR Lens adding buttons to your PR descriptions

Thanks for using PR Lens! It's free for OSS. If it helped, a star helps other teams find it.  Star PR Lens on GitHub

@coderabbitai

coderabbitai Bot commented Oct 27, 2026 •

Copy link
Copy Markdown
Contributor

Review in Change Stack →

No actionable comments were generated in the recent review.

Recent review info
Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: fde7867c-b9c1-47f2-80d8-3c777b1beb89
Commits

Reviewing files that changed from the base of the PR and between 3f6bfea and e7f16e6.

Files selected for processing: 1
  • app/components/AppHeaderBrand.vue
Files with no reviewable changes: 1
  • app/components/AppHeaderBrand.vue

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

Walkthrough

The Vercel link keeps the triangle SVG without the sm:hidden class. The separate wordmark SVG is removed.

Changes: Vercel Header Brand

Layer / File(s) Summary
Vercel logo markup
app/components/AppHeaderBrand.vue
The triangle SVG is no longer limited by the sm:hidden class. The separate wordmark SVG is removed.

Priority: Low

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

Change: Bug fix

Suggested reviewers: benjamincanac

Merge Risk: Minimal · up to e7f16

The header change is mergeable after normal checks; no issue requiring a fix was identified.

Pre-merge checks: Passed 5
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title clearly and concisely describes the main change: removing the OSS wordmark from the Vercel logo in the header.
Docstring Coverage Passed No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Documentation-only edits and deletions within survi…
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.
Full details: Docstring Coverage

Explanation

No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files.

Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@atinux
atinux merged commit 217ac74 into main Oct 27, 2026
10 of 11 checks passed
@atinux
atinux deleted the fix/header-vercel-logo branch October 27, 2026 08:40

This branch was successfully deployed

1 active deployment
Preview — e7f16e64 Deployed Oct 27, 2026 by vercel[bot]
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.

2 participants