Skip to content

Conversation

@gges5110
Copy link
Owner

Summary

  • Migrated PlayerGems component from Tailwind CSS to Material-UI styling
  • Replaced gem-size-medium custom utility class with Material-UI sx props
  • Converted responsive sizing from Tailwind (min-w-8 w-8 sm:w-12 h-8 sm:h-12) to Material-UI breakpoints
  • Maintained consistent gem sizing: 32px on mobile, 48px on desktop
  • Improved consistency with existing Material-UI components in the app

Test plan

  • Full test suite passes (11/11 tests)
  • ESLint passes with no issues
  • Responsive gem sizing behavior preserved
  • Visual appearance remains consistent
  • Component functionality unchanged

🤖 Generated with Claude Code

gges5110 and others added 2 commits June 24, 2025 19:22
Replace Tailwind CSS classes with Material-UI Box components and sx props:
- Convert grid layout classes (gap-2, grid, grid-cols-8) to Material-UI Box with display="grid"
- Replace gem-size, rounded-full, select-none, shadow-xs, flex-initial with sx prop styling
- Maintain consistent 8-column grid layout and gem sizing (48px)
- Use Material-UI Box components for better consistency with the rest of the app

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

Co-Authored-By: Claude <noreply@anthropic.com>
Replace gem-size-medium Tailwind utility class with Material-UI sx props:
- Convert gem-size-medium (min-w-8 w-8 sm:w-12 h-8 sm:h-12) to responsive sx values
- Use explicit pixel values: 32px (xs) and 48px (sm) for width/height
- Maintain responsive behavior with Material-UI breakpoint system
- Remove dependency on custom Tailwind utility class

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

Co-Authored-By: Claude <noreply@anthropic.com>
@gges5110 gges5110 merged commit 17993ca into main Jun 25, 2025
2 checks passed
@gges5110 gges5110 deleted the migrate/playergems-tailwind-removal branch June 25, 2025 00:41
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.

2 participants