Skip to content

Move legal links to Settings#117

Merged
bradleymackey merged 1 commit into
mainfrom
codex/track-35-move-legal-links-settings
Jul 19, 2026
Merged

Move legal links to Settings#117
bradleymackey merged 1 commit into
mainfrom
codex/track-35-move-legal-links-settings

Conversation

@bradleymackey

Copy link
Copy Markdown
Collaborator

What

  • remove Terms, Privacy, and Security from the application sidebar
  • add a compact Legal card to the authenticated Settings page using the shared legal-links component
  • remove the orphaned collapsed-sidebar CSS
  • update component documentation and signed-in, logged-out, Settings, and public legal-page coverage

Why

The legal links occupied persistent navigation space even though they are account-reference material. Settings is a clearer home for authenticated users, while sign-in, sign-up, and legal pages retain their existing links.

Validation

  • make test
  • npm run assets:check
  • focused Settings, sidebar, and public legal-page integration tests
  • browser QA confirming one Legal section on Settings, no sidebar legal content, and no horizontal overflow

Resolves TRACK-35.

@bradleymackey
bradleymackey marked this pull request as ready for review July 19, 2026 12:29
@bradleymackey
bradleymackey merged commit af15ef7 into main Jul 19, 2026
2 checks passed
@bradleymackey
bradleymackey deleted the codex/track-35-move-legal-links-settings branch July 19, 2026 12:29
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