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

Basic garden mouse locking feature #607

Merged
merged 3 commits into from
Mar 28, 2024

Conversation

viciscat
Copy link
Collaborator

Or lower sensitivity whatever you want to call it.
For that one suggestion in the discord

The feature, config and translations added! Yipee!
An image can be seen at the discord link.

@AzureAaron AzureAaron added new feature This issue or PR is a new feature reviews needed This PR needs reviews labels Mar 21, 2024
Copy link
Collaborator

@AzureAaron AzureAaron left a comment

Choose a reason for hiding this comment

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

Perhaps there could also be a key bind toggle? this feels a little restrictive right now

@AzureAaron AzureAaron mentioned this pull request Mar 25, 2024
@AzureAaron AzureAaron linked an issue Mar 25, 2024 that may be closed by this pull request
@kevinthegreat1 kevinthegreat1 added changes requested This PR need changes and removed reviews needed This PR needs reviews labels Mar 26, 2024
Copy link
Collaborator

@kevinthegreat1 kevinthegreat1 left a comment

Choose a reason for hiding this comment

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

Works well! Only question is why is mouse sensitivity set to -1/3?

@kevinthegreat1 kevinthegreat1 added merge me please Pull requests that are ready to merge and removed changes requested This PR need changes labels Mar 27, 2024
@viciscat
Copy link
Collaborator Author

Sensitivity 0 isn't actually 0. Minecraft adds a bit in the code and -1/3 makes it 0.
image

@kevinthegreat1 kevinthegreat1 merged commit edb859d into SkyblockerMod:master Mar 28, 2024
1 check passed
@AzureAaron AzureAaron removed the merge me please Pull requests that are ready to merge label Mar 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new feature This issue or PR is a new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Snap aim feature
3 participants