#91 - New page: About — dedicated program overview - forgejo_admin/westside-app

forgejo_admin commented

2026-03-26 23:49:35 +00:00

Owner

Type

Feature

Lineage

Part of public site evolution. Parent note: westside-playground-overhaul

Repo

forgejo_admin/westside-playground

User Story

As a prospective parent, I want a dedicated About page with full program details so I understand who Westside is before exploring teams and schedules.

story:WS-S26

Context

The About section currently lives as 3 cards + Marcus quote on the landing page. Moving to a dedicated page lets us expand: program history, mission, values, location info. The landing page becomes just Hero → How to Join → CTA, with About one tap away in the nav.

File Targets

Files to create:

about.html — dedicated About page. Expand from the current 3 cards (Structured Training, Competitive Play, Next-Level Prep) + Marcus quote. Add program details: West Valley Utah, AAU basketball, 15U/16U/17U boys and girls, est. 2026.

Files to modify:

index.html — remove About section (3 cards + quote). Landing page becomes: Hero → How to Join → CTA.
All pages with nav — add "About" link to nav between Home and Staff.

Acceptance Criteria

about.html exists with expanded program info
About section removed from index.html
Nav updated on ALL pages: Home | About | Staff | Tryouts | Teams | Schedule | Gear | Sponsors
Mobile renders clean

Test Expectations

All nav links resolve
Visual on 390px

Constraints

Uses shared/style.css and shared/app.js
Component doc at top
Mobile-first

Checklist

Page created
Landing page trimmed
Nav updated everywhere
Lucas phone review

westside-playground-overhaul

### Type Feature ### Lineage Part of public site evolution. Parent note: `westside-playground-overhaul` ### Repo `forgejo_admin/westside-playground` ### User Story As a prospective parent, I want a dedicated About page with full program details so I understand who Westside is before exploring teams and schedules. story:WS-S26 ### Context The About section currently lives as 3 cards + Marcus quote on the landing page. Moving to a dedicated page lets us expand: program history, mission, values, location info. The landing page becomes just Hero → How to Join → CTA, with About one tap away in the nav. ### File Targets Files to create: - `about.html` — dedicated About page. Expand from the current 3 cards (Structured Training, Competitive Play, Next-Level Prep) + Marcus quote. Add program details: West Valley Utah, AAU basketball, 15U/16U/17U boys and girls, est. 2026. Files to modify: - `index.html` — remove About section (3 cards + quote). Landing page becomes: Hero → How to Join → CTA. - All pages with nav — add "About" link to nav between Home and Staff. ### Acceptance Criteria - [ ] `about.html` exists with expanded program info - [ ] About section removed from `index.html` - [ ] Nav updated on ALL pages: Home | About | Staff | Tryouts | Teams | Schedule | Gear | Sponsors - [ ] Mobile renders clean ### Test Expectations - [ ] All nav links resolve - [ ] Visual on 390px ### Constraints - Uses `shared/style.css` and `shared/app.js` - Component doc at top - Mobile-first ### Checklist - [ ] Page created - [ ] Landing page trimmed - [ ] Nav updated everywhere - [ ] Lucas phone review ### Related - `westside-playground-overhaul`

forgejo_admin commented

2026-03-27 00:03:40 +00:00

Author

Owner

Scope Review: NEEDS_REFINEMENT

Review note: review-406-2026-03-25
Three fixable issues found before this ticket can move to next_up.

