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

Add horse armour hack #396

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

Protonull
Copy link
Contributor

This optionally allows players to craft iron, gold, and diamond horse armour like so:
image

The idea being you're taking Leather Horse Armour, which is already craftable, and plating it. But this can be changed.

Horse armour has the unique quality of being invincible, so this hack optionally reimplements durability. This is somewhat hacky though since the client has no idea about this and wont show a durability bar. Open to suggestions on how this could be resolved.

Since horse armour is invincible, the Unbreaking enchant has no effect, so this hack optionally reimplements Unbreaking.

@AngrySoundTech AngrySoundTech added Category: Feature Priority: Low Meta: Needs Discussion Needs to be discussed more before moving further labels May 1, 2024
@Protonull
Copy link
Contributor Author

This is somewhat hacky though since the client has no idea about this and wont show a durability bar.

This'll be resolved with 1.20.5 with the help of item components. It only currently means displaying an durability bar on the item stack, but hopefully Mojang will expand support so that combat damages any item with a damageable component, rather than just predefined items.
image
image

Adds hack that allows the registration of crafting recipes for horse armour, and the custom reimplementation of durability and unbreaking.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Category: Feature Meta: Needs Discussion Needs to be discussed more before moving further Priority: Low
Projects
Status: Ready for Review
Development

Successfully merging this pull request may close these issues.

None yet

2 participants