fix: Role-appropriate bottom nav on all authenticated pages #23

New issue

Closed

opened 2026-03-16 22:50:57 +00:00 by forgejo_admin · 0 comments

forgejo_admin commented

2026-03-16 22:50:57 +00:00

Owner

Lineage

plan-wkq → Phase 10 (Playground)

Repo

forgejo_admin/westside-playground

User Story

As any logged-in user, I want navigation that matches my role — not links to pages I can't use.

Context

Multiple pages have "My Player" in the bottom nav for roles that aren't players. Each role needs its own nav:

File Targets

admin.html — fix bottom nav to: Dashboard (active) | Teams | Sign Out
admin-teams.html — fix bottom nav to: Dashboard | Teams (active) | Sign Out
parent.html — keep as-is (My Player is correct for player role), but add "My Team" link to team.html
player-profile.html — fix bottom nav: My Player | My Team | Sign Out
billing.html — fix bottom nav: My Player | Billing (active) | Sign Out
team.html — fix bottom nav: My Player | My Team (active) | Sign Out
Do NOT touch coach.html — already fixed in PR #20

Acceptance Criteria

Admin pages have: Dashboard | Teams | Sign Out
Player pages have: My Player | My Team | Sign Out (with correct active states)
Billing page has: My Player | Billing (active) | Sign Out
No page shows "My Player" for admin or coach roles
All nav links point to correct pages
Active state on correct tab per page

Test Expectations

No automated tests (static HTML). Visual verification only.

Constraints

Only modify bottom <nav class="bottom-nav"> sections
Do NOT change page content above the nav
Do NOT touch coach.html or index.html or tryouts.html or register.html or signin.html

Checklist

PR opened
No unrelated changes

project-westside-basketball

### Lineage `plan-wkq` → Phase 10 (Playground) ### Repo `forgejo_admin/westside-playground` ### User Story As any logged-in user, I want navigation that matches my role — not links to pages I can't use. ### Context Multiple pages have "My Player" in the bottom nav for roles that aren't players. Each role needs its own nav: **Admin pages** (`admin.html`, `admin-teams.html`): Dashboard | Teams | Sign Out **Coach page** (`coach.html`): Team | Plays | Sign Out **Player pages** (`parent.html`, `player-profile.html`, `billing.html`, `team.html`): My Player | My Team | Sign Out ### File Targets - `admin.html` — fix bottom nav to: Dashboard (active) | Teams | Sign Out - `admin-teams.html` — fix bottom nav to: Dashboard | Teams (active) | Sign Out - `parent.html` — keep as-is (My Player is correct for player role), but add "My Team" link to `team.html` - `player-profile.html` — fix bottom nav: My Player | My Team | Sign Out - `billing.html` — fix bottom nav: My Player | Billing (active) | Sign Out - `team.html` — fix bottom nav: My Player | My Team (active) | Sign Out - Do NOT touch `coach.html` — already fixed in PR #20 ### Acceptance Criteria - [ ] Admin pages have: Dashboard | Teams | Sign Out - [ ] Player pages have: My Player | My Team | Sign Out (with correct active states) - [ ] Billing page has: My Player | Billing (active) | Sign Out - [ ] No page shows "My Player" for admin or coach roles - [ ] All nav links point to correct pages - [ ] Active state on correct tab per page ### Test Expectations - No automated tests (static HTML). Visual verification only. ### Constraints - Only modify bottom `<nav class="bottom-nav">` sections - Do NOT change page content above the nav - Do NOT touch coach.html or index.html or tryouts.html or register.html or signin.html ### Checklist - [ ] PR opened - [ ] No unrelated changes ### Related - `project-westside-basketball`