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): design of thumb #33526

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ValentinaKozlova
Copy link
Contributor

@ValentinaKozlova ValentinaKozlova commented Dec 30, 2024

  • Added more contrast border for the thumb
  • Made preview box in HC with no color adjustment

Previous Behavior

image

New Behavior

image

Copy link

github-actions bot commented Dec 30, 2024

📊 Bundle size report

Package & Exports Baseline (minified/GZIP) PR Change
react-color-picker-preview
ColorArea
50.253 kB
17.547 kB
50.263 kB
17.553 kB
10 B
6 B
react-color-picker-preview
ColorSlider
41.493 kB
15.381 kB
41.503 kB
15.386 kB
10 B
5 B
Unchanged fixtures
Package & Exports Size (minified/GZIP)
react-color-picker-preview
ColorPicker
32.22 kB
11.925 kB
🤖 This report was generated against 0b5ff6e25513a28011ba742c0c479056fcecf68b

Copy link

Pull request demo site: URL

@@ -54,7 +54,7 @@ const useThumbStyles = makeStyles({
outlineStyle: 'none',
Copy link
Collaborator

@fabricteam fabricteam Dec 30, 2024

Choose a reason for hiding this comment

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

🕵🏾‍♀️ visual regressions to review in the fluentuiv9 Visual Regression Report

Avatar Converged 2 screenshots
Image Name Diff(in Pixels) Image Type
Avatar Converged.badgeMask.chromium.png 3 Changed
Avatar Converged.Badge Mask RTL.chromium.png 2 Changed
ColorPicker Converged 5 screenshots
Image Name Diff(in Pixels) Image Type
ColorPicker Converged.Default.chromium.png 34 Changed
ColorPicker Converged.shape.chromium.png 72 Changed
ColorPicker Converged.Default RTL.chromium.png 34 Changed
ColorPicker Converged.Default Dark Mode.chromium.png 21 Changed
ColorPicker Converged.Default High Contrast.chromium.png 79 Changed
Drawer 2 screenshots
Image Name Diff(in Pixels) Image Type
Drawer.Full Overlay RTL.chromium.png 3285 Changed
Drawer.overlay drawer full.chromium.png 1154 Changed

@ValentinaKozlova ValentinaKozlova force-pushed the fix/color-picker-thumb-design branch from 1835946 to b4f47f7 Compare January 2, 2025 15:27
@ValentinaKozlova ValentinaKozlova marked this pull request as ready for review January 2, 2025 15:28
@ValentinaKozlova ValentinaKozlova requested a review from a team as a code owner January 2, 2025 15:28
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.

2 participants