Review: Landing page: restore original energy (re-review)

review-405-2026-03-25-r2 Review

review ready

Project

Verdict: READY

Re-review of board item #405 after refinement comment on Forgejo issue #90. Original review (review-405-2026-03-25) found 3 issues. All 3 are now resolved.

Template Completeness

[x] Lineage — "Follows playground overhaul (#85-#88). Parent note: westside-playground-overhaul"
[x] Repo — forgejo_admin/westside-playground
[x] User Story — prospective parent conversion funnel
[x] Context — thorough comparison of original vs current structure, target 5-section layout
[x] File Targets — index.html to modify, shared/style.css + other pages explicitly excluded
[x] Acceptance Criteria — 7 items, all verifiable
[x] Test Expectations — 3 items including mobile 390px visual check
[x] Constraints — taste decision, brand preservation, sacred subtitle, tryout banner slot-back plan
[x] Checklist — 5 items including Lucas phone review
[x] Related — original site reference + parent overhaul note

All required template sections present.

File Targets

[x] index.html — verified at westside-playground/index.html. Dual hero CTAs confirmed ("Member Sign In" + "View Schedule"). Coaches section confirmed (5 coach-cards). Bottom CTA "Want to Get Involved?" confirmed.
[x] shared/style.css — correctly listed as do-not-touch. CSS already has .step, .step-number, .grid.grid-3 classes needed for the new "How to Join" section.
[x] "How to Join" section — refinement provides exact 3-step copy (Reach Out / Meet the Coaches / Get Placed) with no-tryout-context adaptation. Uses same .step / .step-number / .grid.grid-3 pattern as the original. Agent-executable.
[x] staff.html — verified: 5 coach-card elements present. Removing coaches from index.html will not affect staff page.

Repo Placement

OK. Established convention per #75: westside-app is the issue tracker, westside-playground is the code target. The ### Repo field in the issue body explicitly says forgejo_admin/westside-playground. Agent reads the issue body and follows the Repo field to clone the correct repo. Unambiguous.

Dependencies

#380 / issue #84 (parent overhaul): CLOSED. Column: done. All 4 sub-tickets (#381-384 / issues #85-#88) also done. Refinement clarifies #90 is a follow-up, not a child. No dependency confusion.
#132 Phase 15 (Production Port): Still in_progress. Implicit sequencing: #405 lands in playground, Phase 15 ports to SvelteKit. Acceptable — playground-first is the established gate (per feedback_playground_gate.md).
#356 (travel schedule + nav): Still next_up. #405 does not change nav structure (only removes coaches section from page body). No conflict.

Acceptance Criteria

All 7 criteria are verifiable. The refinement fills the gap from the original review: "How to Join" section now has explicit 3-step copy. "Page feels focused" remains subjective but is gated by the Lucas phone review constraint. No automated tests — acceptable for static HTML playground work.

Blast Radius

Coach CSS classes (.coach-grid, .coach-card) used in 10 files across the playground. Removing HTML from index.html is safe — CSS remains, staff.html still renders all 5 coaches.
Hero CTA classes (.btn-primary-lg, .btn-outline-lg) only used in index.html. Single-CTA replacement is contained.
No downstream consumers affected. Playground is static HTML; westside-app (SvelteKit) is separate, updated by Phase 15.

Recommendation

No action needed. All three issues from the original review are resolved:

Repo convention documented — westside-app tracks, westside-playground targets. Per #75. ### Repo field is unambiguous.
"How to Join" copy provided — exact 3 steps with no-tryout adaptation. CSS classes pre-exist. Agent-executable.
#380 closed — parent overhaul complete. #90 is an independent follow-up.

Ticket is ready for agent execution.