feat(comp:slider): unify trigger conditions for change events (#721)

fix #696
IDuxFE · Jan 13, 2022 · a3209fa · a3209fa
1 parent 8b5212b
commit a3209fa
Showing 1 changed file with 1 addition and 10 deletions.
diff --git a/packages/components/slider/src/useSlider.ts b/packages/components/slider/src/useSlider.ts
@@ -308,7 +308,7 @@ export function useSlider(props: SliderProps): SliderBindings {
       val = [modelValue as number]
     }
 
-    const newVal = val
+    valuesRef.value = val
       .map(num => {
         if (!isNumeric(num)) {
           return props.min
@@ -317,15 +317,6 @@ export function useSlider(props: SliderProps): SliderBindings {
         return calcValueByStep(num)
       })
       .sort((a, b) => a - b) // order
-
-    // When the legal value is not equal to the modelValue, update modelValue
-    if (val.every((v, i) => v !== newVal[i])) {
-      const modelValue = props.range ? newVal : newVal[0]
-      callEmit(props.onChange, modelValue)
-      accessor.setValue(modelValue)
-    }
-
-    valuesRef.value = newVal
   }
 
   return {