fix: onAfterChange should not trigger when click on track

react-component · Nov 29, 2023 · 49e9d35 · 49e9d35
1 parent fa77b3e
commit 49e9d35
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 2 deletions.
diff --git a/docs/examples/debug.tsx b/docs/examples/debug.tsx
@@ -52,6 +52,9 @@ export default () => {
             console.log('Change:', nextValues);
           //   setValue(nextValues as any);
           }}
+          onAfterChange={(v) => {
+            console.log('AfterChange:', v);
+          }}
           // value={value}
 
           min={0}

diff --git a/docs/examples/marks.tsx b/docs/examples/marks.tsx
@@ -25,7 +25,7 @@ export default () => (
   <div>
     <div style={style}>
       <p>Slider with marks, `step=null`</p>
-      <Slider min={-10} marks={marks} step={null} onChange={log} defaultValue={20} />
+      <Slider min={-10} marks={marks} step={null} onChange={log} defaultValue={20} onAfterChange={(v) => console.log('AfterChange:', v)} />
     </div>
 
     <div style={style}>

diff --git a/src/Slider.tsx b/src/Slider.tsx
@@ -329,7 +329,6 @@ const Slider = React.forwardRef((props: SliderProps, ref: React.Ref<SliderRef>)
 
       onBeforeChange?.(getTriggerValue(cloneNextValues));
       triggerChange(cloneNextValues);
-      onAfterChange?.(getTriggerValue(cloneNextValues));
       if (e) {
         onStartDrag(e, valueIndex, cloneNextValues);
       }