fix: prepare for enabling Ivy (#642)

remove use of AsyncPipe with `*ngFor`
- due to angular/angular#31371
remove deprecated `@angular/http` from StackBlitz dependencies
remove out of date `.angular-cli.json` from StackBlitz templates
- it's unclear that moving to the new `angular.json` format is viable
remove unneeded polyfills from StackBlitz templates

Author: Splaktar

package.json

@@ -65,4 +65,4 @@
     "tslint": "^5.20.0",
     "typescript": "3.5.3"
   }
-}
+}

src/app/shared/stackblitz/stackblitz-writer.spec.ts

@@ -74,7 +74,7 @@ describe('StackblitzWriter', () => {
       }
       flushMicrotasks();
 
-      expect(form.elements.length).toBe(15);
+      expect(form.elements.length).toBe(14);
 
       // Should have correct tags
       expect(form.elements[0].getAttribute('name')).toBe('tags[0]');
@@ -94,12 +94,11 @@ describe('StackblitzWriter', () => {
       expect(form.elements[6].getAttribute('name')).toBe('files[index.html]');
       expect(form.elements[7].getAttribute('name')).toBe('files[styles.css]');
       expect(form.elements[8].getAttribute('name')).toBe('files[polyfills.ts]');
-      expect(form.elements[9].getAttribute('name')).toBe('files[.angular-cli.json]');
-      expect(form.elements[10].getAttribute('name')).toBe('files[main.ts]');
-      expect(form.elements[11].getAttribute('name')).toBe('files[material-module.ts]');
-      expect(form.elements[12].getAttribute('name')).toBe('files[app/test.ts]');
-      expect(form.elements[13].getAttribute('name')).toBe('files[app/test.html]');
-      expect(form.elements[14].getAttribute('name')).toBe('files[app/src/detail.ts]');
+      expect(form.elements[9].getAttribute('name')).toBe('files[main.ts]');
+      expect(form.elements[10].getAttribute('name')).toBe('files[material-module.ts]');
+      expect(form.elements[11].getAttribute('name')).toBe('files[app/test.ts]');
+      expect(form.elements[12].getAttribute('name')).toBe('files[app/test.html]');
+      expect(form.elements[13].getAttribute('name')).toBe('files[app/src/detail.ts]');
     });
   }));
 });
@@ -114,7 +113,6 @@ const TEST_URLS = [
   '/assets/stackblitz/index.html',
   '/assets/stackblitz/styles.css',
   '/assets/stackblitz/polyfills.ts',
-  '/assets/stackblitz/.angular-cli.json',
   '/assets/stackblitz/main.ts',
   '/assets/stackblitz/material-module.ts',
   '/docs-content/examples-source/test.ts',

src/app/shared/stackblitz/stackblitz-writer.ts

@@ -24,7 +24,6 @@ const TEMPLATE_FILES = [
   'index.html',
   'styles.css',
   'polyfills.ts',
-  '.angular-cli.json',
   'main.ts',
   'material-module.ts',
 ];
@@ -41,14 +40,12 @@ const dependencies = {
   '@angular/compiler': angularVersion,
   '@angular/core': angularVersion,
   '@angular/forms': angularVersion,
-  '@angular/http': angularVersion,
   '@angular/platform-browser': angularVersion,
   '@angular/platform-browser-dynamic': angularVersion,
   '@angular/router': angularVersion,
   'angular-in-memory-web-api': '~0.5.4',
   'core-js': '^2.6.9',
   'rxjs': '>=6.5.3 <7.0.0',
-  'web-animations-js': '^2.3.2',
   'zone.js': '^0.9.1',
   'hammerjs': '^2.0.8',
   'moment': '^2.24.0',

src/app/shared/version-picker/version-picker.html

@@ -3,7 +3,7 @@
   <mat-icon>arrow_drop_down</mat-icon>
 </button>
 <mat-menu #versionPicker="matMenu">
-  <button mat-menu-item *ngFor="let version of docVersions | async"
+  <button mat-menu-item *ngFor="let version of docVersions"
           (click)="onVersionChanged(version)">
     {{version.title}}
   </button>

src/app/shared/version-picker/version-picker.ts

@@ -16,11 +16,17 @@ export class VersionPicker {
   /** The currently running version of Material. */
   materialVersion = materialVersion;
   /** The possible versions of the doc site. */
-  docVersions = this.http.get<VersionInfo[]>(versionUrl);
+  docVersions: VersionInfo[];
 
-  constructor(private http: HttpClient) {}
+  constructor(private http: HttpClient) {
+    this.http.get<VersionInfo[]>(versionUrl)
+      .subscribe((versions: VersionInfo[]) => this.docVersions = versions);
+  }
 
-  /** Updates the window location if the selected version is a different version. */
+  /**
+   * Updates the window location if the selected version is a different version.
+   * @param version data for use in navigating to the version's path
+   */
   onVersionChanged(version: VersionInfo) {
     if (!version.url.startsWith(window.location.href)) {
       window.location.assign(version.url);

src/assets/stackblitz/.angular-cli.json

@@ -1,5 +1,2 @@
-import 'core-js/es6/reflect';
-import 'core-js/es7/reflect';
 import 'zone.js/dist/zone';
 import 'hammerjs';
-import 'web-animations-js';

src/assets/stackblitz/polyfills.ts

@@ -205,8 +205,8 @@
     tslib "^1.9.0"
 
 "@angular/material-examples@angular/material2-docs-content":
-  version "8.2.2-sha-fc6cb1069"
-  resolved "https://codeload.github.com/angular/material2-docs-content/tar.gz/739fdda7e3fcdc507117001552ab32b63fe7e09a"
+  version "8.2.2-sha-361d1cf80"
+  resolved "https://codeload.github.com/angular/material2-docs-content/tar.gz/050aa5718e2a81f5ee6cbafd038eecaf55c95b6b"
   dependencies:
     tslib "^1.7.1"