fix: landing page hero font size + stats vertical alignment#21
Merged
Conversation
- Reduce hero headline clamp from 80px max to 64px so buttons are visible on a 14-inch MacBook without scrolling - Stats strip: add flexbox centering to each cell and alignItems stretch on the grid so labels align consistently across all three columns - Install @google/genai (prep for P2.4 AI tagging) Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Note
|
| Cohort / File(s) | Summary |
|---|---|
Dependency Update client/package.json |
Added @google/genai at version ^1.46.0 to project dependencies. |
Landing Page Layout & Typography client/src/app/page.tsx |
Updated hero headline responsive font sizing (reduced clamp range), refactored stats strip grid to use flexbox layout with vertical centering, and adjusted stat value typography (increased bottom margin and line-height) with centered text alignment for stat labels. |
Estimated code review effort
🎯 2 (Simple) | ⏱️ ~10 minutes
🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
| Check name | Status | Explanation | Resolution |
|---|---|---|---|
| Docstring Coverage | Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. | Write docstrings for the functions missing them to satisfy the coverage threshold. |
✅ Passed checks (2 passed)
| Check name | Status | Explanation |
|---|---|---|
| Title check | ✅ Passed | The title accurately summarizes the two main UI fixes in the changeset: reducing hero font size and improving stats vertical alignment. |
| Description check | ✅ Passed | The description covers the main changes and includes completed checklist items, but only partially addresses the template structure with minimal context before the template section. |
✏️ Tip: You can configure your own custom pre-merge checks in the settings.
✨ Finishing Touches
📝 Generate docstrings
- Create stacked PR
- Commit on current branch
🧪 Generate unit tests (beta)
- Create PR with unit tests
- Commit unit tests in branch
fix-landing-page-ui
Comment @coderabbitai help to get the list of available commands and usage tips.
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
🧹 Nitpick comments (1)
client/package.json (1)
12-12: Keep guardrail in mind for upcoming feature implementation.The dependency is currently unused (
@google/genaihas no imports). When the AI tagging feature is implemented, ensure imports only occur in server code, not in "use client" components like those inclient/src/components/andclient/src/app/auth/.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@client/package.json` at line 12, The package currently lists the unused dependency "@google/genai"; remove it until the AI tagging feature is implemented, and when you add that feature ensure any imports of "@google/genai" occur only in server-side code (e.g., API routes or server-only modules) and never inside "use client" React components under client/src/components/ or client/src/app/auth/; if server-only code is needed, create a dedicated server module (e.g., an API route or a file without the "use client" directive) to wrap calls to "@google/genai" and import that server module from client code via secure API endpoints instead of direct imports.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In `@client/package.json`:
- Line 12: The package currently lists the unused dependency "@google/genai";
remove it until the AI tagging feature is implemented, and when you add that
feature ensure any imports of "@google/genai" occur only in server-side code
(e.g., API routes or server-only modules) and never inside "use client" React
components under client/src/components/ or client/src/app/auth/; if server-only
code is needed, create a dedicated server module (e.g., an API route or a file
without the "use client" directive) to wrap calls to "@google/genai" and import
that server module from client code via secure API endpoints instead of direct
imports.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 9c61d54a-c81c-4d4f-a53e-23069c6644bd
⛔ Files ignored due to path filters (1)
client/package-lock.jsonis excluded by!**/package-lock.json
📒 Files selected for processing (2)
client/package.jsonclient/src/app/page.tsx
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.
What does this PR do?
Type of change
Checklist
npm run build)npx tsc --noEmit)npm run lint)Summary by CodeRabbit
New Features
Style