Hi,

Thank you for this PR proposal but to keep backward compatibility you need to provide a default value for spec.threejsCanvasId which would be 'threejsCanvas'. I suggest something like:

 const threejsCanvasId = (typeof(spec.threejsCanvasId)==='undefined')?'threejsCanvas':spec.threejsCanvasId;
 threejsCanvas=document.getElementById(threejsCanvasId);

I cannot accept your PR as this since it may break some demos where spec.threejsCanvasId is undefined.

Best,
Xavier

If spec.threejsCanvasId is undefined it should never hit this code block since it's wrapped in an if statement, correct? However, we could add a check to make sure spec.threejsCanvasId is of type string if we're concerned about that.

Yes you are right I didn't read the wrapping if statement