Review of export options

The exporter gives the user a ton of options, but I think a few of them can be simplified.

• There are two separate file -> export entries, one for glTF and one for GLB. The Blender devs asked us to combine this into a single menu entry, so the choice would migrate to the export options panel. The default would be GLB, to produce a single file for non-power users.

• Embed buffers, embed images - These shouldn’t be shown for GLBs, and I’m not sure we need to offer this for text glTFs either. If we do, perhaps there could be a combo box for “Export type” with three options [ Binary GLB, glTF Embedded, and glTF Separate ]

• Strip whitespace - Don’t offer this. Automatically strip whitespace for GLB, don’t strip it from glTF.

• Max indicies and Force Max indicies - Let’s get rid of these. If we really need to keep them they can be hidden behind the “experimental” flag in user prefs. But really the extension should just automatically do the right thing here (16 for smaller, 32 for larger meshes). Artists shouldn’t have to think about this stuff.

• Export Tangents, export Morph Tangents - Currently these default to True, I think we can switch them to False. Most viewers work fine without tangents, for example the Damaged Helmet model works most places without tangents.

• Export all layers - See https://github.com/KhronosGroup/glTF-Blender-IO/issues/193#issuecomment-453543304

• Apply Modifiers - Should this default to True? Or does that mess up skinning if true? (comment)

• Save settings pushpin - Let’s change the pushpin to a traditional checkbox. It could be renamed “Remember settings” with description “Store glTF export settings in the Blender project.”

• Infinite perspective cameras - Let’s remove this.

/cc @donmccurdy Thoughts on all this?

EDIT: Edited by @donmccurdy to add infinite perspective cameras to the list, and update status of other items.

[pjoe]

@emackey Done:

[ghost]

Would you mind removing the uint8_t option for the index buffer? Graphics APIs support only uint16_t and uint32_t. https://github.com/KhronosGroup/glTF-Blender-IO/blob/3bf29638524baa3bdda3105c8b4047355799cb3e/addons/io_scene_gltf2/blender/exp/gltf2_blender_gather_primitives.py#L76