Skip to content

fix: show n/a in empty /ai matrix cells - #118

Merged
romahawk merged 1 commit into
mainfrom
fix/ai-matrix-na
Oct 14, 2026
Merged

romahawk merged 1 commit into
mainfrom
fix/ai-matrix-na

Conversation

@romahawk

Copy link
Copy Markdown
Owner

What and why

Empty cells in the /ai project matrix show "n/a" instead of "—". It removes the last em dash from the page, in line with the content audit (#115).

No linked issue.

Changes

Copy: /ai (src/components/KnowledgeBasePage.jsx)

  • The empty-cell placeholder is now "n/a". The aria-label="Not applicable" and the muted style (.kb-matrix__empty) stay the same

Verification

  • npm run lint exits 0
  • npm run build exits 0 (this runs scripts/check-consistency.mjs first)
  • CI green on this PR (CI + Lighthouse CI)
  • Vercel preview opened and checked, not just localhost

How to test

  1. Open the Vercel preview from the checks below
  2. /ai: scroll to the project matrix. Empty cells show a muted "n/a"
  3. Search the page for "—": no results

Core loop: home → Work → a proof link → Contact (email copy, CV download) still works.

Not tested: nothing was opened in a browser, and the matrix has not been checked on mobile.

Quality gates

  • Mobile checked (iOS Safari or Android Chrome, or a 375px viewport)
  • No console errors in the preview
  • Accessibility kept: landmarks, aria-label on icon-only controls, focus rings, prefers-reduced-motion
  • No new npm dependency, or the reason is given in Notes
  • Lucide icons use named imports and are registered in the component's ICONS map (no icon changes)
  • Redirects in vercel.json checked if a route was added, moved or removed (no route changes)

Docs

  • CHANGELOG.md updated under [Unreleased]
  • docs/ / CLAUDE.md updated if architecture, routing or scope changed (copy only)

Screenshots

None taken. One-word text change.

Notes for review

None.

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@vercel

vercel Bot commented Oct 14, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
portfolio-fullstack Ready Ready Preview Oct 14, 2026 11:30am UTC
portfolio-react Ready Ready Preview Oct 14, 2026 11:30am UTC

@romahawk
romahawk merged commit 04ed945 into main Oct 14, 2026
6 checks passed
@romahawk
romahawk deleted the fix/ai-matrix-na branch October 14, 2026 15:14

This branch was successfully deployed

2 active deployments
Preview – portfolio-fullstack — 72132f17 Deployed Oct 14, 2026 by vercel[bot]
Preview – portfolio-react — 72132f17 Deployed Oct 14, 2026 by vercel[bot]
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.

1 participant