fix(#2664): useColorSliderState remove ternary statement from decrement

majornista · majornista · commit 06bd74d022b0 · 2022-03-03T18:41:29.000-05:00
snapValueToStep always starts from minValue.
diff --git a/packages/@react-stately/color/src/useColorSliderState.ts b/packages/@react-stately/color/src/useColorSliderState.ts
@@ -69,7 +69,7 @@ export function useColorSliderState(props: ColorSliderStateOptions): ColorSlider
     let {maxValue, minValue, step} = color.getChannelRange(channel);
     let channelValue = color.getChannelValue(channel);
     let s = Math.max(stepSize, step);
-    sliderState.setThumbValue(index, channelValue - s < minValue ? minValue : snapValueToStep(channelValue - s, minValue, maxValue, s));
+    sliderState.setThumbValue(index, snapValueToStep(channelValue - s, minValue, maxValue, s));
   }
 
   return {

Original file line number	Diff line number	Diff line change
`@@ -69,7 +69,7 @@ export function useColorSliderState(props: ColorSliderStateOptions): ColorSlider`
`69`	`69`	`let {maxValue, minValue, step} = color.getChannelRange(channel);`
`70`	`70`	`let channelValue = color.getChannelValue(channel);`
`71`	`71`	`let s = Math.max(stepSize, step);`
`72`		`- sliderState.setThumbValue(index, channelValue - s < minValue ? minValue : snapValueToStep(channelValue - s, minValue, maxValue, s));`
	`72`	`+ sliderState.setThumbValue(index, snapValueToStep(channelValue - s, minValue, maxValue, s));`
`73`	`73`	`}`
`74`	`74`
`75`	`75`	`return {`