Skip to content

Unify privacy policy and terms of service layout with main site#101

Open
Divide-By-0 wants to merge 1 commit into
mainfrom
fix/unify-legal-page-layout
Open

Unify privacy policy and terms of service layout with main site#101
Divide-By-0 wants to merge 1 commit into
mainfrom
fix/unify-legal-page-layout

Conversation

@Divide-By-0

Copy link
Copy Markdown

Summary

  • Both legal pages (privacy policy & terms of service) had hardcoded, inconsistent headers (different nav links from each other and from the main site) and were missing the site footer entirely
  • Created a shared PageLayout component that wraps standalone pages with the same header and footer used across the rest of the site
  • Updated both pages to use PageLayout, giving them consistent navigation and the full site footer

Test plan

  • Verify /privacy-policy renders with the site header and footer
  • Verify /terms-of-service renders with the site header and footer
  • Verify the "Coursetexts" breadcrumb in the header links back to /
  • Verify the footer matches what appears on other pages (donation link, course request form, social icons, legal links)

🤖 Generated with Claude Code

Both legal pages had hardcoded, inconsistent headers and were missing the
site footer. Extracted a shared PageLayout component and updated both
pages to use it so they render the same header/footer as the rest of the
site.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@akshithio

Copy link
Copy Markdown
Collaborator

ignore the self-requested review, has to do with the new branch protection rules, might lax them a bit moving forward. you can merge whenever

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.

3 participants