Jump to conversation
Unresolved conversations (6)
@paramat paramat Jun 22, 2017
Since this is an enum, the added material should be added at the end i think, to not change the enumerations of the others.
Outdated
src/client/tile.h
paramat stujones11
@paramat paramat Jun 18, 2017
How about `TILE_MATERIAL_OPAQUE` because it is closely related to `TILE_MATERIAL_LIQUID_OPAQUE`? Opaqueness is also the reason it is being added.
Outdated
src/client/tile.h
stujones11
@paramat paramat Jun 18, 2017
The default for alpha is 255, but i'm not sure any mod nodes other than liquids use the 'alpha' nodedef, they seem to use 'use texture alpha' instead. Also, semi-translucency is buggy in MT so not really usable on normal nodes. So perhaps this check is not needed?
src/nodedef.cpp
paramat stujones11
@numberZero numberZero Jun 13, 2017
maybe change `NDT_NODEBOX` as well?
Outdated
src/nodedef.cpp
paramat stujones11
@numberZero numberZero Jun 13, 2017
why remove this?
Outdated
src/nodedef.cpp
paramat stujones11
@paramat paramat Jun 12, 2017
This case could be combined with `case TILE_MATERIAL_LIQUID_OPAQUE:` a few lines below.
Outdated
src/client/tile.h
stujones11 paramat
Resolved conversations (0)