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

Armor Stand rebase and improvements #5307

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

LittleBigBug
Copy link
Contributor

@LittleBigBug LittleBigBug commented Sep 22, 2021

This PR is a take over of #4330 and rebases fully onto master (commits condensed) with many fixes.

Ticks a checkbox in #3499

I didn't test Lua bindings/markers.

  • Rebase and update
  • Tested on 1.8, 1.9, 1.12
  • Entity and block checks
  • Placing Armor
  • Removing individual pieces of armor
  • Creative mode fixes
  • Entity / Block checks
  • Clang-tidy
  • Arrows Destroying armor stands
  • Gravity on snow layers (?)

Buom01 and others added 3 commits September 21, 2021 22:20
* Send attached state to new player

* Define armor stand basis

* Complete armor model, need tweaks to be used in game
* Handle armor stand and left hand of armor slots

* Handle armor stand storage

* Add support for invisibility

Co-authored-by: Ethan Jones <ethan@yasfu.net>
* add HOOK_PLAYER_CLICKING_AT_ENTITY interaction

* Prepare left hand integration

* Handle interaction ClickedAt

* Replace armor stand property size by isMarker and isSmall

* Enable gravity for armor stand and allow nogravity

* Polishing docs

* Remove TODOs and apply Morgan rule

* Patch HasGravity according to Float encoding

* Save ArmorStand in vanilla way

* Switch rotation from Vector3d to Vector3f and make save working properly according to armor stand data

* Show left hand content to other players
* Send armor rotations if invisible, but not if it's a marker

* Hit -> Click

* IsSmall -> IsSizeSmall

* South -> Z+

Co-authored-by: Ethan Jones <ethan@yasfu.net>
@LittleBigBug LittleBigBug force-pushed the armor-stand-pk branch 6 times, most recently from c23fc34 to acce86b Compare September 23, 2021 18:52
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.

None yet

2 participants