Merge pull request #14 from Microsoft/bugfix/render-content-fails-optimized

Fix tree shaking removing code needed for handling `ng-content` in production builds

Author: bengry

libs/core/src/lib/renderer/registry.ts

@@ -3,9 +3,6 @@
 
 import * as React from 'react';
 
-import { Disguise } from './components/Disguise';
-import { ReactContent } from './react-content';
-
 export type ComponentResolver = () => React.ReactType;
 
 const elementMap = new Map<string, { resolver: ComponentResolver }>();
@@ -49,6 +46,3 @@ export function getComponentClass(elementName: string): React.ReactType {
     throw new TypeError(`Could not load component for: ${elementName}.${e}`);
   }
 }
-
-registerElement('ReactContent', () => ReactContent);
-registerElement('Disguise', () => Disguise);

libs/core/src/lib/renderer/renderer.ts

@@ -4,7 +4,10 @@
 import { Injectable, Renderer2, RendererStyleFlags2, RendererType2 } from '@angular/core';
 import { EventManager, ɵDomRendererFactory2, ɵDomSharedStylesHost } from '@angular/platform-browser';
 import { StringMap } from '../declarations/StringMap';
+import { Disguise } from './components/Disguise';
+import { ReactContent } from './react-content';
 import { isReactNode, ReactNode } from './react-node';
+import { registerElement } from './registry';
 
 const DEBUG = false;
 
@@ -79,7 +82,11 @@ export class ReactRenderer implements Renderer2 {
     },
   };
 
-  constructor(public readonly rootRenderer: AngularReactRendererFactory) {}
+  constructor(public readonly rootRenderer: AngularReactRendererFactory) {
+    // These two elements are essential for the whole experience to be smooth for the user - register them from the get-go.
+    registerElement('ReactContent', () => ReactContent);
+    registerElement('Disguise', () => Disguise);
+  }
 
   destroy(): void {}