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

fix(react-color picker) Thumb design #33743

Conversation

ValentinaKozlova
Copy link
Contributor

Design changes of the thumb according to partner's feedback.
This is partner's design (dark theme):
image
They have different tokens for box-shadow and border-color but our new design uses V9 tokens close to partner's deisgn.

Previous Behavior

light:
image
dark:
image

New Behavior

light:
image
dark:
image

@ValentinaKozlova ValentinaKozlova force-pushed the fix/color-picker-thumb-design branch from 093a1ec to 651be52 Compare January 29, 2025 13:47
Copy link

github-actions bot commented Jan 29, 2025

📊 Bundle size report

Package & Exports Baseline (minified/GZIP) PR Change
react-color-picker-preview
ColorArea
50.298 kB
17.563 kB
50.406 kB
17.661 kB
108 B
98 B
react-color-picker-preview
ColorSlider
41.673 kB
15.46 kB
41.712 kB
15.541 kB
39 B
81 B
Unchanged fixtures
Package & Exports Size (minified/GZIP)
react-color-picker-preview
ColorPicker
18.581 kB
7.371 kB
🤖 This report was generated against 5a9a15f090aa7d01d5dc2f83ef7d22f91390eed2

Copy link

Pull request demo site: URL

@dmytrokirpa dmytrokirpa self-requested a review January 30, 2025 13:04
Copy link
Contributor

@dmytrokirpa dmytrokirpa left a comment

Choose a reason for hiding this comment

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

Consider using strokeWidths tokens instead of hardcoded values for border widths.

ValentinaKozlova and others added 4 commits January 30, 2025 15:12
…rc/components/ColorArea/useColorAreaStyles.styles.ts

Co-authored-by: Dmytro Kirpa <[email protected]>
…rc/components/ColorSlider/useColorSliderStyles.styles.ts

Co-authored-by: Dmytro Kirpa <[email protected]>
…rc/components/ColorSlider/useColorSliderStyles.styles.ts

Co-authored-by: Dmytro Kirpa <[email protected]>
…rc/components/ColorArea/useColorAreaStyles.styles.ts

Co-authored-by: Dmytro Kirpa <[email protected]>
@ValentinaKozlova ValentinaKozlova merged commit 678529c into microsoft:master Jan 30, 2025
16 checks passed
@ValentinaKozlova ValentinaKozlova deleted the fix/color-picker-thumb-design branch January 30, 2025 16:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants