feat: parent dashboard — badge, team link, schedule locations #26

Merged
forgejo_admin merged 1 commit from 25-parent-dashboard-cleanup into main 2026-03-16 22:57:38 +00:00

Summary

Cleans up the parent dashboard to reflect a placed player: correct division badge, team navigation, and placeholder schedule locations.

Changes

  • parent.html — Changed player badge from "Tryout #12" to "16U Boys" since this player is already placed on a team
  • parent.html — Added "View Team" button in the team info card linking to team.html, using existing btn btn-secondary btn-block classes (mirrors the "View Billing" button pattern)
  • parent.html — Replaced all three "Kongo Gym" schedule locations with "TBD" pending gym confirmation

Test Plan

  • Open parent.html in browser
  • Verify badge under player name reads "16U Boys" (not "Tryout #12")
  • Verify team card has a "View Team" button that links to team.html
  • Verify all three schedule entries show "TBD" instead of "Kongo Gym"
  • Confirm no layout or styling regressions (no new CSS added)

Review Checklist

  • Only parent.html modified — no other files touched
  • No new CSS — uses existing btn btn-secondary btn-block classes
  • All three acceptance criteria met (badge, team link, schedule locations)
  • Plan: plan-wkq
  • Closes #25
## Summary Cleans up the parent dashboard to reflect a placed player: correct division badge, team navigation, and placeholder schedule locations. ## Changes - `parent.html` — Changed player badge from "Tryout #12" to "16U Boys" since this player is already placed on a team - `parent.html` — Added "View Team" button in the team info card linking to `team.html`, using existing `btn btn-secondary btn-block` classes (mirrors the "View Billing" button pattern) - `parent.html` — Replaced all three "Kongo Gym" schedule locations with "TBD" pending gym confirmation ## Test Plan - Open `parent.html` in browser - Verify badge under player name reads "16U Boys" (not "Tryout #12") - Verify team card has a "View Team" button that links to `team.html` - Verify all three schedule entries show "TBD" instead of "Kongo Gym" - Confirm no layout or styling regressions (no new CSS added) ## Review Checklist - [x] Only `parent.html` modified — no other files touched - [x] No new CSS — uses existing `btn btn-secondary btn-block` classes - [x] All three acceptance criteria met (badge, team link, schedule locations) ## Related - Plan: `plan-wkq` - Closes #25
Update parent.html per issue #25:
- Change player badge from "Tryout #12" to "16U Boys" (player is placed)
- Add "View Team" button in team info card linking to team.html
- Replace "Kongo Gym" with "TBD" in all schedule locations

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Author
Owner

QA Review — PR #26

Scope Check

  • Only parent.html modified (1 file, +7/-4) — matches issue #25 boundary
  • No new CSS added — all classes (btn, btn-secondary, btn-block, badge-division) verified present in shared/app.css
  • No other files touched

Requirement Verification

Requirement Status Detail
Badge: "Tryout #12" → "16U Boys" PASS Line 36: badge-division class retained, text updated
"View Team" button in team card → team.html PASS Lines 75-77: follows exact same pattern as "View Billing" button (inline padding div + btn btn-secondary btn-block)
Schedule locations: "Kongo Gym" → "TBD" PASS All 3 occurrences replaced (lines 112, 123, 134)

Code Quality

  • The "View Team" button wrapper (div style="padding: 0 1rem 1rem;") is identical to the "View Billing" button wrapper on line 96 — consistent pattern, no structural invention.
  • No orphaned HTML tags, no broken nesting.
  • Diff is minimal and surgical — no unrelated changes.

Nits

None. Clean diff.

VERDICT: APPROVE — All three acceptance criteria from issue #25 are met. No regressions, no new CSS, no structural changes beyond scope.

## QA Review — PR #26 ### Scope Check - [x] Only `parent.html` modified (1 file, +7/-4) — matches issue #25 boundary - [x] No new CSS added — all classes (`btn`, `btn-secondary`, `btn-block`, `badge-division`) verified present in `shared/app.css` - [x] No other files touched ### Requirement Verification | Requirement | Status | Detail | |---|---|---| | Badge: "Tryout #12" → "16U Boys" | PASS | Line 36: `badge-division` class retained, text updated | | "View Team" button in team card → team.html | PASS | Lines 75-77: follows exact same pattern as "View Billing" button (inline padding div + `btn btn-secondary btn-block`) | | Schedule locations: "Kongo Gym" → "TBD" | PASS | All 3 occurrences replaced (lines 112, 123, 134) | ### Code Quality - The "View Team" button wrapper (`div style="padding: 0 1rem 1rem;"`) is identical to the "View Billing" button wrapper on line 96 — consistent pattern, no structural invention. - No orphaned HTML tags, no broken nesting. - Diff is minimal and surgical — no unrelated changes. ### Nits None. Clean diff. **VERDICT: APPROVE** — All three acceptance criteria from issue #25 are met. No regressions, no new CSS, no structural changes beyond scope.
forgejo_admin deleted branch 25-parent-dashboard-cleanup 2026-03-16 22:57:38 +00:00
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
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!26
No description provided.