Move the composer below the carousel and the CTA into the card (staging) - #316
Merged
Merged
Conversation
Follow-up review notes: The composer now closes the page. The order reads headline, carousel, then the box a visitor would type into once they are inside. "Get started free" moves out of the page and into the carousel card, as an outlined button under the copy. The copy block has a fixed height, so the button lands on the same line on every slide instead of drifting with the length of the headline. The computers and terminal slides go back to the original full-size graphics rather than the compact stand-ins, filling the whole first half of the card. A new "cover" fit scales those to the pane width, pins them to the top so the window chrome stays whole, and fades the last stretch out so the overrun reads as a surface continuing past the card rather than a hard slice. The signed-out sidebar drops its empty "Recents" placeholder. Em dashes are gone from the landing copy and its comments. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
This was referenced Aug 23, 2026
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
1 participant
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.
Third round of review notes on the landing page, following #313 / #314.
What changed
The composer closes the page. Order now reads headline, carousel, then the box a visitor would type into once they are inside.
"Get started free" moved into the card, as an outlined
Button variant="outline"under the copy. The copy block has a fixed height, so the button lands on the same line on every slide instead of drifting with the length of the headline.Computers and Developers use the original full-size graphics again, filling the whole first half of the card with the same padding as every other slide. A new
coverfit:The compact stand-ins added in #313 (
computer-mini-visual,terminal-visual) are deleted, since the originals are back.The signed-out sidebar drops its empty "Recents" placeholder.
Em dashes removed from the landing copy and its comments.
Verified
tsc --noEmitclean, production build clean. Every slide and all three viewports captured and reviewed from the production build, with no scrolling in either axis:One bug fixed while building the
coverfit: a flex container sizes its children to their content, soComputerVisualcollapsed to its intrinsic width instead of filling the pane. Covering visuals stay block-level.🤖 Generated with Claude Code