fix(button-toggle): underlying input not disabled when group is disabled

Fixes the underlying `input` element inside a `mat-button-toggle` not being disabled when the parent toggle group is disabled via a data binding.

Fixes #11608.

Author: crisbeto

src/lib/button-toggle/button-toggle.spec.ts

@@ -286,6 +286,17 @@ describe('MatButtonToggle without forms', () => {
       expect(groupNativeElement.getAttribute('aria-disabled')).toBe('true');
     });
 
+    it('should disable the underlying input when the group is disabled', () => {
+      const inputs = buttonToggleNativeElements.map(toggle => toggle.querySelector('input')!);
+
+      expect(inputs.every(input => input.disabled)).toBe(false);
+
+      testComponent.isGroupDisabled = true;
+      fixture.detectChanges();
+
+      expect(inputs.every(input => input.disabled)).toBe(true);
+    });
+
     it('should update the group value when one of the toggles changes', () => {
       expect(groupInstance.value).toBeFalsy();
       buttonToggleLabelElements[0].click();

src/lib/button-toggle/button-toggle.ts

@@ -30,21 +30,14 @@ import {
 } from '@angular/core';
 import {ControlValueAccessor, NG_VALUE_ACCESSOR} from '@angular/forms';
 import {
-  CanDisable,
   CanDisableRipple,
-  mixinDisabled,
   mixinDisableRipple
 } from '@angular/material/core';
 import {SelectionModel} from '@angular/cdk/collections';
 
 /** Acceptable types for a button toggle. */
 export type ToggleType = 'checkbox' | 'radio';
 
-// Boilerplate for applying mixins to MatButtonToggleGroup and MatButtonToggleGroupMultiple
-/** @docs-private */
-export class MatButtonToggleGroupBase {}
-export const _MatButtonToggleGroupMixinBase = mixinDisabled(MatButtonToggleGroupBase);
-
 /**
  * Provider Expression that allows mat-button-toggle-group to register as a ControlValueAccessor.
  * This allows it to support [(ngModel)].
@@ -81,7 +74,6 @@ export class MatButtonToggleChange {
     MAT_BUTTON_TOGGLE_GROUP_VALUE_ACCESSOR,
     {provide: MatButtonToggleGroupMultiple, useExisting: MatButtonToggleGroup},
   ],
-  inputs: ['disabled'],
   host: {
     'role': 'group',
     'class': 'mat-button-toggle-group',
@@ -90,11 +82,11 @@ export class MatButtonToggleChange {
   },
   exportAs: 'matButtonToggleGroup',
 })
-export class MatButtonToggleGroup extends _MatButtonToggleGroupMixinBase implements
-  ControlValueAccessor, CanDisable, OnInit, AfterContentInit {
+export class MatButtonToggleGroup implements ControlValueAccessor, OnInit, AfterContentInit {
 
   private _vertical = false;
   private _multiple = false;
+  private _disabled = false;
   private _selectionModel: SelectionModel<MatButtonToggle>;
 
   /**
@@ -172,13 +164,22 @@ export class MatButtonToggleGroup extends _MatButtonToggleGroupMixinBase impleme
     this._multiple = coerceBooleanProperty(value);
   }
 
+  /** Whether multiple button toggle group is disabled. */
+  @Input()
+  get disabled(): boolean { return this._disabled; }
+  set disabled(value: boolean) {
+    this._disabled = coerceBooleanProperty(value);
+
+    if (this._buttonToggles) {
+      this._buttonToggles.forEach(toggle => toggle._markForCheck());
+    }
+  }
+
   /** Event emitted when the group's value changes. */
   @Output() readonly change: EventEmitter<MatButtonToggleChange> =
       new EventEmitter<MatButtonToggleChange>();
 
-  constructor(private _changeDetector: ChangeDetectorRef) {
-    super();
-  }
+  constructor(private _changeDetector: ChangeDetectorRef) {}
 
   ngOnInit() {
     this._selectionModel = new SelectionModel<MatButtonToggle>(this.multiple, undefined, false);
@@ -210,10 +211,6 @@ export class MatButtonToggleGroup extends _MatButtonToggleGroupMixinBase impleme
   // Implemented as part of ControlValueAccessor.
   setDisabledState(isDisabled: boolean): void {
     this.disabled = isDisabled;
-
-    if (this._buttonToggles) {
-      this._buttonToggles.forEach(toggle => toggle._markForCheck());
-    }
   }
 
   /** Dispatch change event with current selection and group value. */