Add warnings for onFocusIn and onFocusOut props

facebook · Mar 19, 2016 · 8648429 · 8648429
1 parent 67f8524
commit 8648429
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 0 deletions.
diff --git a/src/renderers/dom/shared/ReactDOMComponent.js b/src/renderers/dom/shared/ReactDOMComponent.js
@@ -187,6 +187,12 @@ function assertValidProps(component, props) {
       'those nodes are unexpectedly modified or duplicated. This is ' +
       'probably not intentional.'
     );
+    warning(
+      props.onFocusIn == null &&
+      props.onFocusOut == null,
+      'Props onFocusIn and onFocusOut are not supported. Use onFocus and ' +
+      'onBlur events instead because they are normalized across browsers.'
+    );
   }
   invariant(
     props.style == null || typeof props.style === 'object',

diff --git a/src/renderers/dom/shared/__tests__/ReactDOMComponent-test.js b/src/renderers/dom/shared/__tests__/ReactDOMComponent-test.js
@@ -1234,5 +1234,17 @@ describe('ReactDOMComponent', function() {
       expect(console.error.argsForCall.length).toBe(1);
       expect(console.error.argsForCall[0][0]).toContain('className');
     });
+
+    it('should warn about props that are no longer supported', function() {
+      spyOn(console, 'error');
+      ReactTestUtils.renderIntoDocument(<div />);
+      expect(console.error.argsForCall.length).toBe(0);
+
+      ReactTestUtils.renderIntoDocument(<div onFocusIn={() => {}} />);
+      expect(console.error.argsForCall.length).toBe(1);
+
+      ReactTestUtils.renderIntoDocument(<div onFocusOut={() => {}} />);
+      expect(console.error.argsForCall.length).toBe(2);
+    });
   });
 });