Merge pull request #262 from wwayne/nodelist-convert

Fix nodelist converte in safari

Author: wwayne

src/index.js

@@ -15,8 +15,7 @@ import getEffect from './decorators/getEffect'
 import getPosition from './utils/getPosition'
 import getTipContent from './utils/getTipContent'
 import { parseAria } from './utils/aria'
-
-import createArrayFromMixed from 'fbjs/lib/createArrayFromMixed'
+import nodeListToArray from './utils/nodeListToArray'
 
 /* CSS */
 import cssStyle from './style'
@@ -148,15 +147,13 @@ class ReactTooltip extends Component {
    */
   getTargetArray (id) {
     let targetArray
-
     if (!id) {
       targetArray = document.querySelectorAll('[data-tip]:not([data-for])')
     } else {
       targetArray = document.querySelectorAll(`[data-tip][data-for="${id}"]`)
     }
-
     // targetArray is a NodeList, convert it to a real array
-    return createArrayFromMixed(targetArray)
+    return nodeListToArray(targetArray)
   }
 
   /**

src/utils/nodeListToArray.js

@@ -0,0 +1,13 @@
+/**
+ * Convert nodelist to array
+ * @see https://github.com/facebook/fbjs/blob/e66ba20ad5be433eb54423f2b097d829324d9de6/packages/fbjs/src/core/createArrayFromMixed.js#L24
+ * NodeLists are functions in Safari
+ */
+
+export default function (nodeList) {
+  const length = nodeList.length
+  if (nodeList.hasOwnProperty) {
+    return Array.prototype.slice.call(nodeList)
+  }
+  return new Array(length).fill().map(index => nodeList[index])
+}