feat(material-experimental/chip): add focus indicators (#18261)

Author: zelliott

src/material-experimental/mdc-chips/chip-grid.spec.ts

@@ -239,6 +239,14 @@ describe('MDC-based MatChipGrid', () => {
             expect(chipGridInstance._keyManager.activeColumnIndex).toBe(0);
         }));
       });
+
+      it('should have a focus indicator', () => {
+        const focusableTextNativeElements = Array.from(chipGridNativeElement
+            .querySelectorAll('.mat-chip-row-focusable-text-content'));
+
+        expect(focusableTextNativeElements
+            .every(element => element.classList.contains('mat-mdc-focus-indicator'))).toBe(true);
+      });
     });
 
     describe('keyboard behavior', () => {

src/material-experimental/mdc-chips/chip-icons.ts

@@ -105,8 +105,8 @@ const _MatChipRemoveMixinBase:
   selector: '[matChipRemove]',
   inputs: ['disabled', 'tabIndex'],
   host: {
-    'class':
-      'mat-mdc-chip-remove mat-mdc-chip-trailing-icon mdc-chip__icon mdc-chip__icon--trailing',
+    'class': `mat-mdc-chip-remove mat-mdc-chip-trailing-icon mat-mdc-focus-indicator
+        mdc-chip__icon mdc-chip__icon--trailing`,
     '[tabIndex]': 'tabIndex',
     'role': 'button',
     '(click)': 'interaction.next($event)',

src/material-experimental/mdc-chips/chip-option.html

@@ -1,4 +1,11 @@
 <span class="mdc-chip__ripple"></span>
+
+<span matRipple class="mat-mdc-chip-ripple"
+     [matRippleAnimation]="_rippleAnimation"
+     [matRippleDisabled]="_isRippleDisabled()"
+     [matRippleCentered]="_isRippleCentered"
+     [matRippleTrigger]="_elementRef.nativeElement"></span>
+
 <ng-content select="mat-chip-avatar, [matChipAvatar]"></ng-content>
 <div class="mdc-chip__checkmark" *ngIf="_chipListMultiple">
   <svg class="mdc-chip__checkmark-svg" viewBox="-2 -3 30 30">

src/material-experimental/mdc-chips/chip-option.spec.ts

@@ -3,7 +3,6 @@ import {SPACE} from '@angular/cdk/keycodes';
 import {createKeyboardEvent, dispatchFakeEvent} from '@angular/cdk/testing/private';
 import {Component, DebugElement, ViewChild} from '@angular/core';
 import {async, ComponentFixture, fakeAsync, flush, TestBed} from '@angular/core/testing';
-import {MAT_RIPPLE_GLOBAL_OPTIONS, RippleGlobalOptions} from '@angular/material/core';
 import {By} from '@angular/platform-browser';
 import {Subject} from 'rxjs';
 import {
@@ -20,17 +19,14 @@ describe('MDC-based Option Chips', () => {
   let chipDebugElement: DebugElement;
   let chipNativeElement: HTMLElement;
   let chipInstance: MatChipOption;
-  let globalRippleOptions: RippleGlobalOptions;
 
   let dir = 'ltr';
 
   beforeEach(async(() => {
-    globalRippleOptions = {};
     TestBed.configureTestingModule({
       imports: [MatChipsModule],
       declarations: [SingleChip],
       providers: [
-        {provide: MAT_RIPPLE_GLOBAL_OPTIONS, useFactory: () => globalRippleOptions},
         {provide: Directionality, useFactory: () => ({
           value: dir,
           change: new Subject()
@@ -269,6 +265,10 @@ describe('MDC-based Option Chips', () => {
         expect(chipNativeElement.getAttribute('aria-disabled')).toBe('true');
       });
     });
+
+    it('should have a focus indicator', () => {
+      expect(chipNativeElement.classList.contains('mat-mdc-focus-indicator')).toBe(true);
+    });
   });
 });
 

src/material-experimental/mdc-chips/chip-option.ts

@@ -42,6 +42,7 @@ export class MatChipSelectionChange {
   inputs: ['color', 'disableRipple', 'tabIndex'],
   host: {
     'role': 'option',
+    'class': 'mat-mdc-focus-indicator',
     '[class.mat-mdc-chip-disabled]': 'disabled',
     '[class.mat-mdc-chip-highlighted]': 'highlighted',
     '[class.mat-mdc-chip-with-avatar]': 'leadingIcon',

src/material-experimental/mdc-chips/chip-remove.spec.ts

@@ -150,6 +150,11 @@ describe('MDC-based Chip Remove', () => {
       expect(event.defaultPrevented).toBe(false);
     });
 
+    it('should have a focus indicator', () => {
+      const buttonElement = chipNativeElement.querySelector('button')!;
+
+      expect(buttonElement.classList.contains('mat-mdc-focus-indicator')).toBe(true);
+    });
   });
 });
 

src/material-experimental/mdc-chips/chip-row.html

@@ -1,6 +1,14 @@
+<span class="mdc-chip__ripple"></span>
+
+<span matRipple class="mat-mdc-chip-ripple"
+     [matRippleAnimation]="_rippleAnimation"
+     [matRippleDisabled]="_isRippleDisabled()"
+     [matRippleCentered]="_isRippleCentered"
+     [matRippleTrigger]="_elementRef.nativeElement"></span>
+
 <div role="gridcell">
-  <div #chipContent tabindex="-1" class="mat-chip-row-focusable-text-content">
-     <span class="mdc-chip__ripple"></span>
+  <div #chipContent tabindex="-1"
+       class="mat-chip-row-focusable-text-content mat-mdc-focus-indicator">
   	 <ng-content select="mat-chip-avatar, [matChipAvatar]"></ng-content>
   	 <span class="mdc-chip__text"><ng-content></ng-content></span>
   	 <ng-content select="mat-chip-trailing-icon,[matChipTrailingIcon]"></ng-content>

src/material-experimental/mdc-chips/chip-row.spec.ts

@@ -7,7 +7,6 @@ import {
 } from '@angular/cdk/testing/private';
 import {Component, DebugElement, ViewChild} from '@angular/core';
 import {async, ComponentFixture, TestBed} from '@angular/core/testing';
-import {MAT_RIPPLE_GLOBAL_OPTIONS, RippleGlobalOptions} from '@angular/material/core';
 import {By} from '@angular/platform-browser';
 import {Subject} from 'rxjs';
 import {MatChipEvent, MatChipGrid, MatChipRow, MatChipsModule} from './index';
@@ -18,17 +17,14 @@ describe('MDC-based Row Chips', () => {
   let chipDebugElement: DebugElement;
   let chipNativeElement: HTMLElement;
   let chipInstance: MatChipRow;
-  let globalRippleOptions: RippleGlobalOptions;
 
   let dir = 'ltr';
 
   beforeEach(async(() => {
-    globalRippleOptions = {};
     TestBed.configureTestingModule({
       imports: [MatChipsModule],
       declarations: [SingleChip],
       providers: [
-        {provide: MAT_RIPPLE_GLOBAL_OPTIONS, useFactory: () => globalRippleOptions},
         {provide: Directionality, useFactory: () => ({
           value: dir,
           change: new Subject()

src/material-experimental/mdc-chips/chip.html

@@ -1,4 +1,11 @@
 <span class="mdc-chip__ripple"></span>
+
+<span matRipple class="mat-mdc-chip-ripple"
+     [matRippleAnimation]="_rippleAnimation"
+     [matRippleDisabled]="_isRippleDisabled()"
+     [matRippleCentered]="_isRippleCentered"
+     [matRippleTrigger]="_elementRef.nativeElement"></span>
+
 <ng-content select="mat-chip-avatar, [matChipAvatar]"></ng-content>
 <div class="mdc-chip__text mdc-chip__action--primary"><ng-content></ng-content></div>
 <ng-content select="mat-chip-trailing-icon,[matChipRemove],[matChipTrailingIcon]"></ng-content>

src/material-experimental/mdc-chips/chip.spec.ts

@@ -2,7 +2,7 @@ import {Directionality} from '@angular/cdk/bidi';
 import {createFakeEvent} from '@angular/cdk/testing/private';
 import {Component, DebugElement, ViewChild} from '@angular/core';
 import {async, ComponentFixture, TestBed} from '@angular/core/testing';
-import {MAT_RIPPLE_GLOBAL_OPTIONS, RippleGlobalOptions} from '@angular/material/core';
+import {MatRipple} from '@angular/material/core';
 import {By} from '@angular/platform-browser';
 import {Subject} from 'rxjs';
 import {MatChip, MatChipEvent, MatChipSet, MatChipsModule} from './index';
@@ -13,17 +13,16 @@ describe('MDC-based MatChip', () => {
   let chipDebugElement: DebugElement;
   let chipNativeElement: HTMLElement;
   let chipInstance: MatChip;
-  let globalRippleOptions: RippleGlobalOptions;
+  let chipRippleDebugElement: DebugElement;
+  let chipRippleInstance: MatRipple;
 
   let dir = 'ltr';
 
   beforeEach(async(() => {
-    globalRippleOptions = {};
     TestBed.configureTestingModule({
       imports: [MatChipsModule],
       declarations: [BasicChip, SingleChip],
       providers: [
-        {provide: MAT_RIPPLE_GLOBAL_OPTIONS, useFactory: () => globalRippleOptions},
         {provide: Directionality, useFactory: () => ({
           value: dir,
           change: new Subject()
@@ -35,19 +34,24 @@ describe('MDC-based MatChip', () => {
   }));
 
   describe('MatBasicChip', () => {
-
     beforeEach(() => {
       fixture = TestBed.createComponent(BasicChip);
       fixture.detectChanges();
 
       chipDebugElement = fixture.debugElement.query(By.directive(MatChip))!;
       chipNativeElement = chipDebugElement.nativeElement;
       chipInstance = chipDebugElement.injector.get<MatChip>(MatChip);
+      chipRippleDebugElement = chipDebugElement.query(By.directive(MatRipple))!;
+      chipRippleInstance = chipRippleDebugElement.injector.get<MatRipple>(MatRipple);
     });
 
     it('adds the `mat-mdc-basic-chip` class', () => {
       expect(chipNativeElement.classList).toContain('mat-mdc-basic-chip');
     });
+
+    it('should have its ripple disabled', () => {
+      expect(chipRippleInstance.disabled).toBe(true, 'Expected basic chip ripples to be disabled.');
+    });
   });
 
   describe('MatChip', () => {
@@ -60,6 +64,8 @@ describe('MDC-based MatChip', () => {
       chipDebugElement = fixture.debugElement.query(By.directive(MatChip))!;
       chipNativeElement = chipDebugElement.nativeElement;
       chipInstance = chipDebugElement.injector.get<MatChip>(MatChip);
+      chipRippleDebugElement = chipDebugElement.query(By.directive(MatRipple))!;
+      chipRippleInstance = chipRippleDebugElement.injector.get<MatRipple>(MatRipple);
       testComponent = fixture.debugElement.componentInstance;
     });
 
@@ -115,12 +121,22 @@ describe('MDC-based MatChip', () => {
       expect(chipNativeElement.style.display).toBe('none');
     });
 
-    it('should be able to disable ripples through ripple global options at runtime', () => {
-      expect(chipInstance.rippleDisabled).toBe(false, 'Expected chip ripples to be enabled.');
+    it('should be able to disable ripples with the `[rippleDisabled]` input', () => {
+      expect(chipRippleInstance.disabled).toBe(false, 'Expected chip ripples to be enabled.');
+
+      testComponent.rippleDisabled = true;
+      fixture.detectChanges();
+
+      expect(chipRippleInstance.disabled).toBe(true, 'Expected chip ripples to be disabled.');
+    });
+
+    it('should disable ripples when the chip is disabled', () => {
+      expect(chipRippleInstance.disabled).toBe(false, 'Expected chip ripples to be enabled.');
 
-      globalRippleOptions.disabled = true;
+      testComponent.disabled = true;
+      fixture.detectChanges();
 
-      expect(chipInstance.rippleDisabled).toBe(true, 'Expected chip ripples to be disabled.');
+      expect(chipRippleInstance.disabled).toBe(true, 'Expected chip ripples to be disabled.');
     });
 
     it('should update the aria-label for disabled chips', () => {
@@ -163,7 +179,7 @@ describe('MDC-based MatChip', () => {
         <mat-chip [removable]="removable"
                  [color]="color" [disabled]="disabled"
                  (focus)="chipFocus($event)" (destroyed)="chipDestroy($event)"
-                 (removed)="chipRemove($event)" [value]="value">
+                 (removed)="chipRemove($event)" [value]="value" [disableRipple]="rippleDisabled">
           {{name}}
         </mat-chip>
       </div>
@@ -177,6 +193,7 @@ class SingleChip {
   removable: boolean = true;
   shouldShow: boolean = true;
   value: any;
+  rippleDisabled: boolean = false;
 
   chipFocus: (event?: MatChipEvent) => void = () => {};
   chipDestroy: (event?: MatChipEvent) => void = () => {};