Skip to content

Raise the Motion sensitivity X limit to 600% - #1302

Merged
hckrman101 merged 1 commit into
NotRexed:mainfrom
nicholasgower:feat/motion-sensitivity-x-600
Nov 9, 2026
Merged

hckrman101 merged 1 commit into
NotRexed:mainfrom
nicholasgower:feat/motion-sensitivity-x-600

Conversation

@nicholasgower

Copy link
Copy Markdown
Contributor

What changed

  • gpu/vk/vk_menu.cpp: the Controls > Motion sensitivity X row's maximum is now 600% (was 300%); the minimum (10%) and step (10%) are unchanged.
  • readme.md: the Controls table lists the new range.

Why

The game's horizontal motion aim is weak by default, so at 100% it feels insensitive in the X direction and 300% did not leave enough range to compensate. Doubling the maximum gives players room to make horizontal motion aiming usable. Motion sensitivity Y is unchanged.

Testing

  • tests/test_motion.py, tests/test_mouse_settings.py, tests/test_rumble_menu.py, tests/test_menu_hit.py and tests/test_menu_restart.py pass.
  • The full game-free suite was run; the remaining failures are local environment gaps (no clang, no zstandard, no Windows symlink privilege), not from this change.
  • Not tested in game: no controller or gyro was available here.

The horizontal motion aim is weak at the console's 100% default, so players could not turn it up far enough. Raise the row's maximum from 300% to 600%; the minimum and step are unchanged, and the option keeps saving and loading through the existing range.

Update the readme's Controls table to match.

Assisted-by: OpenCode (DeepSeek V4.1 Flash)

@hckrman101 hckrman101 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Widening the row's hi from 300 to 600 is all that's needed: the options file is clamped to each row's lo/hi when it loads (std::min(std::max(v, r.lo), r.hi)), so older saved values stay valid and a hand-edited value above 600 is pulled back to 600. Nothing else holds a 300 cap for this setting; pad_sdl.c just divides the option by 100 and hands it to hos_motion_update. The slider and stops_at_ends work from lo/hi, and the readme line matches. CI is green.

One thing for merge order: #1237 moves this row to TAB_MOTION and touches the same line (still 300 there), plus the readme's Controls table, so whichever lands second will conflict. It's a trivial fix, just keep the 600 in the moved row.

@hckrman101
hckrman101 merged commit b4d0277 into NotRexed:main Nov 9, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants