Merge pull request #1365 from shakti97/trim-query

catarak · web-flow · commit 5b63410812e9 · 2020-04-14T19:01:19.000-04:00
fix(trim query before search)
diff --git a/client/modules/IDE/components/Searchbar/Searchbar.jsx b/client/modules/IDE/components/Searchbar/Searchbar.jsx
@@ -26,12 +26,13 @@ class Searchbar extends React.Component {
 
   handleSearchEnter = (e) => {
     if (e.key === 'Enter') {
-      this.props.setSearchTerm(this.state.searchValue);
+      this.searchChange();
     }
   }
 
-  searchChange = (value) => {
-    this.props.setSearchTerm(this.state.searchValue);
+  searchChange = () => {
+    if (this.state.searchValue.trim().length === 0) return;
+    this.props.setSearchTerm(this.state.searchValue.trim());
   };
 
   handleSearchChange = (e) => {

Original file line number	Diff line number	Diff line change
`@@ -26,12 +26,13 @@ class Searchbar extends React.Component {`
`26`	`26`
`27`	`27`	`handleSearchEnter = (e) => {`
`28`	`28`	`if (e.key === 'Enter') {`
`29`		`- this.props.setSearchTerm(this.state.searchValue);`
	`29`	`+ this.searchChange();`
`30`	`30`	`}`
`31`	`31`	`}`
`32`	`32`
`33`		`- searchChange = (value) => {`
`34`		`- this.props.setSearchTerm(this.state.searchValue);`
	`33`	`+ searchChange = () => {`
	`34`	`+ if (this.state.searchValue.trim().length === 0) return;`
	`35`	`+ this.props.setSearchTerm(this.state.searchValue.trim());`
`35`	`36`	`};`
`36`	`37`
`37`	`38`	`handleSearchChange = (e) => {`