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

[sparkle] - refactor(Chip): clickable Chip #11508

Merged
merged 2 commits into from
Mar 21, 2025
Merged

Conversation

JulesBelveze
Copy link
Contributor

Description

This PR enhances the Chip component to support both clickable and link behaviors

Risk

Low

Deploy Plan

Publish sparkle

…ilities

 - Advanced the Chip component version to 0.2.444 to include new interactive features
 - Enhanced the Chip component to support both clickable and link behaviors
 - Adjusted the padding in size variants for improved visual consistency
 - Updated IconButton hover state to match the new outlined style variant
 - Aligned the Chip story to reflect the component's ability to handle onClick events

[sparkle] - refactor: streamline Chip props and types for better clarity

 - Divided the ChipProps into ChipBaseProps, ChipLinkProps, and ChipButtonProps for clearer type distinction
 - Extracted common prop types to simplify the Chip component's interface

[sparkle] - fix: IconButton outline variant hover text color

 - Changed the IconButton outline variant hover text color for better consistency with dark mode
@JulesBelveze JulesBelveze force-pushed the sparkle/clickable-chip branch from 85c24fb to 0da55b3 Compare March 20, 2025 17:05
Copy link
Collaborator

@pinotalexandre pinotalexandre left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

…nent

- Replace the complex type Record with a simpler type definition and clear keyof Omit construction
@JulesBelveze JulesBelveze merged commit 0295c65 into main Mar 21, 2025
5 checks passed
@JulesBelveze JulesBelveze deleted the sparkle/clickable-chip branch March 21, 2025 08:03
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