Dylan T.
Dylan T.
next-minor is fine
Yes, but I'll need to re-review it once the branches are in sync
I'd prefer this wasn't dependent on NBT. Bad enough that we have to deal with that for tiles already...
Furthermore, this permanent holder requirement presents unnecessary and annoying obstacles for copying sections of terrain, since objects like tiles are difficult to copy to different locations due to their inventories...
For posterity: One of the main obstacles to introducing an InventoryWrapper type like this was the problem of how to track inventory viewers. Unless the wrapper was cached, every inventory's...
Wrapping inventories in this manner would require a redesign of the handling of hotbar and held item
From a look into the Java code, these appear to have been implemented incorrectly. In addition to the growth issue, only the top block should tick. PM's impl currently ticks...
Maybe you have a problem with your internet connection. Working fine for me.