Settings: Mouse sensitivity: added a text input box next to the slider

Closes #2833 (closed)

Added a text box next to the slider for adjusting mouse sensitivity.

There is primitive input validation which disallows TYPING chars which aren't 0.123456789. This doesn't prevent ftos_mindecimals from changing the value to inf or any repeats of . such as 1.2.3.4. IMHO having float string sanitation is above the scope of this MR and should not be implemented here. Perhaps at the engine level instead.

changed milestone to %Xonotic 0.8.6

added MR SizeTiny Topic: Menu labels

mentioned in issue #2833 (closed)

changed the description

added 1 commit

• 05598418 - explicit include

Compare with previous version

I've never seen a text field for changing the mouse sensitivity anywhere. From an UX perspective, this seems terrible. I think there are two better alternatives here:

1. Add back the slider (with 0.1 step increments) and have a small textbox next to it. The value then needs to be linked between both widgets.
2. Replace the textbox with a spinbox (Example: Qt, but possibly with Up and Down buttons on different sides). I don't think we already have an implementation for that.

In any case, the input for this should be validated (is a number, is in range). It will probably be enough to only have two decimal places.

marked this merge request as draft

added 1 commit

• 477d8458 - complete refactor hack abomination

Compare with previous version

changed the description

changed title from Draft: Change sensitivity from slider to text input box to Draft: Settings: Mouse sensitivity: added a text input box next to the slider

marked this merge request as ready

removed MR SizeTiny label

added 2 commits

• 8a4cbe5c - MenuQC: only draw slider label if textSpace > 0
• 20911bb5 - MenuQC: Sensitivity slider: cleaner solution to hide label

Compare with previous version

Could we also change the default of 6 to something lower?

2 or 3? i think a lot of mice default to somewhere around 1800dpi?

changed milestone to %Xonotic 0.9

marked this merge request as draft

changed the description

added MRWaiting For Patches label

removed MRWaiting For Patches label