Skip to content

Customizable rate limit for cloak and vision when bound to mouse wheel#1826

Merged
AdamTadeusz merged 5 commits intoNeotokyoRebuild:masterfrom
AdamTadeusz:338_customizableRateRontrolRorRloakRndRpticsWhenBoundToMouseWheel
Mar 12, 2026
Merged

Customizable rate limit for cloak and vision when bound to mouse wheel#1826
AdamTadeusz merged 5 commits intoNeotokyoRebuild:masterfrom
AdamTadeusz:338_customizableRateRontrolRorRloakRndRpticsWhenBoundToMouseWheel

Conversation

@AdamTadeusz
Copy link
Contributor

Description

Intended to stop accidentally toggling the two on and off repeatedly when bound to the mouse wheel

Toolchain

  • Windows MSVC VS2022

@AdamTadeusz AdamTadeusz marked this pull request as draft March 11, 2026 14:33
@AdamTadeusz AdamTadeusz marked this pull request as ready for review March 11, 2026 14:47
@AdamTadeusz AdamTadeusz requested a review from a team March 12, 2026 10:14
@DESTROYGIRL
Copy link
Contributor

Wouldn't it be best if this rate limit could be applied to any input or toggle bound to mousewheels rather than exclusively cloak or vision?

@AdamTadeusz
Copy link
Contributor Author

probably, but I was afraid of making too many changes.

@AdamTadeusz
Copy link
Contributor Author

@Agiel would you approve this if i set the default value of cl_neo_mouse_wheel_action_cool_down to 0, so it skips past this custom check unless you actively change the value

Copy link
Contributor

@DESTROYGIRL DESTROYGIRL left a comment

Choose a reason for hiding this comment

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

It looks fine and does not seem to break anything in my testing. Should not change any behaviour with default 0 at least
Ought to be revisited in the future so it can be applied to all inputs and not just these two

@DESTROYGIRL DESTROYGIRL requested a review from a team March 12, 2026 15:58
@AdamTadeusz AdamTadeusz merged commit 36087ff into NeotokyoRebuild:master Mar 12, 2026
7 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.

3 participants