Acceptance criterion 3 references non-existent pages. Nav order lists Tryouts, Teams, Gear — those are separate todo tickets (#92, #93, #94). Update nav spec to only include pages that exist post-merge: Home | About | Staff | Sponsors | Schedule.
Undocumented dependency on #90. Both tickets modify index.html. #90 restructures the landing page; this ticket removes the About section from it. #90 must merge first. Add depends:#90 label.
Repo mismatch. Issue filed on westside-app but ### Repo says westside-playground. File targets are playground HTML. Follows the #85-#88 convention but should be noted explicitly.

## Scope Review: NEEDS_REFINEMENT Review note: `review-406-2026-03-25` Three fixable issues found before this ticket can move to next_up. - **Acceptance criterion 3 references non-existent pages.** Nav order lists Tryouts, Teams, Gear — those are separate todo tickets (#92, #93, #94). Update nav spec to only include pages that exist post-merge: Home | About | Staff | Sponsors | Schedule. - **Undocumented dependency on #90.** Both tickets modify `index.html`. #90 restructures the landing page; this ticket removes the About section from it. #90 must merge first. Add `depends:#90` label. - **Repo mismatch.** Issue filed on westside-app but `### Repo` says westside-playground. File targets are playground HTML. Follows the #85-#88 convention but should be noted explicitly.

forgejo_admin commented

2026-03-27 00:04:23 +00:00

Author

Owner

Refinements from review-406-2026-03-25:

Nav — only link to pages that exist after this ticket. Updated nav for this ticket: Home | About | Staff | Sponsors | Schedule. Tryouts, Teams, Gear get added to nav by their respective tickets.
Dependency on #90. Both modify index.html. #90 must merge first (it restructures the landing page). Adding depends:ws-90.
Repo convention: Documented — westside-app tracks, westside-playground is target.

Also adding to all 5 new tickets: Kings/Queens toggle should use localStorage for site-wide program mode persistence. When user selects Queens anywhere, the color scheme swaps to Queens pink and persists across navigation. Implementation: localStorage.setItem('program', 'queens'), app.js reads on page load, applies .queens-active class to body. CSS swaps --color-accent from red to Queens pink. ~10 lines in shared/app.js.

**Refinements from review-406-2026-03-25:** 1. **Nav — only link to pages that exist after this ticket.** Updated nav for this ticket: `Home | About | Staff | Sponsors | Schedule`. Tryouts, Teams, Gear get added to nav by their respective tickets. 2. **Dependency on #90.** Both modify index.html. #90 must merge first (it restructures the landing page). Adding `depends:ws-90`. 3. **Repo convention:** Documented — westside-app tracks, westside-playground is target. **Also adding to all 5 new tickets:** Kings/Queens toggle should use `localStorage` for site-wide program mode persistence. When user selects Queens anywhere, the color scheme swaps to Queens pink and persists across navigation. Implementation: `localStorage.setItem('program', 'queens')`, `app.js` reads on page load, applies `.queens-active` class to body. CSS swaps `--color-accent` from red to Queens pink. ~10 lines in shared/app.js.

forgejo_admin referenced this issue

2026-03-27 00:05:00 +00:00

New page: Tryouts — history + upcoming with Kings/Queens toggle #92

forgejo_admin commented

2026-03-27 00:10:38 +00:00

Author

Owner

Scope Re-review: NEEDS_REFINEMENT

Review note: review-406-2026-03-25-r2
Refinements partially applied — 2 of 3 original findings resolved, but the issue body was not edited.

RESOLVED: depends:ws-90 label added. Issue #90 is now closed (merged). Dependency satisfied.
RESOLVED: Repo convention documented (westside-app tracks, westside-playground is target).
NOT RESOLVED: AC #3 still reads Home | About | Staff | Tryouts | Teams | Schedule | Gear | Sponsors in the issue body. Refinement comment says the correct nav is Home | About | Staff | Sponsors | Schedule, but the body — what agents execute from — was not updated. Edit the issue body to fix AC #3.
NEW (advisory): Kings/Queens localStorage toggle mentioned in refinement comment is not in the issue body. Recommend a separate cross-cutting ticket rather than duplicating across 5 tickets.

## Scope Re-review: NEEDS_REFINEMENT Review note: `review-406-2026-03-25-r2` Refinements partially applied — 2 of 3 original findings resolved, but the issue body was not edited. - **RESOLVED:** `depends:ws-90` label added. Issue #90 is now closed (merged). Dependency satisfied. - **RESOLVED:** Repo convention documented (westside-app tracks, westside-playground is target). - **NOT RESOLVED:** AC #3 still reads `Home | About | Staff | Tryouts | Teams | Schedule | Gear | Sponsors` in the issue body. Refinement comment says the correct nav is `Home | About | Staff | Sponsors | Schedule`, but the body — what agents execute from — was not updated. Edit the issue body to fix AC #3. - **NEW (advisory):** Kings/Queens localStorage toggle mentioned in refinement comment is not in the issue body. Recommend a separate cross-cutting ticket rather than duplicating across 5 tickets.

forgejo_admin commented

2026-03-27 00:11:09 +00:00

Author

Owner

AC #3 OVERRIDE (issue body has stale nav spec):

The correct nav for this ticket is: Home | About | Staff | Sponsors | Schedule

Do NOT add Tryouts, Teams, or Gear to the nav — those pages don't exist yet. They get added by their own tickets (#92, #93, #94).

This comment supersedes the issue body's AC #3.

**AC #3 OVERRIDE (issue body has stale nav spec):** The correct nav for this ticket is: `Home | About | Staff | Sponsors | Schedule` Do NOT add Tryouts, Teams, or Gear to the nav — those pages don't exist yet. They get added by their own tickets (#92, #93, #94). This comment supersedes the issue body's AC #3.

forgejo_admin referenced this issue from a commit

2026-03-27 00:14:13 +00:00

feat: dedicated About page + landing page trim (#91)

forgejo_admin referenced this issue from a pull request from forgejo_admin/westside-playground that will close it,

2026-03-27 00:14:52 +00:00

feat: dedicated About page + landing page trim (#91) #40

forgejo_admin closed this issue

2026-03-27 00:20:47 +00:00

forgejo_admin commented

2026-03-27 04:52:44 +00:00

Author

Owner