Add setTimeout fallback for requestAnimationFrame

[brentertz]

Address issue #987 - Add setTimeout fallback for requestAnimationFrame

[valtlai]

requestAnimationFrame doesn’t have the second parameter. Can you remove it, please?

[brentertz]

Yep, removed. Not sure why that was in there in the first place or why I didn't notice it.

[Golmote]

Indeed. It was there because it originally was a setTimeout and the person who changed it forgot to remove it.
It went unnoticed because that change was part of an unrelated PR...

[Golmote]

Why 16, by the way?

[brentertz]

60 frames per second (fps) is the target for smooth animations

1s or 1000ms / 60fps = ~16ms

[zeitgeist87]

You could also do it with a one-liner:

(requestAnimationFrame || setTimeout)(_.highlightAll, 0);

But it would be a bit of a hack and your solution is probably better.

[Golmote]

Just noticed you created the PR against the master branch. We currently work on gh-pages. The master is only updated on new release.
I merged this PR manually.