Remove inputRef.blur() call that was causing a focus trap in Safari/IE

react-dates · Jun 18, 2018 · 8416bc9 · 8416bc9
1 parent 0c7d9af
commit 8416bc9
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 19 deletions.
diff --git a/src/components/DateInput.jsx b/src/components/DateInput.jsx
@@ -110,8 +110,6 @@ class DateInput extends React.Component {
 
     if (focused && isFocused) {
       this.inputRef.focus();
-    } else {
-      this.inputRef.blur();
     }
   }
 

diff --git a/test/components/DateInput_spec.jsx b/test/components/DateInput_spec.jsx
@@ -229,12 +229,10 @@ describe('DateInput', () => {
 
     describe('focus/isFocused', () => {
       const el = {
-        blur() {},
         focus() {},
       };
 
       beforeEach(() => {
-        sinon.spy(el, 'blur');
         sinon.spy(el, 'focus');
       });
 
@@ -252,23 +250,8 @@ describe('DateInput', () => {
 
         wrapper.setProps({ focused: true, isFocused: true });
 
-        expect(el.blur).to.have.property('callCount', 0);
         expect(el.focus).to.have.property('callCount', 1);
       });
-
-      it('blurs when becoming unfocused', () => {
-        const wrapper = shallow(
-          <DateInput id="date" focused isFocused />,
-          { disableLifecycleMethods: false },
-        ).dive();
-
-        wrapper.instance().inputRef = el;
-
-        wrapper.setProps({ focused: false, isFocused: false });
-
-        expect(el.blur).to.have.property('callCount', 1);
-        expect(el.focus).to.have.property('callCount', 0);
-      });
     });
 
     /*