Merge pull request #13 from smacker/empty-child

Fix <div>{''}</div>
algolia · Dec 10, 2015 · ffbfc45 · ffbfc45
2 parents b0dad59 + d837683
commit ffbfc45
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/index-test.js b/index-test.js
@@ -417,4 +417,10 @@ describe(`reactElementToJSXString(ReactElement)`, () => {
   123
 </div>`);
   });
+
+  it(`reactElementToJSXString(<div>{''}</div>)`, () => {
+    expect(
+      reactElementToJSXString(<div>{''}</div>)
+    ).toEqual(reactElementToJSXString(<div/>));
+  });
 });
diff --git a/index.js b/index.js
@@ -190,5 +190,5 @@ function noChildren(propName) {
 }
 
 function onlyMeaningfulChildren(children) {
-  return children !== true && children !== false && children !== null;
+  return children !== true && children !== false && children !== null && children !== '';
 }