Repository navigation
feat(hero): add live status link under the terminal and loosen cta spacing - #255
Merged
Merged
Conversation
…acing Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Deploying website with
|
| Latest commit: |
7c49f1c
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://7b8956c0.website-70y.pages.dev |
| Branch Preview URL: | https://claude-hero-section-layout-8.website-70y.pages.dev |
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What changed
Breathing room. The gap above the hero CTA row is larger: 48 → 64px on desktop and 32 → 40px on mobile. The gap between the CTA row and the stat row is 56px on desktop (was 48) and 40px on mobile (was 32).
Status link. A new "See also / Status website →" link sits under the hero terminal and points to
https://stats.decdn.org/. It's a plain link rendered by theHeroserver component: a pulsing green dot, then the text and arrow.--whisper(about 6.6:1 on white). Raw--whisperis about 3.5:1 and fails AA at body size. The dot and arrow keep the full accent.Layout. The hero's inner grid now has two rows (lead + CTA | terminal, then stats | status link).
self-baselineon the second-row items lines up the "SEE ALSO" label with the stat labels. On mobile, DOM order stacks the terminal under the stats and the link last.Files
lib/links.ts: newlinks.statslib/copy.ts: newHERO_STATUSlabel/value copyapp/globals.css:.hero-status,.status-livecomponents/site/Hero.tsx: the grid restructure, the spacing and the inline status linkReviewer notes
HeroStatusclient component whose dot pulsed only when the live stats were current. Both are removed, so the dot now pulses unconditionally and the link adds no client JS.llms-full.txtmirror, because hero links aren't mirrored today.Verification
pnpm typecheck,pnpm lint,pnpm test(510 passing) andpnpm format:checkpass on the final commit.pnpm build(includingcheck:outandcheck:og-image) passed on the first commit and was not rerun after that.getBoundingClientRect) were done after the spacing nudge and before the link was inlined. The "SEE ALSO" and "TARGET PRICE" label bottoms matched at 1440px and 1024px, and the gap measurements above come from that check. The 375px mobile stacking and overflow check dates from the first commit.🤖 Generated with Claude Code