Skip to content

docs: align heartKIT landing page with KIT sites - #55

Merged
apage224 merged 5 commits into
mainfrom
codex/kit-home-consistency
Oct 7, 2026
Merged

apage224 merged 5 commits into
mainfrom
codex/kit-home-consistency

Conversation

@apage224

@apage224 apage224 commented Oct 7, 2026 •

Copy link
Copy Markdown
Collaborator

The home page buried the first steps among detailed dataset, mode and model lists. It now follows the KIT page pattern: branded hero, quick links, short overview, task cards with small icons, installation and whole-card routes into the docs. Detailed material stays on its dedicated pages, and prior home anchors remain available.

Validation: Astro check and build, 11 Playwright tests, output and 190 historical route checks, plus light/dark viewport review.

Shared UI: pinned to the immutable v0.1.0-alpha.24 release. Clean installs reproduce the regular product prefix, bold KIT suffix and responsive HELIA link.

Closes #54.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟡 Changes recommended

Historical fragment links and light-theme button contrast regress, and mobile overflow coverage no longer checks the landing page.

3 open findings
What changed in this PR

Aligns heartKIT’s landing page with the KIT site pattern, emphasizing orientation and routes into detailed documentation.

Changes:

  • Adds product artwork and shared quick-link buttons.
  • Replaces detailed lists with task cards, installation steps, and documentation cards.
  • Updates mobile navigation coverage and records the pending shared-header release dependency.
File Description
HANDOFF.md Records progress, validation, and release dependency.
astro-site/​tests/​site.spec.ts Tests mobile navigation to Quickstart installation.
astro-site/​src/​styles/​site.css Updates card styling and responsive layout.
astro-site/​src/​content/​docs/​index.mdx Simplifies introductory content and documentation navigation.
astro-site/​src/​components/​HomeHero.astro Adds product artwork and shared buttons.
astro-site/​astro.config.mjs Configures the regular-weight header prefix.

🧠 Review effort: Balanced


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread astro-site/src/components/HomeHero.astro
Comment thread astro-site/src/content/docs/index.mdx Outdated
Comment thread astro-site/tests/site.spec.ts

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🔵 Needs a closer look

The header option needs a published shared-library release, updated dependency pins, and clean-install validation.

0 open findings

3 resolved since last review
Previously missed (1)

In code that hasn't changed since last review

Medium severity Pin released shared-header dependency for reproducible installs

astro-site/​astro.config.mjs:62

This option depends on the unreleased shared-header change in helia-ui#193. As HANDOFF.md notes, the preview uses an untracked dependency patch while the committed pin remains unchanged, so a clean install does not reproduce the regular heart prefix. Before merging, pin an immutable release containing that change in package.json and package-lock.json, then rerun validation from a clean install without the preview patch.

🧠 Review effort: Balanced

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🔵 Needs a closer look

The required shared-header release remains unpinned, so clean-install validation of the intended result is still pending.

0 open findings

Previously missed (1)

In code that hasn't changed since last review

Medium severity Pinned helia-ui version lacks header.titleRegularPrefix support

astro-site/​astro.config.mjs:62

The committed helia-ui pin is still v0.1.0-alpha.23, which does not support header.titleRegularPrefix. A clean installation therefore cannot reproduce the preview's regular-weight heart prefix. Keep the documented draft gate until an immutable release containing helia-ui#193 is pinned, the lockfile is regenerated, and validation is rerun without the local package patch.

🧠 Review effort: Balanced

@apage224
apage224 marked this pull request as ready for review October 7, 2026 16:02
@apage224
apage224 requested a balanced review from Copilot October 7, 2026 16:02

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟢 Approval recommended

No blocking issues remain in the scoped landing-page changes; validation was reported but not rerun.

0 open findings

Files not reviewed (1)
  • astro-site/package-lock.json: Generated file

🧠 Review effort: Balanced

@apage224
apage224 merged commit 276f426 into main Oct 7, 2026
6 checks passed
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.

Align the heartKIT landing page with the KIT site pattern

2 participants