GLTFExporter: Added onlyVisible option

[fernandojsg]

I forgot to include this changes in a previous PR 👼

[mrdoob]

Maybe this is easier to read?

if ( child.visible || options.onlyVisible === false ) {

[fernandojsg]

Fixed!

[mrdoob]

Actually. What's the use case for exporting invisible objects?

[fernandojsg]

@mrdoob sometimes you would like to export all the meshes on the scene even if they're hidden, for example you could have an editor where you hide the background to focus on few objects and you don't want to toggle visibility of everything just to export, similar to how 3dsmax or similar works.

[fernandojsg]

but I agree that won't be the most common use case, that's why onlyVisible is set to true by default

[mrdoob]

Wouldn't it be cleaner to use object.layers for that?

[mrdoob]

Or even another scene. In https://threejs.org/editor/ I use another scene for the helpers.

[fernandojsg]

Layers could be another use case yep, but I still feel using just visible for an object could be useful. In aframe-inspector we allow the users to toggle the visibility on the sidebar:

If you're moving things around maybe you want to hide big objects that prevent you to work on the ones you want to focus so you just toggle few of them, but you still want to export the whole scene, and don't want to deal with layers and so on, as it's just part of your creation workflow but you won't use that visibility options on your final scene.
In the inspector I also use a different scene just for the helpers so it won't conflict with this option either.

[mrdoob]

I see I see 👌

[mrdoob]

Thanks!

[fernandojsg]

thanks! ;)