Skip to content

Commit 12c7663

Browse files
authored
Merge pull request #23 from mikestead/fix/allow-render-return-false
Render empty value if render returns false
2 parents fd00b37 + 1616453 commit 12c7663

File tree

2 files changed

+6
-1
lines changed

2 files changed

+6
-1
lines changed

src/index.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@ export default function renderToString(vnode, context, opts, inner, isSvgMode) {
5858
let pretty = opts.pretty,
5959
indentChar = typeof pretty==='string' ? pretty : '\t';
6060

61-
if (vnode==null) {
61+
if (vnode==null || vnode===false) {
6262
return '';
6363
}
6464

test/jsx.js

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -125,17 +125,22 @@ describe('jsx', () => {
125125

126126
it('should render empty resolved children identically to no children', () => {
127127
const Empty = () => null;
128+
const False = () => false;
128129
expect(renderJsx(
129130
<div>
130131
<a />
131132
<b>{null}</b>
132133
<c><Empty /></c>
134+
<d>{false}</d>
135+
<e><False /></e>
133136
</div>
134137
)).to.equal(dedent`
135138
<div>
136139
<a></a>
137140
<b></b>
138141
<c></c>
142+
<d></d>
143+
<e></e>
139144
</div>
140145
`);
141146
});

0 commit comments

Comments
 (0)