fix slider lint issues

Author: zombieJ

src/Slider.tsx

@@ -214,9 +214,11 @@ const Slider = React.forwardRef<SliderRef, SliderProps<number | number[]>>((prop
 
   // ============================ Marks =============================
   const markList = React.useMemo<InternalMarkObj[]>(() => {
-    return Object.keys(marks || {})
+    const markRecord = marks || {};
+
+    return Object.keys(markRecord)
       .map<InternalMarkObj>((key) => {
-        const mark = marks[key];
+        const mark = markRecord[key];
         const markObj: InternalMarkObj = {
           value: Number(key),
         };
@@ -269,7 +271,7 @@ const Slider = React.forwardRef<SliderRef, SliderProps<number | number[]>>((prop
 
       // When count provided or value is `undefined`, we fill values
       if (count || mergedValue === undefined) {
-        const pointCount = count >= 0 ? count + 1 : 2;
+        const pointCount = count !== undefined && count >= 0 ? count + 1 : 2;
         returnValues = returnValues.slice(0, pointCount);
 
         // Fill with count
@@ -287,6 +289,8 @@ const Slider = React.forwardRef<SliderRef, SliderProps<number | number[]>>((prop
 
     return returnValues;
   }, [mergedValue, rangeEnabled, mergedMin, count, formatValue]);
+  const rawValuesRef = React.useRef(rawValues);
+  rawValuesRef.current = rawValues;
 
   // =========================== onChange ===========================
   const getTriggerValue = (triggerValues: number[]) =>
@@ -349,6 +353,8 @@ const Slider = React.forwardRef<SliderRef, SliderProps<number | number[]>>((prop
     rangeEditable,
     minCount,
   );
+  const draggingValueRef = React.useRef(draggingValue);
+  draggingValueRef.current = draggingValue;
 
   /**
    * When `rangeEditable` will insert a new value in the values array.
@@ -455,7 +461,7 @@ const Slider = React.forwardRef<SliderRef, SliderProps<number | number[]>>((prop
 
   React.useEffect(() => {
     if (keyboardValue !== null) {
-      const valueIndex = rawValues.indexOf(keyboardValue);
+      const valueIndex = rawValuesRef.current.indexOf(keyboardValue);
       if (valueIndex >= 0) {
         handlesRef.current.focus(valueIndex);
       }
@@ -485,7 +491,7 @@ const Slider = React.forwardRef<SliderRef, SliderProps<number | number[]>>((prop
   const dragging = draggingIndex !== -1;
   React.useEffect(() => {
     if (!dragging) {
-      const valueIndex = rawValues.lastIndexOf(draggingValue);
+      const valueIndex = rawValuesRef.current.lastIndexOf(draggingValueRef.current);
       handlesRef.current.focus(valueIndex);
     }
   }, [dragging]);
@@ -520,8 +526,9 @@ const Slider = React.forwardRef<SliderRef, SliderProps<number | number[]>>((prop
   }));
 
   // ========================== Auto Focus ==========================
+  const autoFocusRef = React.useRef(autoFocus);
   React.useEffect(() => {
-    if (autoFocus) {
+    if (autoFocusRef.current) {
       handlesRef.current.focus(0);
     }
   }, []);