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

feat: add player profile page prototype #4

Merged
forgejo_admin merged 1 commit from 1-feat-build-all-dashboard-prototypes-with into main 2026-03-16 01:37:06 +00:00
Conversation 1 Commits 1 Files changed 2 +397
forgejo_admin commented 2026-03-16 01:26:52 +00:00
Owner
Copy link

Summary

  • Adds player-profile.html prototype showing the full player detail view for the future /players/:id route
  • Shows player/parent view with profile header, four info cards, and a toggle-able edit form
  • Updates index.html to link the new prototype in the Dashboard Prototypes section

Changes

  • player-profile.html -- New page with profile header (96px avatar circle with initials, name, position/division badges, team), four info cards (Player Info with height/school/grad/hometown/target schools/tryout number, Team & Coach with clickable phone link, Payment Status with CURRENT badge and manage button, Stats & Recruitment placeholder with disabled button), and edit form with height/position/school/grad class/target schools/hometown/photo fields
  • index.html -- Added Player Profile nav card with LIVE status badge in Dashboard Prototypes section

Test Plan

  • Open player-profile.html at 390px viewport width -- verify dark theme, cards stack vertically, all text readable
  • Tap "Edit Profile" -- verify form appears, button hides
  • Tap "Cancel" in edit form -- verify form hides, Edit Profile button returns
  • Verify coach phone (555) 123-4567 is a clickable tel: link
  • Verify "View Stats" button is visually disabled (grayed out, no hover effect)
  • Open index.html -- verify Player Profile card appears in Dashboard Prototypes

Review Checklist

  • Passed automated review-fix loop
  • No secrets committed
  • No unnecessary file changes
  • Commit messages are descriptive

Related

  • Closes #3
  • plan-pal-e-docs -- Phase F11 (Frontend Polish)
## Summary - Adds `player-profile.html` prototype showing the full player detail view for the future `/players/:id` route - Shows player/parent view with profile header, four info cards, and a toggle-able edit form - Updates `index.html` to link the new prototype in the Dashboard Prototypes section ## Changes - `player-profile.html` -- New page with profile header (96px avatar circle with initials, name, position/division badges, team), four info cards (Player Info with height/school/grad/hometown/target schools/tryout number, Team & Coach with clickable phone link, Payment Status with CURRENT badge and manage button, Stats & Recruitment placeholder with disabled button), and edit form with height/position/school/grad class/target schools/hometown/photo fields - `index.html` -- Added Player Profile nav card with LIVE status badge in Dashboard Prototypes section ## Test Plan - [ ] Open `player-profile.html` at 390px viewport width -- verify dark theme, cards stack vertically, all text readable - [ ] Tap "Edit Profile" -- verify form appears, button hides - [ ] Tap "Cancel" in edit form -- verify form hides, Edit Profile button returns - [ ] Verify coach phone `(555) 123-4567` is a clickable `tel:` link - [ ] Verify "View Stats" button is visually disabled (grayed out, no hover effect) - [ ] Open `index.html` -- verify Player Profile card appears in Dashboard Prototypes ## Review Checklist - [ ] Passed automated review-fix loop - [ ] No secrets committed - [ ] No unnecessary file changes - [ ] Commit messages are descriptive ## Related - Closes #3 - `plan-pal-e-docs` -- Phase F11 (Frontend Polish)
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
forgejo_admin commented 2026-03-16 01:27:18 +00:00
Author
Owner
Copy link

Self-Review

Reviewed the diff. No blocking issues found.

Verified:

  • HTML structure follows existing patterns from parent.html, coach.html, admin.html
  • All page-specific styles scoped to <style> block -- shared/app.css untouched
  • Four info cards match design spec: Player Info, Team & Coach, Payment Status, Stats & Recruitment (placeholder)
  • Edit form has all required fields with proper label/for associations
  • Profile header: 96px avatar with red border, name, position/division badges, team
  • Bottom nav with correct active state on Profile
  • index.html change is minimal -- adds one nav card in Dashboard Prototypes section
  • Mobile-first layout using container class and CSS variables
  • Toggle edit form JS is minimal and functional

Minor note (non-blocking):

  • Inline style="color: var(--color-blue);" on coach phone link -- acceptable in prototype context, would use a class in production SvelteKit port
## Self-Review Reviewed the diff. No blocking issues found. **Verified:** - HTML structure follows existing patterns from `parent.html`, `coach.html`, `admin.html` - All page-specific styles scoped to `<style>` block -- `shared/app.css` untouched - Four info cards match design spec: Player Info, Team & Coach, Payment Status, Stats & Recruitment (placeholder) - Edit form has all required fields with proper `label`/`for` associations - Profile header: 96px avatar with red border, name, position/division badges, team - Bottom nav with correct active state on Profile - `index.html` change is minimal -- adds one nav card in Dashboard Prototypes section - Mobile-first layout using container class and CSS variables - Toggle edit form JS is minimal and functional **Minor note (non-blocking):** - Inline `style="color: var(--color-blue);"` on coach phone link -- acceptable in prototype context, would use a class in production SvelteKit port
forgejo_admin deleted branch 1-feat-build-all-dashboard-prototypes-with 2026-03-16 01:37:06 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
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!4
No description provided.