Skip to content

refactor(website): substrate migration batch 4 — landing blocks off inline styles - #852

Merged
blove merged 3 commits into
mainfrom
blove/style-substrate-batch-4
Aug 29, 2026
Merged

refactor(website): substrate migration batch 4 — landing blocks off inline styles#852
blove merged 3 commits into
mainfrom
blove/style-substrate-batch-4

Conversation

@blove

@blove blove commented Aug 29, 2026

Copy link
Copy Markdown
Contributor

What

Batch 4 of the substrate migration (plan, Task 4): the 17 components/landing files move to src/styles/landing.css, including hoisting the embedded <style> tags (the third styling substrate) — their ${tokens.*} interpolations become var(--*) and the per-instance duplicated <style> elements disappear from the DOM.

Three commits: hero + feature blocks, the section strip, demo/media + the three code showcases.

Verified: three pages signature-identical to production

Aggregate per-page signatures (element count + tag-order hash + hash-of-per-element-hashes over 26 computed properties, pinned 1280×900, scroll pinned to top):

page elements result
/ 644 identical to prod
/langgraph 349 identical to prod
/ag-ui 132 identical to prod

Plus vitest at the pre-existing baseline (5 failed | 341 passed — including Differentiator.spec's content drift, unchanged), 0 lint errors, prod build green.

Judgment calls (flagged by the implementer, accepted after verification)

  • HighlightedCode overrides the shared .shiki background; as an inline style it always won, but stylesheet-vs-stylesheet it needed real specificity — now .shiki[data-ui="highlighted-code"]. The signature match on /langgraph proves the cascade lands where the inline style did.
  • DemoModal's active tab reuses the existing aria-selected attribute as its styling hook instead of a new data-*.
  • The three near-triplet code showcases keep separate class prefixes — values that differ subtly were not unified.

The homepage's known 320px overflow bugs migrate verbatim, unfixed — they belong to the polish arc.

🤖 Generated with Claude Code

@blove
blove enabled auto-merge (squash) August 29, 2026 21:40
@vercel

vercel Bot commented Aug 29, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
threadplane Ready Ready Preview Aug 29, 2026 10:04pm

Request Review

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated approval: this PR received an intelligent (AI) code review. See the review comments on this PR.

@github-actions

github-actions Bot commented Aug 29, 2026

Copy link
Copy Markdown
Contributor

Claude finished @blove's task in 0s —— View job


I'll analyze this and get back to you.

@github-actions

github-actions Bot commented Aug 29, 2026

Copy link
Copy Markdown
Contributor

Claude finished @blove's task in 0s —— View job


I'll analyze this and get back to you.

@blove
blove force-pushed the blove/style-substrate-batch-4 branch from 90eb6fd to e58d278 Compare August 29, 2026 21:53
@github-actions

github-actions Bot commented Aug 29, 2026

Copy link
Copy Markdown
Contributor

Claude finished @blove's task in 0s —— View job


I'll analyze this and get back to you.

@blove
blove merged commit 2f9faca into main Aug 29, 2026
27 checks passed
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.

1 participant