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 fixes #1270

Merged
merged 13 commits into from
Feb 25, 2021
Merged

Armor stand fixes #1270

merged 13 commits into from
Feb 25, 2021

Conversation

Camotoy
Copy link
Member

@Camotoy Camotoy commented Sep 10, 2020

  • Armor stands now show armor if invisible. This allows both names and armor to show on an armor stand, and should allow for custom models that use armor stands to show, to an extent. I've been unable to find a custom model plugin at this time so verification on this would be great.
  • Armor stand orientation when visible is now correct.

- Armor stands now show armor if invisible. This allows both names and armor to show on an armor stand, and should allow for custom models that use armor stands to show, to an extent. I've been unable to find a custom model plugin at this time so verification on this would be great.
- Armor stand orientation when visible is now correct.
@Camotoy Camotoy linked an issue Sep 10, 2020 that may be closed by this pull request
@Technoguyfication
Copy link

I noticed armor stands with nametags work fine with Bedrock on PC and Mobile, but on Xbox (possibly PS4 and Switch too) they aren't visible. Will this fix those inconsistencies and allow armor stand names to show on Xbox?

@Camotoy
Copy link
Member Author

Camotoy commented Dec 11, 2020

Note: 047bf5f may make something here irrelevant.

@Technoguyfication
Copy link

Is there anything blocking this PR from being merged?

@Camotoy
Copy link
Member Author

Camotoy commented Dec 13, 2020

I'm simply concerned about performance implications.

@bobhenl
Copy link

bobhenl commented Dec 25, 2020

Hopefully will be merged soon...

@Camotoy Camotoy merged commit 9f5a356 into GeyserMC:master Feb 25, 2021
@Camotoy Camotoy deleted the armorstandfixes branch February 25, 2021 17:54
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.

Armor stand invisibility is inconsistent with Java
4 participants