generated from FabricMC/fabric-example-mod
-
Notifications
You must be signed in to change notification settings - Fork 10
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
[1.20] Camera look straight up down WIP 2 #132
Merged
khanshoaib3
merged 16 commits into
khanshoaib3:1.20.x
from
boholder:1.20.x-camera-look-straight-up-down-wip-2
Aug 20, 2023
Merged
[1.20] Camera look straight up down WIP 2 #132
khanshoaib3
merged 16 commits into
khanshoaib3:1.20.x
from
boholder:1.20.x-camera-look-straight-up-down-wip-2
Aug 20, 2023
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Ready for replacing dynamic camera rotating logic. Simplify dynamic camera rotating logic. Replace small camera rotating methods. Add simpler logic for replacing small methods.
Add real isKeyPressed method. Rename isKeyPressed method to isKeyPressing. Make all KeystrokeChecker test cases use one data supplier. Add isKeyReleased method for KeystrokeChecker. Rename MockKeyStroke. Fix MockKeyStroke bug. Add hasKeyPressedPreviousTick for KeystrokeChecker. Import unit test dependencies into common package.
Replace more if condition with KeystrokeChecker. Add isNotPressing method to KeystrokeChecker. Simplify KeystrokeChecker methods name. Replace logic with KeystrokeChecker in NarratorMenu. Simplify KeystrokeChecker methods name.
New KeystrokeTiming class for reusing. Rename KeystrokeChecker to Keystroke. Move Interval and KeystrokeChecker to new subpackage. Move Interval to independent file. Move KeystrokeChecker to independent file.
Add hasBeenTriggered method. Add pressed trigger count test. Add released trigger count test. Add not pressing trigger count test. Add pressing trigger count test. Change unit test structure for further cases. Add tests for Keystroke.TriggeredAt enums. Arrange logic order in KeystrokeTest.
Make hasPressedPreviousTick private. Rename method. Make isTriggered a pure-function. Simplify keys name in NarratorMenu.
This was referenced Aug 19, 2023
khanshoaib3
approved these changes
Aug 20, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Please merge #131 first for less review burden.
Keystroke
, make it cover most cases we currently encounter (like*PressedPreviousTick
conditions).Interval
andKeystroke
and other new utils into newutils.condition
subpackage, for easier using and testing.Keystroke
onNarratorMenu
,MouseKeySimulation
,BookEditScreenMixin
.NarratorMenu
for better readability.