Skip to content

[codex] fix guide page metadata#601

Closed
Sean-Kenneth-Doherty wants to merge 5 commits into
firstcontributions:mainfrom
Sean-Kenneth-Doherty:codex/fix-guide-page-metadata
Closed

[codex] fix guide page metadata#601
Sean-Kenneth-Doherty wants to merge 5 commits into
firstcontributions:mainfrom
Sean-Kenneth-Doherty:codex/fix-guide-page-metadata

Conversation

@Sean-Kenneth-Doherty
Copy link
Copy Markdown

@Sean-Kenneth-Doherty Sean-Kenneth-Doherty commented May 16, 2026

Summary

  • Fix the guide page canonical URL markup by using a canonical link element.
  • Point guide page Open Graph, Twitter, favicon, and JSON-LD metadata at assets that are part of the current public build.
  • Use a 1200x630 PNG social image for guide Open Graph, Twitter Card, and JSON-LD image metadata.
  • Use Twitter Card name="twitter:*" meta attributes on the guide page.
  • Add an HTML5 doctype and lang="en" to the standalone guide page shell.
  • Tighten the guide <title> markup so the rendered title text stays at 64 characters without extra whitespace.
  • Deduplicate the guide metadata strings so title, description, and URL values stay consistent.

Addresses #348.
Related to #347.

Validation

  • magick public/og-image.svg public/og-image.png
  • file public/og-image.png and identify public/og-image.png confirmed a 1200x630 PNG
  • pnpm build
  • Built HTML check against dist/contribute-to-opensource/index.html confirmed the document starts with <!DOCTYPE html><html lang="en">, the canonical link is present, no meta canonical remains, no firstcontributions.github.io/assets metadata references remain, guide image metadata points to https://firstcontributions.github.io/og-image.png, referenced og-image.png/favicon.svg files exist in dist, guide Twitter Card tags render as name="twitter:*", and guide <title> / metadata titles are 64 characters.
  • git diff --check

Reference: https://developer.x.com/cards/markup

@Sean-Kenneth-Doherty Sean-Kenneth-Doherty marked this pull request as ready for review May 16, 2026 23:53
@Sean-Kenneth-Doherty
Copy link
Copy Markdown
Author

Withdrawing this to keep the contribution queue focused on the highest-signal fixes. I opened too many small slices here, and this one is less important than the concrete broken-link, security, workflow, rendering, and accessibility fixes that remain open.

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