You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
docs: align heartKIT landing page with KIT sites - #55
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.
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.
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
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.
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
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.
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.24release. Clean installs reproduce the regular product prefix, bold KIT suffix and responsive HELIA link.Closes #54.