forgejo_admin/westside-playground
1
0
Fork 0
Code Issues 4 Pull requests Projects Releases Packages Wiki Activity Actions

feat: Coach page — tab navigation (Team / Plays) + remove browse teams #19

New issue
Closed
opened 2026-03-16 22:32:44 +00:00 by forgejo_admin · 1 comment
forgejo_admin commented 2026-03-16 22:32:44 +00:00
Owner
Copy link

Lineage

plan-wkq → Phase 10 (Playground)

Repo

forgejo_admin/westside-playground

User Story

As a coach, when I log in I see my team roster and my plays — nothing else.
Two tabs keep it simple: Team (roster) and Plays (TBD).

Context

The coach page currently shows a team roster grid, then a "Browse Teams" section (added in PR #8). Lucas wants a cleaner coach experience with two tabs:

  • Team tab — shows the existing roster grid (player cards with parent contact, payment badges). This is what's on the page now minus the Browse Teams section.
  • Plays tab — TBD placeholder. Show a centered message like "Plays & Playbooks — Coming Soon" with the same placeholder style used elsewhere.

Remove the "Browse Teams" section — coaches don't need to browse other teams from their landing page. If they need team detail, they can go via the roster player links.

File Targets

  • coach.html — add tab navigation at top, move roster into "Team" tab content, add "Plays" placeholder tab, remove "Browse Teams" section

Acceptance Criteria

  • Two tabs visible at top: "Team" (active by default) and "Plays"
  • "Team" tab shows the existing roster grid unchanged
  • "Plays" tab shows a TBD placeholder message
  • "Browse Teams" section removed
  • Tab switching works with minimal JS (show/hide divs)
  • Tabs use existing CSS patterns or minimal new styles matching the design system
  • No other files changed

Test Expectations

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

Constraints

  • HTML + CSS only, minimal JS for tab switching
  • Use existing CSS from shared/app.css
  • Keep the roster grid exactly as-is — just wrap it in a tab container

Checklist

  • PR opened
  • No unrelated changes

Related

  • project-westside-basketball
### Lineage `plan-wkq` → Phase 10 (Playground) ### Repo `forgejo_admin/westside-playground` ### User Story As a coach, when I log in I see my team roster and my plays — nothing else. Two tabs keep it simple: Team (roster) and Plays (TBD). ### Context The coach page currently shows a team roster grid, then a "Browse Teams" section (added in PR #8). Lucas wants a cleaner coach experience with two tabs: - **Team tab** — shows the existing roster grid (player cards with parent contact, payment badges). This is what's on the page now minus the Browse Teams section. - **Plays tab** — TBD placeholder. Show a centered message like "Plays & Playbooks — Coming Soon" with the same placeholder style used elsewhere. Remove the "Browse Teams" section — coaches don't need to browse other teams from their landing page. If they need team detail, they can go via the roster player links. ### File Targets - `coach.html` — add tab navigation at top, move roster into "Team" tab content, add "Plays" placeholder tab, remove "Browse Teams" section ### Acceptance Criteria - [ ] Two tabs visible at top: "Team" (active by default) and "Plays" - [ ] "Team" tab shows the existing roster grid unchanged - [ ] "Plays" tab shows a TBD placeholder message - [ ] "Browse Teams" section removed - [ ] Tab switching works with minimal JS (show/hide divs) - [ ] Tabs use existing CSS patterns or minimal new styles matching the design system - [ ] No other files changed ### Test Expectations - No automated tests (static HTML). Visual verification only. ### Constraints - HTML + CSS only, minimal JS for tab switching - Use existing CSS from `shared/app.css` - Keep the roster grid exactly as-is — just wrap it in a tab container ### Checklist - [ ] PR opened - [ ] No unrelated changes ### Related - `project-westside-basketball`
forgejo_admin commented 2026-03-16 22:36:44 +00:00
Author
Owner
Copy link

Additional scope (from live review):

The bottom nav on coach.html currently shows "My Player | My Team | Sign Out". Coaches aren't players — "My Player" shouldn't appear.

Update the bottom nav to match the tab structure:

  • Team (links to Team tab / stays on page)
  • Plays (links to Plays tab / stays on page)
  • Sign Out

Remove "My Player" from the coach bottom nav entirely.

**Additional scope (from live review):** The bottom nav on `coach.html` currently shows "My Player | My Team | Sign Out". Coaches aren't players — "My Player" shouldn't appear. Update the bottom nav to match the tab structure: - **Team** (links to Team tab / stays on page) - **Plays** (links to Plays tab / stays on page) - **Sign Out** Remove "My Player" from the coach bottom nav entirely.
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
main
33-fix-coach-bottom-nav-replace-my-player-w
29-feat-registration-flow-player-first-with
30-feat-rename-families-crm-to-players-crm
23-fix-role-appropriate-bottom-nav-on-all-a
25-feat-parent-dashboard-rename-to-my-playe
11-admin-schedule-page
10-billing-tiered-pricing-split-payment
12-playbook-upload-team-schedule
10-feat-billing-page-tiered-pricing-selecto
11-feat-practice-schedule-management-page-f
12-feat-playbook-upload-ux-on-coach-view-te
13-expense-tracking-dashboard
13-feat-expense-tracking-dashboard-for-admi
3-feat-add-player-profile-page-prototype
No results found.
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
forgejo_admin/westside-playground#19
No description provided.