Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Themes #202

Draft
wants to merge 6 commits into
base: main
Choose a base branch
from
Draft

Themes #202

wants to merge 6 commits into from

Conversation

mathewjordan
Copy link
Member

@mathewjordan mathewjordan commented Apr 27, 2024

This work introduces Radix UI Themes to the Clover IIIF suite. Since its early versions, Clover has utilized unstyled Radix Primitives for core components. By implementing themes, we seek to replace, wherever possible, these primitives with the relative styled component in the @radix-ui/themes library. Benefits of the themes integration include:

  • Less overall code maintenance
  • Greater accessibility coverage and general usability enhancement
  • Easy control of basic theme props accentColor, grayColor, and radius
  • Proven contrast of colors for dark and light modes
  • Ability for Clover IIIF to be more tightly integrated into out themes based projects
  • Grants an exit path from the no longer maintained @stitches/react CSS-in-JS styling package

As this change breaks previous styling methods, it likely should be v3.0.0.

@mathewjordan mathewjordan changed the title Themes (Proof of Concept) Themes Jul 10, 2024
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