Problems with rest parameter on IE11

[mojoaxel]

I got an error debugging visjs/vis-timeline#72 on IE11.
There seams to be a problem in the selectiveExtend function.

It looks to me that the babel transform of the rest parameter "others" is buggy:

I cannot get this function run in IE without an error 😭

[Thomaash]

Vis Timeline uses Vis Util 1.1.4. Does it happen with 1.1.8?

[mojoaxel]

Vis Timeline uses Vis Util 1.1.4. Does it happen with 1.1.8?

I'm working with the current master branch of vis-util.

[Thomaash]

Hmm, I don't see any spreading being done in the function.

[mojoaxel]

Hmm, I don't see any spreading being done in the function.

It's the "rest parameter" others:
https://github.com/visjs/vis-util/blob/master/src/util.ts#L249

[Thomaash]

But that's not spreading (it just uses the same symbol). Atop of that this gets transformed into a for loop which also doesn't use any spreading.

[Thomaash]

In the bundle you have:

for (var _len = arguments.length, others = new Array(_len > 2 ? _len - 2 : 0), _key = 2; _key < _len; _key++) {
  others[_key - 2] = arguments[_key];
}

[mojoaxel]

You are right...I don't know anymore..I give up!
Maybe It's time that you get yourself a virtual machine with IE11 😉

The timeline is still broken on IE11 (see visjs/vis-timeline#72) and I really want to fix this before releasing vis-timeline@6

[yotamberk]

Seems to work for me on IE11 now on latest version.