Move SliderRoot.onValueChange's activeThumbIndex to third position

mui · Nov 19, 2024 · 76f5098 · 76f5098
1 parent 38afd34
commit 76f5098
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/packages/mui-base/src/Slider/Root/SliderRoot.test.tsx b/packages/mui-base/src/Slider/Root/SliderRoot.test.tsx
@@ -1255,7 +1255,7 @@ describeSkipIf(typeof Touch === 'undefined')('<Slider.Root />', () => {
     });
 
     it('should pass "name" and "value" as part of the event.target for onValueChange', async () => {
-      const handleValueChange = stub().callsFake((newValue, thumbIndex, event) => event.target);
+      const handleValueChange = stub().callsFake((newValue, event) => event.target);
 
       const { getByRole } = await render(
         <TestSlider onValueChange={handleValueChange} name="change-testing" value={3} />,

diff --git a/packages/mui-base/src/Slider/Root/useSliderRoot.ts b/packages/mui-base/src/Slider/Root/useSliderRoot.ts
@@ -233,7 +233,7 @@ export function useSliderRoot(parameters: useSliderRoot.Parameters): useSliderRo
         value: { value, name },
       });
 
-      onValueChange(value, thumbIndex, clonedEvent);
+      onValueChange(value, clonedEvent, thumbIndex);
     },
     [name, onValueChange],
   );
@@ -546,7 +546,7 @@ export namespace useSliderRoot {
      * You can pull out the new value by accessing `event.target.value` (any).
      * **Warning**: This is a generic event not a change event.
      */
-    onValueChange?: (value: number | number[], activeThumb: number, event: Event) => void;
+    onValueChange?: (value: number | number[], event: Event, activeThumbIndex: number) => void;
     /**
      * Callback function that is fired when the `pointerup` is triggered.
      *