fix: onAfterChange now returns the correct value when used as a con…

…trolled component Fixes #201
zillow · Nov 10, 2020 · 03fcc17 · 03fcc17
1 parent 0f921e3
commit 03fcc17
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/src/components/ReactSlider/ReactSlider.jsx b/src/components/ReactSlider/ReactSlider.jsx
@@ -425,15 +425,16 @@ class ReactSlider extends React.Component {
     };
 
     onEnd(eventMap) {
-        // Allow controlled updates to continue
-        this.setState({ pending: false });
-
         if (eventMap) {
             removeHandlers(eventMap);
         }
         if (this.hasMoved) {
             this.fireChangeEvent('onAfterChange');
         }
+
+        // Allow controlled updates to continue
+        this.setState({ pending: false });
+
         this.hasMoved = false;
     }