On Sun, 24 Mar 2024 15:11:16 GMT, drmarmac <d...@openjdk.org> wrote: > This PR should fix the issue and cover all relevant cases with new tests. > > Note: This involves a small behavior change, as can be seen in > dblSpinner_testWrapAround_decrement_twoSteps() in SpinnerTest.java:749. With > this change the wraparound behavior is similar to that of the IntegerSpinner.
the end result of stepping once by `amountToStepBy` must be equivalent of incrementing by one a total of `amountToStepBy` times, wouldn't you agree? so the formula for positive `amountToStepBy` values should look like `((val + amountToStepBy) % (max - min)) + min` or something along these lines. ------------- PR Comment: https://git.openjdk.org/jfx/pull/1431#issuecomment-2023596265
