fix(website): restore Get started section and tighten hero terminal height - #312
Conversation
…eight The hero Get started button linked to #get-started but the section was not rendered on the home page, so the link did nothing. Re-add the GetStarted component to the en/ca/es home pages. Also reduce the hero terminal min-height from 27em to 21em so short scenes no longer leave a large empty dark area below the content.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (4)
WalkthroughThis PR integrates the GetStarted component into the English, Catalan, and Spanish homepage variants. It also reduces the hero terminal's CSS minimum height from 27em to 21em for tighter spacing. ChangesGetStarted and Hero Updates
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request reduces the minimum height of the .hero-term-screen in the Hero.astro component from 27em to 21em. It also imports and adds the GetStarted component to the Catalan, Spanish, and default landing pages. I have no feedback to provide as there are no review comments to address.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
There was a problem hiding this comment.
Pull request overview
Restores the missing “Get started” section on localized home pages so the hero CTA (#get-started) scroll target exists again, and reduces the hero terminal minimum height to avoid excessive empty space under shorter scenes.
Changes:
- Re-added the existing
GetStartedsection to theen,ca, andeshome pages so#get-startedanchors resolve correctly. - Reduced
.hero-term-screenmin-heightfrom27emto21emto tighten the hero terminal’s vertical footprint.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| src/website/src/pages/index.astro | Renders GetStarted on the default (en) home page to restore the #get-started anchor target. |
| src/website/src/pages/ca/index.astro | Renders GetStarted on the Catalan home page to restore the #get-started anchor target. |
| src/website/src/pages/es/index.astro | Renders GetStarted on the Spanish home page to restore the #get-started anchor target. |
| src/website/src/components/Hero.astro | Lowers the hero terminal screen minimum height to reduce empty space for short scenes. |
Summary
Fixes two issues reported on the live site after PR #311 was merged: the
hero "Get started" button did nothing, and the hero terminal showed a large
empty dark area below short scenes.
Changes
GetStartedcomponent to the en/ca/es home pages.The hero button links to
#get-started, but the section was not renderedon the home page, so the anchor pointed nowhere. The component already
existed and is fully translated; it was just disconnected from the page.
min-heightfrom27emto21emso short scenes (CLI, Node.js, Python, .NET) no longer leave a big empty
void below the content, which was especially visible on the light theme.
Testing
pnpm lintpassesconfirmed it scrolls to the Get started section; checked the reduced
terminal void across scenes and widths in the browser)
Related
N/A
Summary by CodeRabbit
New Features
Style