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

PR for Issue #32 - Scroll adjust periodic attack interval while holding the keybind #33

Conversation

morgantic
Copy link

I managed to create a working build for the Feature Request from #32

This is my first PR on GitHub in ages, I probably did everything wrong, sorry about that X)

Added function to adjust TWEAK_PERIODIC_ATTACK_INTERVAL by scrolling while holding the TWEAK_PERIODIC_ATTACK keybind
Added "tweakeroo.message.set_periodic_attack_interval_to" String
@zly2006
Copy link

zly2006 commented Jul 22, 2024

Thanks for your PR. it will be nice if you can also add this to period use.

@morgantic
Copy link
Author

morgantic commented Jul 22, 2024

Thanks for your PR. it will be nice if you can also add this to period use.

Oh, totally. I completely forgot there are other periodic features. I'll add that as well!

Do you think it would also make sense to be able to scroll adjust the period of the periodicHoldUse and periodicHoldAttack? The hold duration would remain only changeable in the config.

@zly2006
Copy link

zly2006 commented Jul 22, 2024

Do you think it would also make sense to be able to scroll adjust the period of the periodicHoldUse and periodicHoldAttack? The hold duration would remain only changeable in the config.

yes I agree.

Added scroll handler for PERIODIC_USE_INTERVAL, PERIODIC_HOLD_ATTACK_INTERVAL and PERIODIC_HOLD_USE_INTERVAL
Added strings "tweakeroo.message.set_periodic_use_interval_to", "tweakeroo.message.set_periodic_hold_attack_interval_to" and "tweakeroo.message.set_periodic_hold_use_interval_to"
@morgantic
Copy link
Author

Added PERIODIC_USE_INTERVAL, PERIODIC_HOLD_ATTACK_INTERVAL and PERIODIC_HOLD_USE_INTERVAL scroll handlers.

@sakura-ryoko sakura-ryoko merged commit 117b5f0 into sakura-ryoko:pre-rewrite/fabric/1.21 Jul 23, 2024
2 checks passed
@sakura-ryoko
Copy link
Owner

Seems ok, so I'll merge it.

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