use onInput instead of onChange

Financial-Times · Dec 18, 2018 · 259c3dd · 259c3dd
1 parent d14273d
commit 259c3dd
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/__tests__/__snapshots__/snapshots.test.js.snap b/__tests__/__snapshots__/snapshots.test.js.snap
@@ -5555,9 +5555,9 @@ exports[`@financial-times/x-topic-search renders a default Topic Search Bar x-to
       data-trackable="topic-search"
       id="topic-search-input"
       onBlur={[Function]}
-      onChange={[Function]}
       onClick={[Function]}
       onFocus={[Function]}
+      onInput={[Function]}
       placeholder="Search and add topics"
       type="search"
     />

diff --git a/components/x-topic-search/__tests__/x-topic-search.test.jsx b/components/x-topic-search/__tests__/x-topic-search.test.jsx
@@ -31,7 +31,7 @@ describe('x-topic-search', () => {
 
 			const subject = mount(<TopicSearch {...props}/>);
 			const input = subject.find('input');
-			await input.prop('onChange')({ target: { value: searchTerm }});
+			await input.prop('onInput')({ target: { value: searchTerm }});
 
 			expect(subject.render()).toMatchSnapshot();
 		})
@@ -43,7 +43,7 @@ describe('x-topic-search', () => {
 
 			const subject = mount(<TopicSearch {...props}/>);
 			const input = subject.find('input');
-			await input.prop('onChange')({ target: { value: searchTerm }});
+			await input.prop('onInput')({ target: { value: searchTerm }});
 
 			expect(subject.render()).toMatchSnapshot();
 		})
@@ -60,7 +60,7 @@ describe('x-topic-search', () => {
 
 			const subject = mount(<TopicSearch {...props}/>);
 			const input = subject.find('input');
-			await input.prop('onChange')({ target: { value: searchTerm }});
+			await input.prop('onInput')({ target: { value: searchTerm }});
 
 			expect(subject.render()).toMatchSnapshot();
 		})

diff --git a/components/x-topic-search/src/TopicSearch.jsx b/components/x-topic-search/src/TopicSearch.jsx
@@ -71,7 +71,7 @@ const TopicSearch = topicSearchActions(({ searchTerm, showResult, result, action
 				placeholder="Search and add topics"
 				className={ classNames(styles["input"]) }
 				data-trackable="topic-search"
-				onChange={ actions.checkInput }
+				onInput={ actions.checkInput }
 				onClick={ actions.selectInput }
 				onFocus={ actions.selectInput }
 				onBlur={ actions.hideResult }/>