-
Notifications
You must be signed in to change notification settings - Fork 1
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
Fix misc. #49
Conversation
It might be a better idea to move the enums to separate files, but this will do for now.
Also reordered some of the overridden methods, so that they appear in the same order as in the abstract class.
This removes the need to check whether an element exists in the set, which is fairly inefficient compared to just using a set.
This makes the value of `CurrentState` visible in the inspector, but unable to be written to.
This fixes a bug that let the player change the direction of nearby towers while in the turret selection menu.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! Tested and read.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure if interactable functions should be abstact instead of virtual.
# Conflicts: # Assets/Scripts/Player/PlayerStateController.cs # Assets/Scripts/Player/PlayerUi.cs
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works on my machine™
Highlights:
Back
button toCancel
, and changed the key to Esc (b33da98)