fix: implement landing page matching mockup design#1
fix: implement landing page matching mockup design#1kristinaquinones merged 3 commits into2025-12-23from
Conversation
Replaced MVP scaffold page with landing page design from mockup-02-developer-dark.html. Key changes: - Stats section now properly positioned inside hero-content (left column) - Stats styled with minimal left border design instead of card boxes - Hero maintains two-column grid (content | terminal) - Removed default header/footer from layout to allow landing page full control - Added landing.css with all mockup styles - Terminal displays full installation sequence with proper color coding This matches the mockup specification exactly with stats integrated into the hero grid instead of appearing as separate card elements below.
Signed-off-by: Kristina <cheskaquinones@gmail.com>
There was a problem hiding this comment.
This is the final PR Bugbot will review for you during this billing cycle
Your free Bugbot reviews will reset on January 24
Details
You are on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle.
To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.
| <ul className="nav-links"> | ||
| <li><a href="#features">Features</a></li> | ||
| <li><a href="#docs">Docs</a></li> | ||
| <li><a href="#github">GitHub</a></li> |
There was a problem hiding this comment.
Navigation anchor links point to non-existent IDs
The navigation links use anchor references (#features, #docs, #github) but the target elements don't have matching id attributes. The features section only has className="features" without an id, and there are no elements with id="docs" or id="github" on the page at all. Clicking these navigation links won't scroll to any section. The old Header component linked Docs to EMBERDOCS_BASE_ROUTE and GitHub to the actual GitHub URL.
Additional Locations (1)
| Beautiful, fast, and maintainable. Deploy in 30 seconds, | ||
| customize when you need to. | ||
| </p> | ||
| <a href="#" className="btn btn-primary">npx emberdocs init →</a> |
There was a problem hiding this comment.
CTA buttons use placeholder hrefs instead of routes
The primary call-to-action buttons use href="#" placeholder values instead of actual navigation links. The "Get Started" button and the "npx emberdocs init" button previously linked to ${EMBERDOCS_BASE_ROUTE}/getting-started/introduction. Now clicking these buttons does nothing useful, breaking the main user conversion flow on the landing page.
Additional Locations (1)
| rel="stylesheet" | ||
| /> | ||
| </head> | ||
| <body className="flex flex-col min-h-screen bg-[var(--bg)] text-[var(--text)]"> |
There was a problem hiding this comment.
Layout changes remove Header and SearchPalette from docs pages
The root layout now renders only {children} without the Header and SearchPalette components. While the landing page has its own header, the docs pages at src/app/docs/[...slug]/page.tsx do not include their own site header. This means docs pages lose the main navigation header (logo, nav links, theme toggle, version switcher) and the keyboard search functionality (Cmd+K). Users visiting documentation pages cannot navigate back to the home page or use search.
Added TypeScript preset and ignore patterns to ensure Next.js 16 lint command works correctly with ESLint 9. This resolves CI lint errors about invalid project directory.
Replaced MVP scaffold page with landing page design from mockup-02-developer-dark.html.
Key changes:
This matches the mockup specification exactly with stats integrated into the hero grid instead of appearing as separate card elements below.
Note
Introduces a full landing page per mockup with dedicated styling and removes global chrome to give the page full control.
src/app/landing.csswith theme variables and components for header, hero, stats, terminal, features grid, comparison table, and footersrc/app/page.tsxto static markup using the new styles (hero + in-column stats, terminal demo, features, comparison, footer)src/app/layout.tsx: removes shared header/footer, keeps font links, and updatesmetadatatitle/description.eslintrc.json: extendsnext/typescriptand adds ignore patternsWritten by Cursor Bugbot for commit b2fdb60. This will update automatically on new commits. Configure here.