Add Schedule link to admin bottom navigation #171

New issue

Closed

opened 2026-03-29 03:47:13 +00:00 by forgejo_admin · 1 comment

forgejo_admin commented

2026-03-29 03:47:13 +00:00

Contributor

Type

Feature

Lineage

Follow-up from forgejo_admin/westside-landing#166 (admin schedule view, merged). Reviewer noted nav link was deferred as MVP scope.

Repo

forgejo_admin/westside-landing

User Story

story:WS-S13 As an admin, I want a Schedule link in the bottom nav so that I can access the schedule view without typing the URL.

Context

The admin schedule page (/admin/schedule) is live but not linked from the bottom navigation bar. Currently, admin nav shows: Dashboard, CRM, Teams, Sign Out. Schedule should be added between Teams and Sign Out.

File Targets

Files to modify:

src/routes/(app)/+layout.svelte — add Schedule nav item to admin bottom nav links array

Pattern reference (do NOT modify):

src/routes/(app)/admin/schedule/+page.svelte — the target page (already exists)

Files NOT to touch:

Any other route files — this is nav only

Acceptance Criteria

Admin bottom nav shows: Dashboard, CRM, Teams, Schedule, Sign Out
Schedule link navigates to /admin/schedule
Active state highlights correctly when on /admin/schedule
No visual regression on other admin pages

Test Expectations

Visual verification: nav link appears and works on mobile (390px) and desktop
Run command: npm run build (no build errors)

Constraints

Follow existing nav link pattern exactly (same HTML structure, same CSS classes)
Use calendar or schedule icon if nav uses icons (match existing icon pattern)

Checklist

PR opened
No unrelated changes
Tested on mobile viewport (390px)

forgejo_admin/westside-landing#166 — admin schedule view (parent, merged)
project-westside-basketball

### Type Feature ### Lineage Follow-up from `forgejo_admin/westside-landing#166` (admin schedule view, merged). Reviewer noted nav link was deferred as MVP scope. ### Repo `forgejo_admin/westside-landing` ### User Story `story:WS-S13` As an admin, I want a Schedule link in the bottom nav so that I can access the schedule view without typing the URL. ### Context The admin schedule page (`/admin/schedule`) is live but not linked from the bottom navigation bar. Currently, admin nav shows: Dashboard, CRM, Teams, Sign Out. Schedule should be added between Teams and Sign Out. ### File Targets Files to modify: - `src/routes/(app)/+layout.svelte` — add Schedule nav item to admin bottom nav links array Pattern reference (do NOT modify): - `src/routes/(app)/admin/schedule/+page.svelte` — the target page (already exists) Files NOT to touch: - Any other route files — this is nav only ### Acceptance Criteria - [ ] Admin bottom nav shows: Dashboard, CRM, Teams, Schedule, Sign Out - [ ] Schedule link navigates to `/admin/schedule` - [ ] Active state highlights correctly when on `/admin/schedule` - [ ] No visual regression on other admin pages ### Test Expectations - [ ] Visual verification: nav link appears and works on mobile (390px) and desktop - Run command: `npm run build` (no build errors) ### Constraints - Follow existing nav link pattern exactly (same HTML structure, same CSS classes) - Use calendar or schedule icon if nav uses icons (match existing icon pattern) ### Checklist - [ ] PR opened - [ ] No unrelated changes - [ ] Tested on mobile viewport (390px) ### Related - `forgejo_admin/westside-landing#166` — admin schedule view (parent, merged) - `project-westside-basketball`

forgejo_admin commented

2026-03-29 03:49:53 +00:00

Author

Contributor

Scope Review: APPROVED

Review note: review-648-2026-03-28
All template sections present, traceability complete, file targets verified. Single-file change adding one nav link to existing admin bottom nav. No blockers, no decomposition needed.

## Scope Review: APPROVED Review note: `review-648-2026-03-28` All template sections present, traceability complete, file targets verified. Single-file change adding one nav link to existing admin bottom nav. No blockers, no decomposition needed.