Skip to content

feat: redesign home page, add mobile nav, clean up invite pages - #34

Merged
lbowsher merged 3 commits into
mainfrom
lb/ui-tier3-redesign
Mar 22, 2026
Merged

lbowsher merged 3 commits into
mainfrom
lb/ui-tier3-redesign

Conversation

@lbowsher

@lbowsher lbowsher commented Mar 22, 2026

Copy link
Copy Markdown
Owner

Summary

  • Home page redesign: Replaced flat list with responsive Card grid (1 col mobile, 2 col desktop), added league type badges (NFL/NBA/NCAAM), moved "New League" button into header
  • Empty state: When user has no teams, shows Trophy icon with "No teams yet" message and CTA to create a league
  • Mobile NavBar: Added hamburger menu (Menu/X icons) at md: breakpoint — slides down panel with nav links, theme toggle, and auth button. Increased nav height to 56px for better touch targets
  • Invite pages cleanup: Removed duplicate custom header bars (each had its own ThemeToggle + AuthButtonServer), now uses the global NavBar. Replaced raw HTML inputs/buttons with shadcn Card, Button, Input, Label components. Bundle size reduced ~45%
  • Loading skeleton: Updated to match new card grid layout with avatar circles and text placeholders

Test plan

  • Home page: verify card grid renders with team name, league name, and league type badge
  • Home page: test with 0 teams — should show empty state with Trophy icon and "Create a League" button
  • Home page: verify 2-column grid on desktop, 1-column on mobile
  • NavBar: on mobile (<768px) verify hamburger icon appears, desktop links are hidden
  • NavBar: tap hamburger — panel slides down with Home, Rankings, theme toggle, logout
  • NavBar: navigate to a page — mobile menu auto-closes
  • Invite pages: visit /invite/team/[id] and /invite/league/[id] — should use global NavBar (no duplicate header)
  • Invite pages: verify form still works (join team, select team from dropdown)
  • Loading state: navigate to home with slow connection — should show skeleton card grid

🤖 Generated with Claude Code

Luke Bowsher and others added 2 commits March 22, 2026 02:36
…obile improvements

- Replace all alert() calls with sonner toast notifications (success/error/warning)
- Fix getSession() → getUser() security issue in login page and auth button
- Fix hardcoded text-gray-400 → text-muted-foreground in auth button
- Add prefers-reduced-motion CSS media query for accessibility
- Improve mobile responsiveness: commissioner controls wrap, weekly scores hide on small screens
- Add skip-to-content link and focus-visible rings for keyboard navigation
- Upgrade home page loading skeleton with card-shaped placeholders

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
- Redesign home page with Card grid layout, league type badges, and responsive 2-column grid
- Add empty state with Trophy icon and CTA when user has no teams
- Move "New League" button into header row alongside "My Teams" heading
- Upgrade NavBar: increase height to h-14, add hamburger menu for mobile with slide-down panel
- Desktop nav links hidden on small screens, mobile panel shows nav + theme toggle + auth
- Clean up invite pages: remove duplicate custom header bars (ThemeToggle + AuthButton),
  replace raw HTML with shadcn Card/Button/Input/Label components
- Update loading skeleton to match new card grid layout
- Invite page bundle size reduced ~45% by removing duplicate component imports

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@vercel

vercel Bot commented Mar 22, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
fanteasy Ready Ready Preview, Comment Mar 22, 2026 5:30pm

Team name and league name are now independent links again — team name
navigates to the team page, league name to the league home. Each
highlights individually on hover.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@lbowsher
lbowsher merged commit 6bf4b96 into main Mar 22, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant