SpinBox: Fix custom_arrow_step by snapping it to step#108196
SpinBox: Fix custom_arrow_step by snapping it to step#108196Repiteo merged 1 commit intogodotengine:masterfrom
custom_arrow_step by snapping it to step#108196Conversation
|
does this break compatibility? |
|
#97561 seems to be Reproducible again |
b63d7be to
28e9cfa
Compare
|
|
28e9cfa to
15ffd92
Compare
Calinou
left a comment
There was a problem hiding this comment.
Tested locally, it works as expected. Code and documentation look good to me.
|
One issue I noticed though is that if This is effectively what #107302 aimed to fix, but this PR reverts it to fix another issue. We probably need to think of a better solution in the long run. |
|
So from now on |
Not exactly, it works the same as a multiple of step, but doesn't need to be a multiple of step. |
I do see some spinbox does not round by arrow step by default (e.g. in KDE Plasma), but there seem to be some are. |
|
seems like a good idea. But it can probably wait till 4.6 |
03e5b23 to
453f4f1
Compare
|
Opened a separate PR #108335 for that. |
|
Thanks! |
Fixes #108191.
The documentation is correct before #107302. But now when the two steps are inconsistent,
steptakes precedence.