Pixelgaffer / MineFluence

2 stars 0 forks source link

rework block rendering #6

Closed CodeCrafter47 closed 9 years ago

CodeCrafter47 commented 9 years ago

use new json stuff

benefits:

CodeCrafter47 commented 9 years ago

The json models are now updated to look same as the model in the tileentity renderer.

The tile entity renderer is still in effect for rendering placed blocks. Using the json models for this is difficult as long as the different machine blocks are just metadata based subblocks. Being effectively the same block requires all machine blocks to have the same properties, but in effect they have quite different properties. For example the display block can be aligned to multiple directions and also has the progress properties, while the needs neither of these properties. A future thermometer might need a heat property. To be clear this is just about the properties which are important to the client.

For this reason I would suggest to create a separate block for each machine block. Any thoughts?

CherryWorm commented 9 years ago

Yeah, that propably wouldnt be that hard and it will make the code more readable as well.

CodeCrafter47 commented 9 years ago

Done.