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

Skillet stuff #890

Open
wants to merge 25 commits into
base: 1.19
Choose a base branch
from
Open

Conversation

MehVahdJukaar
Copy link
Contributor

This PR does the following:

  • Add skillet flipping animation
  • Makes skillet use Item Renderer instead of baked models while still keeping it json model based to easily position it with blcokbench.
  • Adds skillet cook bar while cooking (replaces durability bar there)
  • replaces various instances of InteractionResult.SUCCESS with the correct .sideSuccess(level.isClientSide)
  • simplifies some logic around the above mentioned code.

Also i realized i accidentally worked on the 1.19 branch. This PR should be able to be cherry picked into 1.20 main aswell with minimal changes (A single argument in ISTER method changed in newer forge)

@MehVahdJukaar
Copy link
Contributor Author

In last commits I added the following:

  • Replaced all RegistryObject fields with Supplier fields. This is just so multi loader addons can reference both versions registries directly. Same was done for Configuration class
  • Added skillet block flipping and sound

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.

2 participants