Slider onChangeEnd event fires before onChange when using arrow keys

[kgarner7]

Dependencies check up

• I have verified that I use latest version of all @mantine/* packages

What version of @mantine/* packages do you have in package.json?

7.7.2

What package has an issue?

@mantine/core

What framework do you use?

Vite

In which browsers you can reproduce the issue?

All

Describe the bug

When using a Slider component with onChange and onChangeEnd events, using arrow keys to change the value will cause the onChangeEnd event to fire before onChange

If possible, include a link to a codesandbox with a minimal reproduction

No response

Possible fix

Reorder the events at

mantine/packages/@mantine/core/src/components/Slider/Slider/Slider.tsx

Lines 239 to 240 in 4c9a875

	onChangeEnd?.(nextValue);
	setValue(nextValue);

Self-service

• I would be willing to implement a fix for this issue

[rtivital]

onChnageEnd is called whenever the user stops any interactions with the component. onChange is called when value changes. The logic is correct, the order of functions calls does not matter.

[kgarner7]

I disagree. The order matters if you are relying on onChangeEnd to denote that seeking has finished. This caused a real bug jeffvli/feishin#581

[rtivital]

Okay, I do not mind to merge the PR