Skip to content

Fix collapsed sidebar toggle#122

Merged
bradleymackey merged 1 commit into
mainfrom
codex/track-40-sidebar-toggle
Jul 19, 2026
Merged

Fix collapsed sidebar toggle#122
bradleymackey merged 1 commit into
mainfrom
codex/track-40-sidebar-toggle

Conversation

@bradleymackey

Copy link
Copy Markdown
Collaborator

Summary

  • replace the collapsed sidebar's hidden-checkbox label with a real accessible button
  • keep collapse state in one data-sidebar-collapsed source and persist it to local storage
  • update the button label, expanded state, and Lucide icon for both transitions
  • cover signed-in and logged-out sidebar rendering plus the generated asset contract

Testing

  • go test -count=1 -run 'TestUIShellRendersResponsiveAccessibleSidebar|TestUIShellRemovesCollapsedLabelsFromLoggedOutSidebar|TestUIShellProvidesAccessibleIconTooltips' ./internal/server
  • DATABASE_URL='postgres://track:track@localhost:5436/track?sslmode=disable' go test -count=1 -coverprofile=/tmp/track40-server-cover.out ./internal/server
  • node --check internal/server/static/app.js

TRACK-40

@bradleymackey
bradleymackey merged commit 5c6e68d into main Jul 19, 2026
2 checks passed
@bradleymackey
bradleymackey deleted the codex/track-40-sidebar-toggle branch July 19, 2026 20: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