3D Tiles Sandcastle example has mouse issues in IE 11

[mramato]

Obviously not a big deal until we are ready to bring 3d-tiles into master, but I wanted to submit this before I forget.

The mouse doesn't behave as expected when running the 3D Tiles Sandcastle example in IE11. Panning and rotation are wildly inaccurate. Running the demo in Chrome works fine. It looks like other examples work fine in the 3d-tiles branch, so maybe it's something specific to the new example.

[bicealyh]

The master branch also has the same problem. The mouse wheel does not respond In IE11.

[mramato]

@bicealyh you are correct, but the two are unrelated. This mouse wheel is actually a much bigger issue since it's broken in master. I submitted #3643 for us to fix this before the next release. Thanks for the heads up.

[pjcozzi]

@TomPed would you like to try this one when you are available?

[TomPed]

I'm trying to load the sandcastle example and I am getting an error:

An error occurred while rendering.  Rendering has stopped.
undefined
TypeError: Object doesn't support property or method 'fill'
TypeError: Object doesn't support property or method 'fill'
``'

[TomPed]

This is the support for the fill method for Internet Explorer.

TypedArray.prototype.fill()

[mramato]

Up to @lilleyse and @pjcozzi, but if we think this will affect performance, we can write a helper function that calls fill if it's defined and falls back to iteration if it's not.

[pjcozzi]

@mramato's approach is fine. Look in the file history; we just added fill so we were basically using the fallback beforehand.

[TomPed]

Panning and rotation are wildly inaccurate.

@mramato when you say this, are you referring to the granularity of the panning and rotation, or when you go to pan and rotate, sometimes, no movement occurs. The latter is what I see.

[mramato]

@JoshuaStorm can you take a look at this? Create a new private function in Core called arrayFill, which uses the polyfill from https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/fill, but instead of patching the prototype, have the function take the array as the first parameter. You'll also want to clean up the formatting and error checks to match our conventions as well as add specs.

In order to take advantage of fill on browsers that do support it, in arrayFill check if Array.prototype.fill exists first and if it does just us that instead. See getTimestamp for a simple example of setting this up.

Then you can update Cesium3DTileBatchTableResource.js and replace the calls to fill with the new function and verify it works in all browsers.

[JoshuaStorm]

@mramato I cannot seem to find Cesium3DTileBatchTableResources.js in this repo. Am I missing something?

[mramato]

This needs to be done in a branch off of 3d-tiles

[JoshuaStorm]

Makes sense, figured I was forgetting something like that.

[mramato]

There still seem to be several issues in the IE11 3D Tiles demo, but I'll try and write up more specific issues in place of this current one.