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

Hotkey Updates #34

Merged
merged 17 commits into from
Dec 31, 2024
Merged

Hotkey Updates #34

merged 17 commits into from
Dec 31, 2024

Conversation

Akeianova
Copy link
Contributor

@Akeianova Akeianova commented Nov 20, 2024

Adds new modifier to increase granularity in jump and speed modularity. Also ensures that increments are locked to multiples of 5 to avoid value drift.

@wlhlm wlhlm added the ongoing freeze - don't merge Not just a bug fix and thus affected by a current freeze for a upcoming version label Nov 23, 2024
@Dream-Master Dream-Master requested a review from a team November 23, 2024 22:49
@Dream-Master Dream-Master removed the ongoing freeze - don't merge Not just a bug fix and thus affected by a current freeze for a upcoming version label Dec 8, 2024
Copy link
Member

@serenibyss serenibyss left a comment

Choose a reason for hiding this comment

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

@Akeianova sorry for taking so long to review! I left a couple comments and questions

src/main/java/thaumicboots/main/Config.java Outdated Show resolved Hide resolved
src/main/java/thaumicboots/main/Config.java Outdated Show resolved Hide resolved
src/main/java/thaumicboots/api/IBoots.java Outdated Show resolved Hide resolved
src/main/java/thaumicboots/api/IBoots.java Outdated Show resolved Hide resolved
Removed incorrect non-support comment
fix jump and speed change function logic
@Akeianova
Copy link
Contributor Author

I also apologize, but I do not have access to my dev environment at the moment so if this is incorrect I will have to fix it properly at a later date

@serenibyss
Copy link
Member

@Akeianova Yeah, the change to Config I meant to change the text like:

- The rate that boot jump and speed modifiers change when the Jump/Speed Modulation key is pressed. 'Modified values will not receive support'
+ The rate that boot jump and speed modifiers change when the Jump/Speed Modulation key is pressed.

rather than remove the whole line. No rush, take your time

@serenibyss serenibyss added the 🚧 Testing on Zeta Do not merge yet, testing this PR on Zeta label Dec 29, 2024
@Akeianova
Copy link
Contributor Author

fixed the config changes, my bad

@serenibyss serenibyss merged commit 59f6fe5 into GTNewHorizons:master Dec 31, 2024
1 check passed
@serenibyss serenibyss removed the 🚧 Testing on Zeta Do not merge yet, testing this PR on Zeta label Jan 13, 2025
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.

4 participants