Skip to content

Conversation

@gges5110
Copy link
Owner

Summary

  • Replaced TailwindCSS classes with Material-UI equivalents in GemDisplay component
  • Removed clsx dependency and className prop
  • Added size and showBorder props for better component API
  • Updated component test to remove TailwindCSS class assertion

Test plan

  • Verify GemDisplay component renders correctly with different props
  • Check that existing usages of GemDisplay continue to work
  • Run test suite to ensure no regressions

🤖 Generated with Claude Code

gges5110 and others added 2 commits June 24, 2025 16:02
Replaced TailwindCSS classes with Material-UI equivalents:
- Removed clsx dependency and className prop
- Replaced shadow-xs with boxShadow: 1
- Replaced border classes with Material-UI border styling
- Added size and showBorder props for better component API
- Updated test to remove TailwindCSS class assertion

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

Co-Authored-By: Claude <noreply@anthropic.com>
Standardized the use of the 'size' prop for GemDisplay and added the 'showBorder' prop in CardDisplay. Updated HelpPage to use consistent double-quoted style object keys and improved Box component usage for better readability and maintainability. Minor reordering and cleanup in NobleDisplay for prop consistency.
@gges5110 gges5110 merged commit 778bf2e into main Jun 24, 2025
1 check failed
@gges5110 gges5110 deleted the migrate/remove-tailwind-from-gem-display branch June 24, 2025 21:04
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