feat(react-i18n): allow react element in HtmlTrans element prop

BedrockStreaming · Dec 18, 2018 · 1737abb · 1737abb
1 parent 654b030
commit 1737abb
Show file tree

Hide file tree

Showing 3 changed files with 18 additions and 1 deletion.
diff --git a/...ages/react-i18n/src/components/__tests__/__snapshots__/i18nElement.component.spec.js.snap b/...ages/react-i18n/src/components/__tests__/__snapshots__/i18nElement.component.spec.js.snap
@@ -19,3 +19,13 @@ exports[`i18n.renderProps should return html translation in a div 1`] = `
   }
 />
 `;
+
+exports[`i18n.renderProps should return html translation in a react element 1`] = `
+<Dummy
+  dangerouslySetInnerHTML={
+    Object {
+      "__html": "<div>foo.bar</div>",
+    }
+  }
+/>
+`;
diff --git a/packages/react-i18n/src/components/__tests__/i18nElement.component.spec.js b/packages/react-i18n/src/components/__tests__/i18nElement.component.spec.js
@@ -20,4 +20,11 @@ describe('i18n.renderProps', () => {
 
     expect(wrapper.shallow()).toMatchSnapshot();
   });
+
+  it('should return html translation in a react element', () => {
+    const Dummy = props => <dummy {...props} />;
+    const wrapper = getWrapper({ element: Dummy });
+
+    expect(wrapper.shallow()).toMatchSnapshot();
+  });
 });
diff --git a/packages/react-i18n/src/components/i18nElement.component.js b/packages/react-i18n/src/components/i18nElement.component.js
@@ -15,7 +15,7 @@ HtmlTrans.defaultProps = {
 };
 
 HtmlTrans.propTypes = {
-  element: PropTypes.string,
+  element: PropTypes.oneOfType([PropTypes.string, PropTypes.element]),
   i18nKey: PropTypes.string.isRequired,
   data: PropTypes.object,
   number: PropTypes.number,