rename gltf_render_enabled to gltf_enable_standard_materials

[ChristopherBiscardi]

Objective

We currently describe the new gltf_render_enabled flag as "disabling PBR rendering", which is much broader than what it does in practice. In its current form it controls the construction and insertion of StandardMaterials on entities, but does not affect the PBR rendering functionality of the application, or of glTF assets. (A user could still create their own StandardMaterials or StandardMaterial extensions and insert them on the required meshes).

Solution

Rename the flag to more accurately depict what it does in practice, which is enable/disable the automatic construction and insertion of StandardMaterials when a glTF file is loading.

Testing

There's only one example that turns this off, gltf_extension_mesh_2d, which continues to run as expected.

[IceSentry]

That one is definitely better but to me the name should probably reflects a bit more that it inserts or generates something?

Something like insert_standard_material_on_gltf, gltf_insert_standard_material or something like that?

Not attached to any one in particular tbh but yeah, the current naming on main doesn't make sense to me.

[AntoninD-Galadrim]

My 2 cents idea, gltf_generate_standard_materials as an alternative