Skip to content

fix: improve blog heading semantics#40

Merged
corwinm merged 1 commit into
mainfrom
fix/blog-heading-semantics
Jul 13, 2026
Merged

fix: improve blog heading semantics#40
corwinm merged 1 commit into
mainfrom
fix/blog-heading-semantics

Conversation

@corwinm

@corwinm corwinm commented Jul 13, 2026

Copy link
Copy Markdown
Owner

Summary

  • use each blog article title as its single h1
  • promote article section headings from h3 to h2 for a correct heading hierarchy
  • preserve the existing 24px section-heading presentation
  • rename the article return link from Back to overview to Back to Blog
  • add regression coverage that discovers every blog.*.mdx article and enforces the heading contract

Test plan

  • TDD RED: all three article-heading cases and the return-label case initially failed
  • TDD GREEN: focused semantics tests pass 4/4
  • pnpm lint — 0 warnings/errors
  • pnpm typecheck
  • pnpm test — 40/40 passing
  • pnpm build
  • git diff --check
  • production SSR smoke test: all three article routes contain one h1, h2 sections, and Back to Blog
  • browser accessibility-tree verification on /blog/arashi: one level-1 title followed by level-2 sections
  • browser computed-style verification: section headings remain 24px
  • independent review caught an initial h1h3 hierarchy skip; fixed by promoting sections to h2, then re-reviewed successfully with no security or logic concerns

@vercel

vercel Bot commented Jul 13, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
corwinm-remix Ready Ready Preview, Comment Jul 13, 2026 8:55pm

@corwinm
corwinm merged commit 95a1194 into main Jul 13, 2026
3 checks passed
@corwinm
corwinm deleted the fix/blog-heading-semantics branch July 13, 2026 21:04
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