Open
Conversation
…criptions, add heading tests Co-authored-by: benbalter <282759+benbalter@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Audit SEO of pages
Audit and fix SEO issues across site pages
Feb 17, 2026
benbalter
approved these changes
Feb 17, 2026
Contributor
There was a problem hiding this comment.
Pull request overview
Addresses site-wide SEO audit findings by fixing heading hierarchy (ensuring a single H1 and no skipped heading levels) and refreshing/shortening several meta descriptions. This aligns static pages with the existing PageLayout contract (title rendered as H1) and adds automated checks to prevent regressions.
Changes:
- Fixed heading hierarchy: added an H1 to the homepage and updated section headings on several pages from H3 → H2.
- Updated/shortened meta descriptions (including resume job title update).
- Added Playwright E2E tests to validate single-H1 pages and no skipped heading levels.
Reviewed changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| src/pages/talks.astro | Changes section headings to H2 and updates local styles accordingly to avoid H1→H3 skips. |
| src/pages/other-recommended-reading.astro | Shortens meta description and changes category headings to H2 to preserve hierarchy. |
| src/pages/index.astro | Adds a visually hidden H1 to ensure the homepage has exactly one H1. |
| src/pages/fine-print.astro | Updates the “Licensing” heading and related styles from H3 → H2. |
| src/pages/contact.astro | Expands the page description to be more informative for SERPs. |
| src/content/pages/resume.md | Updates resume meta description to reflect current role/title. |
| e2e/seo-astro.spec.ts | Adds “Heading Hierarchy” E2E suite to enforce single H1 and no skipped heading levels. |
| { url: '/about/', name: 'About' }, | ||
| { url: '/resume/', name: 'Resume' }, | ||
| { url: '/contact/', name: 'Contact' }, | ||
| { url: '/talks/', name: 'Talks' }, |
There was a problem hiding this comment.
The new Heading Hierarchy suite doesn’t include the pages that were updated in this PR for heading fixes (e.g., /fine-print/ and /other-recommended-reading/). Adding them to the pages list would prevent regressions on the exact pages this PR is addressing.
Suggested change
| { url: '/talks/', name: 'Talks' }, | |
| { url: '/talks/', name: 'Talks' }, | |
| { url: '/fine-print/', name: 'Fine Print' }, | |
| { url: '/other-recommended-reading/', name: 'Other Recommended Reading' }, |
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Comprehensive SEO audit of all pages revealed missing H1, broken heading hierarchy, and stale/malformed meta descriptions.
Heading structure
<h1 class="sr-only">Ben Balter</h1>) — page had no H1<h3>→<h2>for section headings — all three skipped from H1 (provided byPageLayout) directly to H3Meta descriptions
Tests
Heading Hierarchye2e test suite validating single H1 and no skipped heading levels across key pagesOriginal prompt
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.