Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Moves click_adjacent logic to MouseDown #6582

Closed
wants to merge 2 commits into from

Conversation

Git-Nivrak
Copy link
Contributor

@Git-Nivrak Git-Nivrak commented Jun 28, 2024

About the pull request

Makes it so adjacent clicks are handled on when you press down the button instead of when you press it up
This affects things like pointblanks, melee attacks, picking up items etc etc

Did some testing and fixed things like clicks not working or no click delay melee attacks but there are probably more so it needs to be tested

Explain why it's good for the game

Pointblank wise all of the gun code was already using MouseDown so no reason for pointblanks to stay behind, Other than that this should make adjacent clicks get "eaten" less (since if you too slow to mouse up the click won't be counted) and overall faster reaction time to your actions

Testing Photographs and Procedure

Screenshots & Videos

Put screenshots and videos here with an empty line between the screenshots and the <details> tags.

Changelog

🆑
balance: Adjacent clicks are now registered on mouse down instead of mouse up, this affects pointblanks, melee attacks etc.
/:cl:

@github-actions github-actions bot added the Balance You need to be a professional veteran game maintainer to comprehend what is being done here. label Jun 28, 2024
code/_onclick/click.dm Outdated Show resolved Hide resolved
@Git-Nivrak Git-Nivrak closed this Jun 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Balance You need to be a professional veteran game maintainer to comprehend what is being done here.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants