- PlacedFeature: Added mappings for `random`
- PlacedFeatures: Used more descriptive names for configured feature
registry entries
- TreeDecorator: Fixed a typo
* Misc small things
Looked at the mapping stats graph and went through and mapped some things
* Update BuiltinBiomes.mapping
* Update mappings/net/minecraft/world/gen/structure/RuinedPortalStructure.mapping
---------
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
* Minor fixes
- Fix casing in the name of GlImportProcessor, fixes#3488
- Use full word "playerInventory" instead of "inv" in
ScreenHandlerFactory for clarity. The conventions instruct to avoid
unnecessary abbreviations and this name could be mistaken for the
inventory of the block that the screen handler is for.
- BlockRenderManager.renderBlockAsEntity: vertexConsumer ->
vertexConsumers since it's a VertexConsumerProvider
* Rename stack parameter in block breaking methods to tool
Closes#2988. "Tool" more clearly shows what stack the param is supposed
to be.
* Block/AbstractBlockState.isTranslucent -> isTransparent
It's true for any block with a non-full shape as well as
vines, (colourless) glass and plants - none of these are translucent.
- "button" -> "widget" in many places as the widgets are often sliders
and can be any ClickableWidget
- ButtonListWidget -> OptionListWidget, since it's only used and usable
for options and can contain any option widget (closes#2932)
Closes#2028. The previous name can easily be confused with
getPlayerLookDirection. The rename shows that it's specifically
horizontal and also matches PlayerEntity.getHorizontalFacing.
(This essentially reverts a rename from #712, albeit with a more natural
order for the words in the name now.)
* ChestBlockEntity.onInvOpenOrClose -> onViewerCountUpdate
The new name is more accurate - the method takes in the old and new
viewer counts as parameters.
* ScreenHandler.close -> onClosed
Fixes#3503. The method doesn't close anything, it's only called when the screen handler gets closed.
* PlayerEntity.closeScreenHandler -> onHandledScreenClosed
Fixes#2612. The method closes nothing, it just runs the closing tasks
*after* the screen has closed. The current name can easily be confused
with closeHandledScreen(), which does actually close the screen.
* Document PlayerEntity.closeHandledScreen
* 23w07a stuff
Got a bit carried away...
Some names aren't ideal, and there's still more that could be done, so feedback is appreciated
* More + suggestions
With the introduction of sub-packs included in the base game, there are several classes for the 1.20 datapack and several for the vanilla datapack which would benefit from being renamed and put into packages to make this clearer.
Classes which are only used for the vanilla datapack have been put into a `vanilla` subpackage, and ones for the 1.20 datapack have been put into a `onetwenty` subpackage.
Furthermore, classes which are only used for the vanilla datapack have been prefixed with `Vanilla`.
The above changes should not be considered impactful as these classes are strictly for generating the vanilla and 1.20 datapacks and are incredibly unlikely to be referenced by modders.
However, tag providers which were named with an `Abstract` prefix have had this removed. This makes them match with other providers. This is a somewhat impactful refactor.
* 23w04a stuff
mostly smithing related
* fix conflict
* Add PalettedPermutationsAtlasSource
* Update RecipeProvider.mapping
* a few more missing things
* Update mappings/net/minecraft/client/gui/screen/recipebook/RecipeResultCollection.mapping
* Apply suggestions from code review
* damnit
* more
* Armor trim changes
* Map more armor stuff
* Armor trim recipe and more
* Begin work on forging screen changes
* More armor trim related mappings
* Fix mistakes
* Fix screen handler package
* More fixes
* Forgot one
* More descriptive name
* migrate from openjdk base image to eclipse-temurin
https://hub.docker.com/_/openjdk has been deprecated for some time and will no longer get security updates
* Update publish.yml
* Update release-filament.yml
* 1.19.3-pre2 stuff
yarn is very inconsistent with whether to use `index`, `id`, or even in one scenario `ordinal` for integer indexes in value lists...
* Apply suggestions from reviews
named some methods
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: modmuss50 <modmuss50@gmail.com>