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(react-color-picker) Combined components of the ColorPicker #33273

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

Conversation

ValentinaKozlova
Copy link
Contributor

@ValentinaKozlova ValentinaKozlova commented Nov 14, 2024

New Behavior

  • Refactoring of Sliders
  • Moved state to useColorSliderState
  • Used one HSV color format everywhere
  • Added ColorArea to the ColorPicker
  • Added onChange from context to ColorArea
  • Removed overflowColor from AlphaSlider (Probably, we'll remove this component at all if we don't need transparent prop)

color-picker

@ValentinaKozlova ValentinaKozlova self-assigned this Nov 14, 2024
@ValentinaKozlova ValentinaKozlova mentioned this pull request Oct 18, 2024
28 tasks
@ValentinaKozlova ValentinaKozlova marked this pull request as ready for review November 14, 2024 16:03
@ValentinaKozlova ValentinaKozlova requested a review from a team as a code owner November 14, 2024 16:03
Copy link

github-actions bot commented Nov 14, 2024

📊 Bundle size report

✅ No changes found

Copy link

Pull request demo site: URL

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.

1 participant