-
Notifications
You must be signed in to change notification settings - Fork 349
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
Hand handling improvement #2142
base: 1_20_5
Are you sure you want to change the base?
Conversation
…ack.STRICT_NETWORK_TYPE to represent this)
…t to correct drink/eat time from component, send damage event on living entity damage
… are still in ItemComponent)
…ing type. add wolf variant meta field
src/main/java/net/minestom/server/event/player/EntitySwingHandEvent.java
Outdated
Show resolved
Hide resolved
/** | ||
* Represents the main or off hand of the player. | ||
*/ | ||
public enum Hand { |
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.
I think this should stay on Player. Even if it is slightly incorrect (eg for LivingEntity#swingHand to take a Player.Hand), I believe most people will look for player hand. Also makes this change less breaking
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.
I don't know if it's that breaking, using Player.Hand
is still correct. Do you still want this enum to be in a player class?
575b59b
to
39badc3
Compare
95f7327
to
05a4bb7
Compare
This pull request:
EntitySwingHandEvent
inLivingEntity#swingHand(Hand)
method instead of in a packet listener.I think that it would be nice if this pull request was merged, because we could avoid messy code like this.