Skip to content

fix: UI cleanup — migrate legacy tokens and polish interactions - #31

Merged
lbowsher merged 1 commit into
mainfrom
lb/draft-rankings-integration
Mar 22, 2026
Merged

lbowsher merged 1 commit into
mainfrom
lb/draft-rankings-integration

Conversation

@lbowsher

@lbowsher lbowsher commented Mar 22, 2026

Copy link
Copy Markdown
Owner

Summary

  • Migrated all legacy color tokens (text-primary-text, text-secondary-text, bg-surface) to shadcn semantic tokens (text-foreground, text-muted-foreground, bg-card) across 15 files
  • Removed unused legacy CSS variables and Tailwind aliases from globals.css and tailwind.config.ts
  • Replaced emoji icons in theme toggle with Lucide SVG icons (Moon, Sun, Stars, TreePine, Flame, Sunset, Snowflake)
  • Replaced text arrows (▲/▼) with Lucide ChevronUp/ChevronDown in Previous Leagues toggle
  • Fixed non-functional hover:bg-opacity-80 in league home (→ hover:brightness-110)
  • Added cursor-pointer to interactive team rows and Previous Leagues toggle
  • Fixed generic alt text on team avatars to use actual team name

Test plan

  • Toggle all 7 themes and verify colors render correctly (no missing/broken styles)
  • Check theme toggle dropdown renders SVG icons instead of emojis
  • Verify Previous Leagues toggle shows chevron icons and has pointer cursor
  • Hover over team rows on league home page — should brighten subtly
  • Hover over team rows on home page — should show pointer cursor
  • Inspect team avatar alt text in dev tools — should show team name

🤖 Generated with Claude Code

…er states

- Migrate all legacy color tokens (text-primary-text, text-secondary-text, bg-surface)
  to shadcn semantic tokens (text-foreground, text-muted-foreground, bg-card) across 15 files
- Remove legacy CSS variables and Tailwind aliases that are no longer referenced
- Replace emoji icons in theme toggle with Lucide SVG icons (Moon, Sun, Stars, etc.)
- Replace text arrows (▲/▼) with Lucide ChevronUp/ChevronDown in Previous Leagues toggle
- Fix non-functional hover:bg-opacity-80 → hover:brightness-110 in league home
- Add cursor-pointer to interactive team rows and Previous Leagues toggle
- Fix generic alt text on team avatars to use actual team name

🤖 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 6:42am

@lbowsher
lbowsher merged commit ddb0ef7 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