Skip to content

docs: align compressionKIT landing page with KIT sites - #80

Merged
apage224 merged 6 commits into
mainfrom
issue-79-kit-home-consistency
Oct 7, 2026
Merged

apage224 merged 6 commits into
mainfrom
issue-79-kit-home-consistency

Conversation

@apage224

@apage224 apage224 commented Oct 7, 2026 •

Copy link
Copy Markdown
Contributor

The home page had uneven card interaction and less direct first-step guidance. It now follows the KIT page pattern with a branded hero, icon-matched yellow-to-green headline, shared quick links, concise overview, whole-card feature and navigation links, and a source-checkout installation path.

Validation: Astro check and build, 12 content tests, 1,861-link and 404-layout checks, plus light/dark viewport review and card click checks.

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 #79.

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

The pinned helia-ui release does not support the newly configured header option.

1 open finding
What changed in this PR

Aligns the compressionKIT landing page with other KIT documentation sites.

Changes:

  • Refreshes hero branding, colors, and quick links.
  • Adds whole-card navigation and source installation guidance.
  • Configures regular-weight header prefixing.
File Description
HANDOFF.md Records rollout status and dependency follow-up.
astro-site/​src/​styles/​site.css Updates accents and card styling.
astro-site/​src/​content/​docs/​index.mdx Restructures landing-page content and cards.
astro-site/​src/​components/​HomeHero.astro Refreshes hero artwork, gradient, and buttons.
astro-site/​astro.config.mjs Requests regular-weight header prefixing.

🧠 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/astro.config.mjs

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

The shared-header dependency remains unpinned and hero CSS overrides the new buttons with inaccessible light-theme colors.

2 open findings

🧠 Review effort: Balanced

Comment thread astro-site/src/components/HomeHero.astro Outdated

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 shared header dependency must be released and immutably pinned before the new configuration and validation are reproducible.

1 open finding
1 resolved since last review

🧠 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 helia-ui dependency must be pinned to a release supporting titleRegularPrefix.

1 open finding

🧠 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

The changes satisfy the linked issue while preserving legacy anchors, routes, responsive behavior, and interaction coverage.

0 open findings

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

🧠 Review effort: Balanced

@apage224
apage224 merged commit c19d8af into main Oct 7, 2026
3 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 compressionKIT landing page with the KIT site pattern

2 participants