add tests for nested tooltips

Author: mohsinulhaq

.size-snapshot.json

@@ -1,25 +1,25 @@
 {
   "dist/index.js": {
-    "bundled": 7461,
-    "minified": 7461,
-    "gzipped": 2211
+    "bundled": 7333,
+    "minified": 7333,
+    "gzipped": 2207
   },
   "lib/cjs/index.js": {
-    "bundled": 14080,
-    "minified": 7781,
-    "gzipped": 2143
+    "bundled": 13954,
+    "minified": 7653,
+    "gzipped": 2137
   },
   "lib/esm/index.js": {
-    "bundled": 14028,
-    "minified": 7748,
-    "gzipped": 2136,
+    "bundled": 13902,
+    "minified": 7620,
+    "gzipped": 2132,
     "treeshaked": {
       "rollup": {
-        "code": 7187,
+        "code": 7059,
         "import_statements": 331
       },
       "webpack": {
-        "code": 8421
+        "code": 8293
       }
     }
   }

.travis.yml

@@ -1,14 +1,14 @@
-language: node_js
 dist: xenial
+language: node_js
 node_js:
-  - 'stable'
+  - '--lts'
 cache: yarn
 branches:
   except:
     - gh-pages
 script:
   - yarn lint
-  - yarn test --coverage
+  - yarn test --ci --coverage
   - yarn build
   - yarn docs:build && cp .docz/dist/index.html .docz/dist/404.html
 deploy:

docs/examples/BasicTooltipTrigger.js

@@ -12,14 +12,14 @@ const BasicTooltipTrigger = memo(({tooltip, children, hideArrow, ...props}) => (
       arrowRef,
       placement
     }) => (
-      <span
+      <div
         {...getTooltipProps({
           ref: tooltipRef,
           className: 'tooltip-container'
         })}
       >
         {!hideArrow && (
-          <span
+          <div
             {...getArrowProps({
               ref: arrowRef,
               'data-placement': placement,
@@ -28,7 +28,7 @@ const BasicTooltipTrigger = memo(({tooltip, children, hideArrow, ...props}) => (
           />
         )}
         {tooltip}
-      </span>
+      </div>
     )}
   >
     {({getTriggerProps, triggerRef}) => (

package.json

@@ -45,30 +45,30 @@
   "author": "Mohsin Ul Haq <[email protected]>",
   "license": "MIT",
   "peerDependencies": {
-    "react": "^16.6.3",
-    "react-dom": "^16.6.3"
+    "react": "^16.7.0",
+    "react-dom": "^16.7.0"
   },
   "dependencies": {
     "@babel/runtime": "^7.2.0",
     "react-popper": "^1.3.2"
   },
   "devDependencies": {
-    "@babel/cli": "^7.2.0",
+    "@babel/cli": "^7.2.3",
     "@babel/core": "^7.2.2",
-    "@babel/plugin-proposal-class-properties": "^7.2.1",
+    "@babel/plugin-proposal-class-properties": "^7.2.3",
     "@babel/plugin-transform-runtime": "^7.2.0",
-    "@babel/preset-env": "^7.2.0",
+    "@babel/preset-env": "^7.2.3",
     "@babel/preset-react": "^7.0.0",
     "@types/jest": "^23.3.10",
-    "@types/react": "^16.7.17",
+    "@types/react": "^16.7.18",
     "@types/react-dom": "^16.0.11",
-    "babel-core": "^7.0.0-bridge",
+    "babel-core": "^7.0.0-bridge.0",
     "babel-eslint": "^10.0.1",
     "babel-jest": "^23.6.0",
     "babel-plugin-transform-react-remove-prop-types": "^0.4.21",
-    "docz": "^0.13.2",
+    "docz": "^0.13.5",
     "docz-plugin-css": "^0.11.0",
-    "docz-theme-default": "^0.13.1",
+    "docz-theme-default": "^0.13.5",
     "eslint": "^5.10.0",
     "eslint-config-prettier": "^3.3.0",
     "eslint-plugin-jest": "^22.1.2",
@@ -80,11 +80,11 @@
     "pre-commit": "^1.2.2",
     "prettier": "^1.15.3",
     "prop-types": "^15.6.2",
-    "react": "^16.6.3",
-    "react-dom": "^16.6.3",
-    "react-testing-library": "^5.4.0",
+    "react": "^16.7.0",
+    "react-dom": "^16.7.0",
+    "react-testing-library": "^5.4.2",
     "rimraf": "^2.6.2",
-    "rollup": "^0.68.0",
+    "rollup": "^0.68.1",
     "rollup-plugin-babel": "^4.1.0",
     "rollup-plugin-commonjs": "^9.2.0",
     "rollup-plugin-node-resolve": "^4.0.0",

src/Tooltip.js

@@ -86,8 +86,8 @@ export default class Tooltip extends Component {
         removeParentOutsideClickHandler,
         removeParentOutsideRightClickHandler
       } = this.props;
-      document.addEventListener('click', this._handleOutsideClick);
-      document.addEventListener('contextmenu', this._handleOutsideRightClick);
+      this._addOutsideClickHandler();
+      this._addOutsideRightClickHandler();
       removeParentOutsideClickHandler && removeParentOutsideClickHandler();
       removeParentOutsideRightClickHandler &&
         removeParentOutsideRightClickHandler();
@@ -103,11 +103,8 @@ export default class Tooltip extends Component {
         addParentOutsideClickHandler,
         addParentOutsideRightClickHandler
       } = this.props;
-      document.removeEventListener('click', this._handleOutsideClick);
-      document.removeEventListener(
-        'contextmenu',
-        this._handleOutsideRightClick
-      );
+      this._removeOutsideClickHandler();
+      this._removeOutsideRightClickHandler();
       this._handleOutsideClick = undefined;
       this._handleOutsideRightClick = undefined;
       addParentOutsideClickHandler && addParentOutsideClickHandler();