Skip to content

docs: adopt official Ambiq footer artwork - #53

Merged
apage224 merged 1 commit into
mainfrom
codex/official-ambiq-logo
Oct 7, 2026
Merged

apage224 merged 1 commit into
mainfrom
codex/official-ambiq-logo

Conversation

@apage224

@apage224 apage224 commented Oct 7, 2026

Copy link
Copy Markdown
Collaborator

Problem

heartKIT still uses an older helia-ui footer, so it does not show the approved Ambiq blue logo in light mode and official white logo in dark mode.

Approach

Pin the published helia-ui v0.1.0-alpha.23 tag and regenerate the npm lockfile. The shared footer defaults to blue in light mode, supports an optional black light-mode choice, and uses white in dark mode.

Validation

npm ci --ignore-scripts, npm run check, npm run build, npm run check:output, and 8 Playwright tests passed. The rendered footer was checked in both themes: blue #0047BA and white artwork loaded successfully at matching dimensions.

Risks and follow-up

The site will use the new shared UI release. Deployment occurs through the site's usual workflow after merge. Rollout tracking: AmbiqAI/helia-ui#189.

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.

Copilot review overview

🟢 Approval recommended

The release tag and locked commit match, the footer configuration remains compatible, and no blocking issues were identified.

Review effort: Balanced
Findings: None

What changed in this PR

Updates the documentation site’s shared UI dependency to adopt official Ambiq footer artwork.

Changes:

  • Pins helia-ui to v0.1.0-alpha.23, providing blue light-mode and white dark-mode logos.
  • Updates the lockfile to the matching release commit.
File Description
astro-site/​package.json Selects the new helia-ui release tag.
astro-site/​package-lock.json Locks the release to its resolved commit.
Files not reviewed (1)
  • astro-site/package-lock.json: Generated file

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

@apage224
apage224 merged commit a9083e3 into main Oct 7, 2026
5 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.

2 participants