fix(progress-spinner): unable to change mode on spinner directive

Currently we have the `mat-spinner` directive which is a shortcut to a `mat-progress-spinner` with `mode="indeterminate"`. Since the spinner inherits all of the inputs from the progress spinner, there's nothing stoping people from changing the mode back to `determinate`, however the element will look half-broken because the host bindings assume that the mode won't change. These changes update the host bindings to allow switching between modes.

Fixes #14511.

Author: crisbeto

src/material/progress-spinner/progress-spinner-module.ts

@@ -8,19 +8,17 @@
 import {NgModule} from '@angular/core';
 import {CommonModule} from '@angular/common';
 import {MatCommonModule} from '@angular/material/core';
-import {MatProgressSpinner, MatSpinner} from './progress-spinner';
+import {MatProgressSpinner} from './progress-spinner';
 
 
 @NgModule({
   imports: [MatCommonModule, CommonModule],
   exports: [
     MatProgressSpinner,
-    MatSpinner,
     MatCommonModule
   ],
   declarations: [
     MatProgressSpinner,
-    MatSpinner
   ],
 })
 class MatProgressSpinnerModule {}

src/material/progress-spinner/progress-spinner.spec.ts

@@ -27,6 +27,7 @@ describe('MatProgressSpinner', () => {
         ProgressSpinnerWithStringValues,
         IndeterminateSpinnerInShadowDom,
         IndeterminateSpinnerInShadowDomWithNgIf,
+        SpinnerWithMode,
       ],
     }).compileComponents();
   }));
@@ -428,6 +429,14 @@ describe('MatProgressSpinner', () => {
       expect(shadowRoot.querySelector('style[mat-spinner-animation="27"]')).toBeTruthy();
     });
 
+  it('should be able to change the mode on a mat-spinner', () => {
+    const fixture = TestBed.createComponent(SpinnerWithMode);
+    fixture.detectChanges();
+
+    const progressElement = fixture.debugElement.query(By.css('mat-spinner')).nativeElement;
+    expect(progressElement.getAttribute('mode')).toBe('determinate');
+  });
+
 });
 
 
@@ -494,3 +503,7 @@ class IndeterminateSpinnerInShadowDomWithNgIf {
   diameter: number;
 }
 
+
+@Component({template: '<mat-spinner mode="determinate"></mat-spinner>'})
+class SpinnerWithMode { }
+

src/material/progress-spinner/progress-spinner.ts

@@ -105,11 +105,12 @@ const INDETERMINATE_ANIMATION_TEMPLATE = `
  */
 @Component({
   moduleId: module.id,
-  selector: 'mat-progress-spinner',
+  selector: 'mat-progress-spinner, mat-spinner',
   exportAs: 'matProgressSpinner',
   host: {
     'role': 'progressbar',
-    'class': 'mat-progress-spinner',
+    // `mat-spinner` is here for backward compatibility.
+    'class': 'mat-progress-spinner mat-spinner',
     '[class._mat-animation-noopable]': `_noopAnimations`,
     '[style.width.px]': 'diameter',
     '[style.height.px]': 'diameter',
@@ -202,6 +203,10 @@ export class MatProgressSpinner extends _MatProgressSpinnerMixinBase implements
     this._noopAnimations = animationMode === 'NoopAnimations' &&
         (!!defaults && !defaults._forceAnimations);
 
+    if (_elementRef.nativeElement.nodeName.toLowerCase() === 'mat-spinner') {
+      this.mode = 'indeterminate';
+    }
+
     if (defaults) {
       if (defaults.diameter) {
         this.diameter = defaults.diameter;
@@ -297,42 +302,6 @@ export class MatProgressSpinner extends _MatProgressSpinnerMixinBase implements
   }
 }
 
-
-/**
- * `<mat-spinner>` component.
- *
- * This is a component definition to be used as a convenience reference to create an
- * indeterminate `<mat-progress-spinner>` instance.
- */
-@Component({
-  moduleId: module.id,
-  selector: 'mat-spinner',
-  host: {
-    'role': 'progressbar',
-    'mode': 'indeterminate',
-    'class': 'mat-spinner mat-progress-spinner',
-    '[class._mat-animation-noopable]': `_noopAnimations`,
-    '[style.width.px]': 'diameter',
-    '[style.height.px]': 'diameter',
-  },
-  inputs: ['color'],
-  templateUrl: 'progress-spinner.html',
-  styleUrls: ['progress-spinner.css'],
-  changeDetection: ChangeDetectionStrategy.OnPush,
-  encapsulation: ViewEncapsulation.None,
-})
-export class MatSpinner extends MatProgressSpinner {
-  constructor(elementRef: ElementRef<HTMLElement>, platform: Platform,
-              @Optional() @Inject(DOCUMENT) document: any,
-              @Optional() @Inject(ANIMATION_MODULE_TYPE) animationMode: string,
-              @Inject(MAT_PROGRESS_SPINNER_DEFAULT_OPTIONS)
-                  defaults?: MatProgressSpinnerDefaultOptions) {
-    super(elementRef, platform, document, animationMode, defaults);
-    this.mode = 'indeterminate';
-  }
-}
-
-
 /** Gets the shadow root of an element, if supported and the element is inside the Shadow DOM. */
 export function _getShadowRoot(element: HTMLElement, _document: Document): Node | null {
   // TODO(crisbeto): see whether we should move this into the CDK

src/material/progress-spinner/public-api.ts

@@ -6,12 +6,22 @@
  * found in the LICENSE file at https://angular.io/license
  */
 
+import {MatProgressSpinner} from './progress-spinner';
+
 export * from './progress-spinner-module';
 export {
   MatProgressSpinner,
-  MatSpinner,
   MAT_PROGRESS_SPINNER_DEFAULT_OPTIONS,
   ProgressSpinnerMode,
   MatProgressSpinnerDefaultOptions,
   MAT_PROGRESS_SPINNER_DEFAULT_OPTIONS_FACTORY,
 } from './progress-spinner';
+
+
+/**
+ * @deprecated Import `MatProgressSpinner` instead. Note that the
+ *    `mat-spinner` selector isn't deprecated.
+ * @breaking-change 8.0.0
+ */
+// tslint:disable-next-line:variable-name
+export const MatSpinner = MatProgressSpinner;

tools/public_api_guard/material/progress-spinner.d.ts

@@ -24,8 +24,6 @@ export interface MatProgressSpinnerDefaultOptions {
     strokeWidth?: number;
 }
 
-export declare class MatSpinner extends MatProgressSpinner {
-    constructor(elementRef: ElementRef<HTMLElement>, platform: Platform, document: any, animationMode: string, defaults?: MatProgressSpinnerDefaultOptions);
-}
+export declare const MatSpinner: typeof MatProgressSpinner;
 
 export declare type ProgressSpinnerMode = 'determinate' | 'indeterminate';