Skip to content

fix: avoid duplicate docs shell headings#602

Open
zicojiao wants to merge 1 commit into
mainfrom
codex/fix-duplicate-docs-headings
Open

fix: avoid duplicate docs shell headings#602
zicojiao wants to merge 1 commit into
mainfrom
codex/fix-duplicate-docs-headings

Conversation

@zicojiao

@zicojiao zicojiao commented Jul 1, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Render docs page content once in DocsMainColumn so SEO crawlers and heading-outline tools do not see duplicate article headings.
  • Keep a single shell-level Agora footer instead of separate mobile and desktop footer copies.
  • Change footer navigation labels such as Contact Us, Why Agora, Company, and Get Started from h2 elements to non-heading text.
  • Add docs-shell regressions for single article/footer DOM output and footer labels staying out of the page heading outline.

Verification

  • bun run test src/components/docs-shell/DocsContent.test.tsx src/components/docs-shell/DocsShell.test.tsx
  • bun run types:check
  • ./node_modules/.bin/biome check --vcs-use-ignore-file=false src/components/docs-shell/DocsMainColumn.tsx src/components/docs-shell/DocsShell.tsx src/components/docs-shell/DocsSiteFooter.tsx src/components/docs-shell/DocsContent.test.tsx src/components/docs-shell/DocsShell.test.tsx
  • git diff --check

@github-actions

github-actions Bot commented Jul 1, 2026

Copy link
Copy Markdown

Vercel Preview

Preview URL: https://docs-portal-jxdwo3dpv-agora-devx.vercel.app

@zicojiao
zicojiao marked this pull request as ready for review July 1, 2026 12:22
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