feat(progress-spinner): add injection token for configuring the diameter and stroke globally

[crisbeto]

Adds the MAT_PROGRESS_SPINNER_DEFAULT_OPTIONS injection token that can be used to globally configure the diameter and stroke (and any other options we decide to add) for all progress spinners.

Fixes #11490.

[CharlBest]

Thank you for this. I think it's worth putting in the "strokeWidth" as well to set your own default globally if it's not to much extra work. Probably not part of this PR just thought I'd say something. :) Thanks again.

[crisbeto]

Done.

[jelbourn]

LGTM

[josephperrott]

Should we have our own defaults value in a instance of MatProgressSpinnerDefaultOptions?

Then we can just Object.assign it, rather than holding our defaults inline in location.

In the file above the class:

export interface MatProgressSpinnerDefaultOptions {
  /** Diameter of the spinner. */
  diameter?: number;
  /** Width of the spinner's stroke. */
  strokeWidth?: number;
}

const BASE_OPTIONS  = {
  diameter: 100,
  strokeWidth: 10
};

In the class:

export class MatProgressSpinner 
  ...
  private defaults: MatProgressSpinnerDefaultOptions
  ...
  constructor(
  ...
  @Inject(MAT_PROGRESS_SPINNER_DEFAULT_OPTIONS) defaults: MatProgressSpinnerDefaultOptions) {
    this.defaults = Object.assign(BASE_OPTIONS, defaults);
  ...
  }
...

[crisbeto]

Our defaults are provided as the MAT_PROGRESS_SPINNER_DEFAULT_OPTIONS token. I kept the inline defaults until we can make the defaults a required parameter.

[josephperrott]

My concern here is that since the interface for MatProgressSpinnerDefaultOptions has all optional properties, that someone who provides a MatProgressSpinnerDefaultOptions, would not provide everything thats needed.

[angular-automatic-lock-bot]

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

_{This action has been performed automatically by a bot.}