search

Buuz135 / FunctionalStorage

FunctionalStorage
MIT License
30 stars 32 forks source link

[1.20.1] Compacting Drawers not using stone tag #284

Open vizthex123 opened 3 months ago

vizthex123 commented 3 months ago

I noticed that the other blocks like them use the forge:stone tag, but the Compacting Drawers don't.

Functional Storage Version: 1.2.10

Titanium Version: 3.8.28

Buuz135 commented 3 months ago

What?

vizthex123 commented 3 months ago

The recipes don't use the forge:stone tag, which is a bit annoying since they'll only accept vanilla Stone.

ffuentesm commented 1 month ago

I think that is the intent. The recipe asks for Vanilla Stone, not any stone block.

vizthex123 commented 1 month ago

yes, but it should use the tag to have better mod compatibility.

ffuentesm commented 1 month ago

Hmm, I see what you mean. You are mostly referring to how the other drawers use the stone tag in their crafting recipe, while the compacting drawer does not. image

On that note, I would also recommend changing the fluid drawers, from the Minecraft:planks to the stones tag, due to their texture color being more in line with the compacting drawer and drawer controller.