[RaphaelFakhri]

Summary:

On web, the slider snaps a dragged or tapped value to the nearest multiple of step counted from 0 instead of from minimumValue. When minimumValue is not a multiple of step, the slider returns values outside the step grid. For example, with minimumValue={1}, maximumValue={10} and step={2}, releasing a quarter of the way along the track reports 4 instead of 3, and every reachable value is even, so 1, 3, 5, 7, 9 are never produced. iOS already counts steps from the minimum (RNCSlider.m), and Android counts its progress steps from the minimum too.

This change rounds (value - minimumValue) / step and adds minimumValue back, matching the native platforms. The existing precision rounding and bounds clamping in updateValue still apply.

Test Plan:

Added package/__test__/RNCSliderNativeComponent.web.test.tsx. It renders the web component with minimumValue={1}, maximumValue={10}, step={2}, releases the thumb at given track positions, and checks the value passed to onRNCSliderSlidingComplete, both normal and inverted.

• Without the fix: npx jest __test__/RNCSliderNativeComponent.web.test.tsx fails 2 of 2 tests (Expected: 3, Received: 4).
• With the fix: the same command passes 2 of 2 tests.
• npm test: 5 suites, 18 tests pass.
• npm run lint: no errors (only warnings that were already there).

Fixes #840

[BartoszKlonowski]

This is a valid fix and I'll approve it, but the merge can only happen as soon as v6 will land on main branch.
The reason I don't want to merge it to v5 is that the fix may in fact be not backward-compatible. If a user already adjusted the code to work around the bug, then shipping this fix will require certain refactor on their side.
Thank you @RaphaelFakhri for the fix and for the patience :)

[BartoszKlonowski]

Suggested change

// Steps are counted from minimumValue, as on the native platforms

[BartoszKlonowski]

I think the same calculation should be done for the accessibilityActions function to keep the behavior consistent.

[BartoszKlonowski]

As per: https://github.com/callstack/react-native-slider/pull/841/changes#r4335127201

Suggested change

	updateValue(Math.round((value + step - minimumValue) / step) * step + minimumValue);

[RaphaelFakhri]

@BartoszKlonowski Thanks for the review. 9f0b648 applies all three threads:

• Removes the comment line.
• Uses the same calculation for both accessibilityActions cases: Math.round((value ± step - minimumValue) / step) * step + minimumValue. The suggested line divides by step, and Slider defaults step to 0, so the calculation runs only when step is set. Without a step, the actions still move by a tenth of the range, as before.
• Adds a test for the accessibility actions. With value: 5, minimumValue: 1 and step: 3, increment now gives 7 and decrement gives 1.

npx jest passes (5 suites, 19 tests). Without the accessibilityActions change, the new test fails (expected 7, received 8). npx eslint reports no errors on the changed files.