dmurdoch/rgl

texmode = "replace" doesn't handle lighting

dmurdoch opened this issue · 2 comments

The new texmode = "replace" material property default looks bad in e.g. persp3d()`, because it ignores lighting. What we really want is "modulate" but with a base colour of "white".

Maybe change the base colour to "white" in some conditions, but leave the default as "modulate"?

PR #321 changes the default back to "modulate", but doesn't make automatic colour changes -- they look too messy.

Instead, I should add more detailed documentation about how textures are handled.

Fixed by #323.