-
Notifications
You must be signed in to change notification settings - Fork 8
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
Localization and meta-blocktypes #43
Conversation
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.
Looks good to me except for one very very small nitpick
src/main/java/io/github/debuggyteam/architecture_extensions/TypedGroupedBlockItem.java
Outdated
Show resolved
Hide resolved
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.
Looks good to me
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.
Nice work
Moves localization off of mixins and onto ordinary TranslatableTexts supplied by the BlockItem subclass. Should produce identical results for both current and legacy translation keys.
Allows localization overrides (see Create uk_ua.json #42) - for example,
block.architecture_extensions.minecraft.stone_arch
for the blockarchitecture_extensions:minecraft/stone_arch
Adds meta-blocktype selectors in the staticdata schema. Allows people to future-proof by specifying
#stone
as a blockType instead of copying in every type stone blocks use.Adds a 'creative' recipeConfigurator that will result in no recipes being generated.
Adds renderLayer to GroupedBlock, and
render_layer
key to the static-data schema, so you can use translucent base blocks