Merge pull request algolia#949 from algolia/fix/searchBox/do-not-upda…

…te-when-focused

fix(searchBox): do not update the input when focused

dev/app.js

@@ -349,4 +349,3 @@ search.once('render', function() {
 });
 
 search.start();
-

src/widgets/search-box/__tests__/search-box-test.js

@@ -357,10 +357,21 @@ describe('searchBox()', () => {
     container.value = 'initial';
     widget = searchBox({container});
     widget.init({state, helper, onHistoryChange});
+    container.blur();
     widget.render({helper: {state: {query: 'new value'}}});
     expect(container.value).toBe('new value');
   });
 
+  it('does not update the input value when focused', () => {
+    container = document.body.appendChild(document.createElement('input'));
+    container.value = 'initial';
+    container.focus();
+    widget = searchBox({container});
+    widget.init({state, helper, onHistoryChange});
+    widget.render({helper: {state: {query: 'new value'}}});
+    expect(container.value).toBe('initial');
+  });
+
   context('autofocus', () => {
     beforeEach(() => {
       container = document.body.appendChild(document.createElement('input'));
@@ -440,4 +451,3 @@ describe('searchBox()', () => {
     });
   });
 });
-

src/widgets/search-box/search-box.js

@@ -197,6 +197,11 @@ function searchBox({
     render({helper}) {
       // updating the query from the outside using the helper
       // will fall in this case
+      // If the input is focused, we do not update it.
+      if (document.activeElement === this._input) {
+        return;
+      }
+
       if (helper.state.query !== this._input.value) {
         this._input.value = helper.state.query;
       }