feat: clickable team cards, role-aware team detail, clearer admin stats #8

Merged

forgejo_admin merged 1 commit from 7-feat-clickable-team-cards-role-aware-tea into main

2026-03-16 21:56:44 +00:00

forgejo_admin commented

2026-03-16 21:49:13 +00:00

Owner

Summary

Three UX improvements to the playground prototype: clearer admin stat labels, clickable team cards on admin dashboard, admin-only edit section on team detail, and a "Browse Teams" section for coaches.

Changes

admin.html -- Renamed stat card labels ("Active Players" to "Registered", "Current" to "Paid Up"). Wrapped all 6 team cards in <a> tags with red border hover effect. First card links to team.html, others use # placeholder hrefs.
team.html -- Added "Admin Tools" section at the bottom with "Edit Roster" button, "Export CSV" button, and "Assign Players" placeholder. Wrapped in HTML comment marking it admin-only for production role gating.
coach.html -- Added "Browse Teams" section below coach actions with 3 placeholder team cards (Kings, Queens, 15U Boys) linking to team.html.

Test Plan

Open admin.html -- verify stat labels read "Registered" and "Paid Up"
Hover team cards on admin dashboard -- verify red border hover, no blue hyperlink styling
Click "16U Boys Kings" card -- verify it navigates to team.html
Open team.html -- scroll to bottom, verify "Admin Tools" section with Edit Roster / Assign Players
Open coach.html -- scroll below roster grid, verify "Browse Teams" section with 3 team cards

Review Checklist

Only touches files specified in the issue (admin.html, team.html, coach.html)
No new CSS classes added to shared/app.css -- page-specific styles only
Clickable cards look like cards, not blue hyperlinks
No JavaScript added
No unrelated changes

Plan: plan-wkq (traceability)
Forgejo issue: #7

Closes #7

## Summary Three UX improvements to the playground prototype: clearer admin stat labels, clickable team cards on admin dashboard, admin-only edit section on team detail, and a "Browse Teams" section for coaches. ## Changes - **`admin.html`** -- Renamed stat card labels ("Active Players" to "Registered", "Current" to "Paid Up"). Wrapped all 6 team cards in `<a>` tags with red border hover effect. First card links to `team.html`, others use `#` placeholder hrefs. - **`team.html`** -- Added "Admin Tools" section at the bottom with "Edit Roster" button, "Export CSV" button, and "Assign Players" placeholder. Wrapped in HTML comment marking it admin-only for production role gating. - **`coach.html`** -- Added "Browse Teams" section below coach actions with 3 placeholder team cards (Kings, Queens, 15U Boys) linking to `team.html`. ## Test Plan - Open `admin.html` -- verify stat labels read "Registered" and "Paid Up" - Hover team cards on admin dashboard -- verify red border hover, no blue hyperlink styling - Click "16U Boys Kings" card -- verify it navigates to `team.html` - Open `team.html` -- scroll to bottom, verify "Admin Tools" section with Edit Roster / Assign Players - Open `coach.html` -- scroll below roster grid, verify "Browse Teams" section with 3 team cards ## Review Checklist - [x] Only touches files specified in the issue (admin.html, team.html, coach.html) - [x] No new CSS classes added to shared/app.css -- page-specific styles only - [x] Clickable cards look like cards, not blue hyperlinks - [x] No JavaScript added - [x] No unrelated changes ## Related - Plan: `plan-wkq` (traceability) - Forgejo issue: #7 Closes #7

forgejo_admin added 1 commit

2026-03-16 21:49:13 +00:00

feat: clickable team cards, role-aware team detail, clearer admin stat labels 9e89189fb8

- Rename admin stat labels: "Active Players" -> "Registered", "Current" -> "Paid Up"
- Wrap admin team cards in <a> tags linking to team.html (red border hover)
- Add admin-only "Admin Tools" section to team.html with Edit Roster and Assign Players placeholders
- Add "Browse Teams" section to coach.html with 3 placeholder team cards

Closes #7

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

forgejo_admin commented

2026-03-16 21:49:56 +00:00

Author

Owner

PR Review -- #8

Acceptance Criteria Check

Criterion	Status
Admin stat cards read "Registered" and "Paid Up"	Pass
Team cards on admin dashboard are clickable `<a>` tags	Pass
team.html has admin-only edit section with "Edit Roster" / "Assign Players"	Pass
coach.html has "Browse Teams" section with placeholder team cards	Pass
All changes use existing CSS from shared/app.css -- no new shared styles	Pass
Clickable cards look like cards, not blue hyperlinks	Pass
No JavaScript added	Pass
Only touches admin.html, team.html, coach.html	Pass

Observations (non-blocking)

Duplicated a.team-card CSS in both admin.html and coach.html page-specific styles. Fine for playground; would move to shared/app.css in production.
One inline style in team.html admin section (style="padding: 1rem; display: flex; gap: 0.5rem;"). Acceptable for a prototype placeholder.

Diff Stats

3 files changed, +92 / -14
No changes to shared/app.css or any files outside scope

VERDICT: APPROVE

## PR Review -- #8 ### Acceptance Criteria Check | Criterion | Status | |-----------|--------| | Admin stat cards read "Registered" and "Paid Up" | Pass | | Team cards on admin dashboard are clickable `<a>` tags | Pass | | team.html has admin-only edit section with "Edit Roster" / "Assign Players" | Pass | | coach.html has "Browse Teams" section with placeholder team cards | Pass | | All changes use existing CSS from shared/app.css -- no new shared styles | Pass | | Clickable cards look like cards, not blue hyperlinks | Pass | | No JavaScript added | Pass | | Only touches admin.html, team.html, coach.html | Pass | ### Observations (non-blocking) - **Duplicated `a.team-card` CSS** in both `admin.html` and `coach.html` page-specific styles. Fine for playground; would move to `shared/app.css` in production. - **One inline style** in `team.html` admin section (`style="padding: 1rem; display: flex; gap: 0.5rem;"`). Acceptable for a prototype placeholder. ### Diff Stats - 3 files changed, +92 / -14 - No changes to `shared/app.css` or any files outside scope **VERDICT: APPROVE**