Juuz

Results 95 issues of Juuz

It's only used for options and has a lot of code dealing with `Option`, so this would make sense IMO

refactor

It's the tool used to drop the stacks (= break) the block. The current name could also suggest that it's an item stack of this block.

refactor

Some implementations (see `BrewingStandBlockEntity`) also check clearly insertion-related things like whether the slot is empty (for slots that only accept a stack with a single item). It's also only used...

discussion
refactor

(`SE` = `StatusEffect`) This would make it match things like `Entity` and `EntityType`.

refactor

This is only used for implementing `SlotActionType.QUICK_MOVE` in `internalOnSlotClick`, and in all classes where it's overridden, it implements the quick move logic. The `int index` parameter is the index of...

discussion
refactor

This interface isn't implemented on any sort of tooltips, "orderable" or not. It's simply for widgets that have a `List getOrderedTooltip()` method.

bug
refactor

It's true for vines, regular glass and plants, for example - none of which are translucent. This is probably `isTransparent`?

bug

- `modifiers(int)` uses `WORLD_SURFACE_WG_HEIGHTMAP` and is used for general vegetation (grass, ferns, flowers etc) - `modifiersWithChance(int, PlacementModifier)` uses `MOTION_BLOCKING_HEIGHTMAP` and is used for mushrooms - Suggested rename: `mushroomModifiers` - `modifiersBuilder(PlacementModifier)`,...

bug
refactor

As said in the conventions, > Use "write" for methods that save data to an *existing* object passed as a parameter. (This method writes into an existing `JsonObject`.) `serialize` is...

bug
refactor

To match the capitalisation used elsewhere in Yarn and Minecraft (the game uses "offhand" in localised strings).

bug
refactor