Skip to content

Restyle customer story headers to match the editorial site - #3860

Open
marcklingen wants to merge 5 commits into
mainfrom
cursor/customer-story-header-c16c
Open

marcklingen wants to merge 5 commits into
mainfrom
cursor/customer-story-header-c16c

Conversation

@marcklingen

@marcklingen marcklingen commented Sep 19, 2026 •

Copy link
Copy Markdown
Member

The customer story header used the old centered blog treatment (white logo card, default heading font). It now matches /users: Analog title, unboxed logo, left-aligned stack.

Editorial layout, without the redundant “Customer story” label, date, or yellow video tag.

Result

Ramp customer story header with extra space before the story

Canva keeps the author byline:

Canva customer story header with extra space before About Canva

What changed

  • New CustomerStoryHeader for customer stories: breadcrumb, unboxed logo (dark-mode support), Analog H1, description, optional authors
  • BlogHeader routes customer stories through that component; regular blog posts are unchanged
  • Review follow-up: dark logos are resolved from story frontmatter automatically, and the optional hero image prop is forwarded again
  • Extra bottom margin under the header so the story body has more breathing room
Open in Web Open in Cursor 

RetriggerConfidence Score: 4/5

The PR is not ready to merge as the proposed design-review URLs cannot select the advertised masthead and centered variants.

Summary

The PR routes customer-story BlogHeader instances through a new editorial-style header with Analog typography, mono date metadata, unboxed logos, and optional dark-mode logo assets.

  • Regular blog headers remain on the existing rendering path.
  • Ramp now supplies its dedicated dark logo.
  • The selectable masthead and centered variants described for design review are not implemented at the reviewed head.
  • The advertised video-story tag is also absent from the detail header.

Reviews (1) · Last reviewed commit: "Keep the editorial customer story header..."

Replace the boxed, centered blog header on customer stories with three
Analog-based options (editorial, masthead, centered). Default to the
left-aligned editorial layout; compare via ?header= on any story page.

Co-authored-by: marc.klingen <marc.klingen@clickhouse.com>
@vercel

vercel Bot commented Sep 19, 2026 •

Copy link
Copy Markdown

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

Project Deployment Actions Updated
langfuse-docs Ready Ready Preview Sep 21, 2026 10:17am UTC

Request Review

Remove the redundant Customer story label, the video tag, and the unused
masthead/centered variants now that editorial is the chosen layout.

Co-authored-by: marc.klingen <marc.klingen@clickhouse.com>
@marcklingen
marcklingen marked this pull request as ready for review September 19, 2026 12:54
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 19, 2026 •

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-19T12:57:59.070587Z 27862fd Draft marked ready
ℹ️ 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.

@github-actions

Copy link
Copy Markdown

@claude review

@claude claude 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.

Claude Code Review

No review was started: this request came from a bot account. Manual reviews can only be requested by someone with write access to this repository. Ask a maintainer to comment @claude review, or have your automation post the comment from a user account with write access.

Tip: disable this comment in your organization's Code Review settings.

Comment thread components/customers/CustomerStoryHeader.tsx
Comment thread components/customers/CustomerStoryHeader.tsx

@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: 27862fdeaf

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

Comment thread components/blog/BlogHeader.tsx

@cursor cursor 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.

Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 27862fd. Configure here.

Comment thread components/customers/CustomerStoryHeader.tsx
Co-authored-by: marc.klingen <marc.klingen@clickhouse.com>

@claude claude 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.

Findings marked 🟡 are optional suggestions and need no follow-up push.

Comment thread components/customers/CustomerStoryHeader.tsx
Comment thread components/blog/BlogHeader.tsx
Customer stories already store customerLogoDark in frontmatter, but only
Ramp passed it into BlogHeader. Look the dark asset up from the story
index so Canva, Cresta, and the rest keep branded marks in dark mode.
Also keep forwarding an optional hero image on customer stories.

Co-authored-by: marc.klingen <marc.klingen@clickhouse.com>
Co-authored-by: marc.klingen <marc.klingen@clickhouse.com>

This branch was successfully deployed

1 active deployment
Preview — f62a40b9 Deployed Sep 21, 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