From 7a2d0b365e0fd901b3b15b2a154a99b76f78d9ef Mon Sep 17 00:00:00 2001
From: Timmy Huang <thuang@chanzuckerberg.com>
Date: Tue, 23 Jul 2019 14:25:35 -0700
Subject: [PATCH] fix(src/index.js): Overwrite `delayHide` on scroll

Add new param `options` to `hideTooltip()` and update `addScrollListener()`
to set `options` to `{ isScroll: true }`, so `hideTooltip()` can set
`delayHide` to `0` if `isScroll` is `true`

fix #474
---
 src/index.js | 18 ++++++++++++++----
 1 file changed, 14 insertions(+), 4 deletions(-)

diff --git a/src/index.js b/src/index.js
index 746ae4876..be929e8a1 100644
--- a/src/index.js
+++ b/src/index.js
@@ -100,6 +100,7 @@ class ReactTooltip extends React.Component {
       'showTooltip',
       'updateTooltip',
       'hideTooltip',
+      'hideTooltipOnScroll',
       'getTooltipContent',
       'globalRebuild',
       'globalShow',
@@ -425,8 +426,10 @@ class ReactTooltip extends React.Component {
   /**
    * When mouse leave, hide tooltip
    */
-  hideTooltip (e, hasTarget) {
-    const {delayHide, disable} = this.state
+  hideTooltip (e, hasTarget, options = { isScroll: false }) {
+    const {disable} = this.state
+    const {isScroll} = options
+    const delayHide = isScroll ? 0 : this.state.delayHide
     const {afterHide} = this.props
     const placeholder = this.getTooltipContent()
     if (!this.mount) return
@@ -463,17 +466,24 @@ class ReactTooltip extends React.Component {
     }
   }
 
+  /**
+   * When scroll, hide tooltip
+   */
+  hideTooltipOnScroll (event, hasTarget) {
+    this.hideTooltip(event, hasTarget, { isScroll: true })
+  }
+
   /**
    * Add scroll event listener when tooltip show
    * automatically hide the tooltip when scrolling
    */
   addScrollListener (currentTarget) {
     const isCaptureMode = this.isCapture(currentTarget)
-    window.addEventListener('scroll', this.hideTooltip, isCaptureMode)
+    window.addEventListener('scroll', this.hideTooltipOnScroll, isCaptureMode)
   }
 
   removeScrollListener () {
-    window.removeEventListener('scroll', this.hideTooltip)
+    window.removeEventListener('scroll', this.hideTooltipOnScroll)
   }
 
   // Calculation the position