Remove empty string from React DOM booleans/enums

This is a design choice documented in facebook#6727 (comment)
villesau · May 7, 2019 · 694e943 · 694e943
1 parent aedba0d
commit 694e943
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 6 deletions.
diff --git a/lib/react-dom.js b/lib/react-dom.js
@@ -620,7 +620,7 @@ type ReactDOM$HTMLElementProps = ReactDOM$ElementProps & {
   spellCheck?: ReactDOM$BooleanishString,
   suppressContentEditableWarning?: boolean,
   title?: string,
-  translate?: $TODO$CaseInsensitive<"" | "yes" | "no">
+  translate?: $TODO$CaseInsensitive<"yes" | "no">
 };
 
 type ReactDOM$SVGElementProps = ReactDOM$ElementProps & {
@@ -642,12 +642,10 @@ type ReactDOM$SVGElementProps = ReactDOM$ElementProps & {
 };
 
 type ReactDOM$BooleanishString =
-  | ""
   | $TODO$CaseInsensitive<"true" | "false">
   | boolean;
 
-type ReactDOM$Boolean<AttributeName: string = ""> =
-  | ""
+type ReactDOM$Boolean<AttributeName: string> =
   | $TODO$CaseInsensitive<AttributeName>
   | boolean;
 

diff --git a/tests/react_dom/props_boolean.js b/tests/react_dom/props_boolean.js
@@ -7,7 +7,7 @@ import React from "react";
 <span hidden={null} />; // Error: Expected DOM boolean
 <span hidden="true" />; // Error: Expected DOM boolean
 <span hidden="false" />; // Error: Expected DOM boolean
-<span hidden="" />; // OK
+<span hidden="" />; // Error: Expected DOM boolean
 <span hidden="yes" />; // Error: Expected DOM boolean
 <span hidden="hidden" />; // OK
 

diff --git a/tests/react_dom/props_booleanish_string.js b/tests/react_dom/props_booleanish_string.js
@@ -7,6 +7,6 @@ import React from "react";
 <div contentEditable={null} />; // Error: Expected booleanish string
 <div contentEditable="true" />; // OK
 <div contentEditable="false" />; // OK
-<div contentEditable="" />; // OK
+<div contentEditable="" />; // Error: Expected booleanish string
 <div contentEditable="yes" />; // Error: Expected booleanish string
 <div contentEditable="contentEditable" />; // Error: Expected booleanish string