Merge pull request #364 from madeinfree/delay-search-issue

urgent to fixed the delay search bug.
AllenFang · Mar 28, 2016 · d601337 · d601337
2 parents 0928909 + 6df046d
commit d601337
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 8 deletions.
diff --git a/examples/js/manipulation/debounce-search-table.js b/examples/js/manipulation/debounce-search-table.js
@@ -24,7 +24,7 @@ addProducts(5);
 export default class DebounceSearchTable extends React.Component {
   render() {
     const options = {
-      searchDelayTime: 500
+      searchDelayTime: 3000
     };
 
     return (

diff --git a/src/toolbar/ToolBar.js b/src/toolbar/ToolBar.js
@@ -20,6 +20,15 @@ class ToolBar extends Component {
     };
   }
 
+  componentWillMount() {
+    const delay = this.props.searchDelayTime ? this.props.searchDelayTime : 0;
+    this.debounceCallback = this.handleDebounce(() => {
+      this.props.onSearch(this.refs.seachInput.value);
+    },
+      delay
+    );
+  }
+
   componentWillUnmount() {
     this.clearTimeout();
   }
@@ -156,13 +165,9 @@ class ToolBar extends Component {
     };
   }
 
-  handleKeyUp = () => {
-    const delay = this.props.searchDelayTime ? this.props.searchDelayTime : 0;
-    this.handleDebounce(() => {
-      this.props.onSearch(this.refs.seachInput.value);
-    },
-      delay
-    )();
+  handleKeyUp = (event) => {
+    event.persist();
+    this.debounceCallback(event);
   }
 
   handleExportCSV = () => {