Skip to content

fix: implement landing page matching mockup design#1

Merged
kristinaquinones merged 3 commits into2025-12-23from
claude/fix-mockup-implementation-IF3Pt
Dec 25, 2025
Merged

fix: implement landing page matching mockup design#1
kristinaquinones merged 3 commits into2025-12-23from
claude/fix-mockup-implementation-IF3Pt

Conversation

@kristinaquinones
Copy link
Contributor

@kristinaquinones kristinaquinones commented Dec 25, 2025

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.


Note

Introduces a full landing page per mockup with dedicated styling and removes global chrome to give the page full control.

  • New src/app/landing.css with theme variables and components for header, hero, stats, terminal, features grid, comparison table, and footer
  • Rewrites src/app/page.tsx to static markup using the new styles (hero + in-column stats, terminal demo, features, comparison, footer)
  • Simplifies src/app/layout.tsx: removes shared header/footer, keeps font links, and updates metadata title/description
  • Updates .eslintrc.json: extends next/typescript and adds ignore patterns

Written by Cursor Bugbot for commit b2fdb60. This will update automatically on new commits. Configure here.

claude and others added 2 commits December 25, 2025 16:06
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>
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

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>
Copy link

Choose a reason for hiding this comment

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

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)

Fix in Cursor Fix in Web

Beautiful, fast, and maintainable. Deploy in 30 seconds,
customize when you need to.
</p>
<a href="#" className="btn btn-primary">npx emberdocs init →</a>
Copy link

Choose a reason for hiding this comment

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

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)

Fix in Cursor Fix in Web

rel="stylesheet"
/>
</head>
<body className="flex flex-col min-h-screen bg-[var(--bg)] text-[var(--text)]">
Copy link

Choose a reason for hiding this comment

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

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.

Fix in Cursor Fix in Web

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.
@kristinaquinones kristinaquinones merged commit 06dd420 into 2025-12-23 Dec 25, 2025
1 of 2 checks passed
@kristinaquinones kristinaquinones deleted the claude/fix-mockup-implementation-IF3Pt branch December 25, 2025 16:59
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.

2 participants