Skip to content

Commit 832163e

Browse files
committed
Revert "feat: support dangerouslySetInnerHTML on Fragments"
This reverts commit 70356f1.
1 parent 180565b commit 832163e

3 files changed

Lines changed: 0 additions & 61 deletions

File tree

.changeset/neat-books-remain.md

Lines changed: 0 additions & 5 deletions
This file was deleted.

src/index.js

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -368,9 +368,6 @@ function _renderToString(
368368
// Fragments are the least used components of core that's why
369369
// branching here for comments has the least effect on perf.
370370
return '<!--' + encodeEntities(props.UNSTABLE_comment) + '-->';
371-
} else if (props.dangerouslySetInnerHTML) {
372-
let html = props.dangerouslySetInnerHTML.__html || '';
373-
return '<!--$h-->' + html + '<!--/$h-->';
374371
}
375372

376373
rendered = props.children;

test/render.test.jsx

Lines changed: 0 additions & 53 deletions
Original file line numberDiff line numberDiff line change
@@ -853,59 +853,6 @@ describe('render', () => {
853853
);
854854
expect(rendered).to.equal('<div>foo</div>');
855855
});
856-
857-
it('should support dangerouslySetInnerHTML on Fragments', () => {
858-
let rendered = render(
859-
<div>
860-
foo
861-
<Fragment dangerouslySetInnerHTML={{ __html: 'bar' }} />
862-
baz
863-
</div>
864-
);
865-
expect(rendered).to.equal('<div>foo<!--$h-->bar<!--/$h-->baz</div>');
866-
});
867-
868-
it('should ignore children on Fragments with dangerouslySetInnerHTML', () => {
869-
let rendered = render(
870-
<div>
871-
foo
872-
<Fragment dangerouslySetInnerHTML={{ __html: 'bar' }}>
873-
<p>ignored</p>
874-
</Fragment>
875-
baz
876-
</div>
877-
);
878-
expect(rendered).to.equal('<div>foo<!--$h-->bar<!--/$h-->baz</div>');
879-
});
880-
881-
it('should support falsy dangerouslySetInnerHTML on Fragments', () => {
882-
let rendered = render(
883-
<div>
884-
foo
885-
<Fragment dangerouslySetInnerHTML={{ __html: null }} />
886-
baz
887-
</div>
888-
);
889-
expect(rendered).to.equal('<div>foo<!--$h--><!--/$h-->baz</div>');
890-
891-
rendered = render(
892-
<div>
893-
foo
894-
<Fragment dangerouslySetInnerHTML={{ __html: undefined }} />
895-
baz
896-
</div>
897-
);
898-
expect(rendered).to.equal('<div>foo<!--$h--><!--/$h-->baz</div>');
899-
900-
rendered = render(
901-
<div>
902-
foo
903-
<Fragment dangerouslySetInnerHTML={{ __html: false }} />
904-
baz
905-
</div>
906-
);
907-
expect(rendered).to.equal('<div>foo<!--$h--><!--/$h-->baz</div>');
908-
});
909856
});
910857

911858
describe('className / class massaging', () => {

0 commit comments

Comments
 (0)