Fix NodeList to Array convertion

[huumanoid]

Related to issue #256

Hope this PR would fix it once and for all.

Thanks @LeoIannacone for his diagnostics and issue report.
Thanks @tasiek for his extremely helpful diagnostics and feedback about my patch.
Thanks to @gauthierj for his important feedback, I'm glad that this approach OK for him.

Thanks everyone who has faced this issue and test this fix to make sure problem it gone.

npm install 'huumanoid/react-tooltip#fix-undefined-is-not-a-function'

Discuss:
Should this library directly depend on fbjs? react-dom is already depends on fbjs.

[wwayne]

@huumanoid Thanks a lot :), but the CI failed for Error: Cannot find module 'fbjs/lib/createArrayFromMixed', do we need to add fbjs as dependencies then?

[huumanoid]

Thanks for your response!
The situation is quite hard.
As mentioned at https://github.com/facebook/fbjs, it's not reliably to depend on fbjs. They don't follow semver strictly.
Both legacy and modern react are depend on fbjs.

I see three possible solutions.

1. Just add fbjs to dependencies. Con: it may break at any time.
2. Add strict dependency on the exact current version. Pro: breakage is less likely, however, is still possible, I guess. Con: when they update fbjs, we will have at least two copies of fbjs in node_modules (one for react-tooltip and one for react-dom. Who cares?
3. Just copy&paste required code to this project. Pros: reliably and lightweight. Cons: Copypasting of the already available code? Err... Moreover, are licenses compatible?

I can't do such a decision by myself.
Copy&paste is easy to do, required code has zero dependencies.
I trust in your wisdom. What should be do in this situation?

[wwayne]

@huumanoid I prefer to check their solution first and write it by ourselves (well it's like copy paste, haha). I can do it in these 2 days.

Thanks again, your comment is really helped.