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

feat: ink kit pill component #88

Merged
merged 2 commits into from
Feb 6, 2025
Merged

feat: ink kit pill component #88

merged 2 commits into from
Feb 6, 2025

Conversation

ink-victor
Copy link
Contributor

@ink-victor ink-victor commented Jan 24, 2025

This PR introduces a new Tag component built using CVA (Class Variance Authority), following shadcn/ui's proven component architecture. This approach provides a flexible and type-safe way to manage component variants while maintaining a clean, maintainable codebase. By leveraging CVA's compound variants and TypeScript integration, we get robust type checking, predictable styling, and an extensible API that makes it easy to add new variants or modify existing ones. The component seamlessly integrates with our design system's tokens and follows best practices for React component development.

image

@ink-victor ink-victor requested a review from a team as a code owner January 24, 2025 12:15
Copy link

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-88.d2yrqj5dnlvopf.amplifyapp.com

@ink-victor ink-victor force-pushed the feat/ink-kit-pill-component branch 2 times, most recently from d3fd142 to 7802835 Compare January 24, 2025 12:37
@ink-victor ink-victor changed the title draft: feat: ink kit pill component feat: ink kit pill component Jan 24, 2025
@ink-victor ink-victor force-pushed the feat/ink-kit-pill-component branch from 7802835 to 8fb32f2 Compare January 24, 2025 12:38
@ink-victor ink-victor force-pushed the feat/ink-kit-pill-component branch from 8fb32f2 to 34265ce Compare January 24, 2025 12:42
@fran-ink fran-ink merged commit 9a16c44 into main Feb 6, 2025
5 checks passed
@fran-ink fran-ink deleted the feat/ink-kit-pill-component branch February 6, 2025 14:28
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