Skip to content

test(react-color-picker): add hook state coverage - #36527

Open
Dmytro Kirpa (dmytrokirpa) wants to merge 1 commit into
masterfrom
test/react-color-picker-hooks
Open

test(react-color-picker): add hook state coverage#36527
Dmytro Kirpa (dmytrokirpa) wants to merge 1 commit into
masterfrom
test/react-color-picker-hooks

Conversation

@dmytrokirpa

Copy link
Copy Markdown
Contributor

Motivation

Lock down the existing color-picker hook behavior before the reusable base-hook extraction in the next PR. Keeping this coverage in a separate bottom PR makes the behavioral expectations reviewable independently from the API refactor.

Changes

  • add hook-level tests for useAlphaSlider_unstable
  • add hook-level tests for useColorArea_unstable
  • add hook-level tests for useColorSlider_unstable
  • add hook-level tests for useColorPicker_unstable
  • cover default shape, context inheritance, prop precedence, state from props, and color-change forwarding

Validation

  • yarn nx test react-color-picker (13 suites, 132 tests, 4 snapshots)
  • yarn nx lint react-color-picker

PR stack

  1. This PR: add hook state coverage
  2. feat(react-color-picker): expose headless base APIs #36517: expose reusable color-picker base APIs
  3. feat(react-headless-components-preview): add color picker controls #36518: add headless color-picker controls

Please review and merge the stack in this order.

@github-actions

github-actions Bot commented Aug 6, 2026

Copy link
Copy Markdown

📊 Bundle size report

✅ No changes found

@github-actions

github-actions Bot commented Aug 6, 2026

Copy link
Copy Markdown

Pull request demo site: URL

@dmytrokirpa
Dmytro Kirpa (dmytrokirpa) force-pushed the test/react-color-picker-hooks branch from 1e79d2b to f1a28ac Compare August 7, 2026 12:38
@dmytrokirpa
Dmytro Kirpa (dmytrokirpa) force-pushed the test/react-color-picker-hooks branch from f1a28ac to 39796c4 Compare August 7, 2026 15:20
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