address code review comment

Author: Vivian Hu

src/demo-app/list/list-demo.html

@@ -105,12 +105,12 @@ <h2>Nav lists</h2>
   </div>
 
   <div>
-    <h2>Button lists</h2>
-    <mat-button-list>
-      <button mat-list-item  *ngFor="let link of links" (click)="onButtonClick(link.name)">
-        {{ link.name }}
+    <h2>Action list</h2>
+    <mat-action-list>
+      <button mat-list-item  *ngFor="let link of links" (click)="alertItem(link.name)">
+        {{link.name}}
       </button>
-    </mat-button-list>
+    </mat-action-list>
   </div>
 
   <div>

src/demo-app/list/list-demo.ts

@@ -70,7 +70,7 @@ export class ListDemo {
     this.modelChangeEventCount++;
   }
 
-  onButtonClick(msg: string) {
+  alertItem(msg: string) {
     alert(msg);
   }
 }

src/lib/list/_list-theme.scss

@@ -8,7 +8,7 @@
   $background: map-get($theme, background);
   $foreground: map-get($theme, foreground);
 
-  .mat-list, .mat-nav-list, .mat-button-list, .mat-selection-list {
+  .mat-list, .mat-nav-list, .mat-selection-list {
     .mat-list-item {
       color: mat-color($foreground, text);
     }
@@ -27,7 +27,7 @@
   }
 
   .mat-list-option,
-  .mat-nav-list .mat-button-list .mat-list-item {
+  .mat-nav-list .mat-list-item {
     &:hover, &.mat-list-item-focus {
       background: mat-color($background, 'hover');
     }
@@ -46,7 +46,7 @@
   }
 
   // Default list
-  .mat-list, .mat-nav-list, .mat-button-list, .mat-selection-list {
+  .mat-list, .mat-nav-list, .mat-selection-list {
     .mat-list-item {
       font-size: mat-font-size($config, subheading-2);
       @include mat-line-base(mat-font-size($config, body-1));

src/lib/list/list-module.ts

@@ -20,7 +20,7 @@ import {
   MatListAvatarCssMatStyler,
   MatListIconCssMatStyler,
   MatListItem,
-  MatListSubheaderCssMatStyler, MatButtonList,
+  MatListSubheaderCssMatStyler,
 } from './list';
 import {MatListOption, MatSelectionList} from './selection-list';
 import {MatDividerModule} from '@angular/material/divider';
@@ -31,7 +31,6 @@ import {MatDividerModule} from '@angular/material/divider';
   exports: [
     MatList,
     MatNavList,
-    MatButtonList,
     MatListItem,
     MatListAvatarCssMatStyler,
     MatLineModule,
@@ -46,7 +45,6 @@ import {MatDividerModule} from '@angular/material/divider';
   declarations: [
     MatList,
     MatNavList,
-    MatButtonList,
     MatListItem,
     MatListAvatarCssMatStyler,
     MatListIconCssMatStyler,

src/lib/list/list.scss

@@ -206,7 +206,7 @@ $mat-list-item-inset-divider-offset: 72px;
   }
 }
 
-.mat-list, .mat-nav-list, .mat-button-list, .mat-selection-list {
+.mat-list, .mat-nav-list, .mat-selection-list {
   padding-top: $mat-list-top-padding;
   display: block;
 
@@ -262,7 +262,7 @@ $mat-list-item-inset-divider-offset: 72px;
   }
 }
 
-.mat-button-list {
+mat-action-list {
   //remove the native button look and make is look like a list item
   button {
     background: none;

src/lib/list/list.spec.ts

@@ -20,7 +20,7 @@ describe('MatList', () => {
         ListWithMultipleItems,
         ListWithManyLines,
         NavListWithOneAnchorItem,
-        ButtonListWithOneItem
+        ActionListWithOneItem
       ],
     });
 
@@ -145,8 +145,8 @@ describe('MatList', () => {
     items.forEach(item => expect(item._isRippleDisabled()).toBe(true));
   });
 
-  it('button list should be created successfully', () => {
-    let fixture = TestBed.createComponent(ButtonListWithOneItem);
+  it('action list should be created successfully', () => {
+    let fixture = TestBed.createComponent(ActionListWithOneItem);
     fixture.detectChanges();
 
     const items = fixture.componentInstance.listItems;
@@ -205,12 +205,12 @@ class NavListWithOneAnchorItem extends BaseTestList {
 }
 
 @Component({template: `
-  <mat-button-list>
+  <mat-action-list>
     <button mat-list-item>
         Paprika
     </button>
-  </mat-button-list>`})
-class ButtonListWithOneItem extends BaseTestList {
+  </mat-action-list>`})
+class ActionListWithOneItem extends BaseTestList {
   @ViewChildren(MatListItem) listItems: QueryList<MatListItem>;
 }
 

src/lib/list/list.ts

@@ -46,26 +46,9 @@ export const _MatListItemMixinBase = mixinDisableRipple(MatListItemBase);
 })
 export class MatNavList extends _MatListMixinBase implements CanDisableRipple {}
 
-
-@Component({
-  moduleId: module.id,
-  selector: 'mat-button-list',
-  exportAs: 'matButtonList',
-  host: {
-    'role': 'button list',
-    'class': 'mat-button-list'
-  },
-  templateUrl: 'list.html',
-  styleUrls: ['list.css'],
-  inputs: ['disableRipple'],
-  encapsulation: ViewEncapsulation.None,
-  changeDetection: ChangeDetectionStrategy.OnPush,
-})
-export class MatButtonList extends _MatListMixinBase implements CanDisableRipple {}
-
 @Component({
   moduleId: module.id,
-  selector: 'mat-list',
+  selector: 'mat-list, mat-action-list',
   exportAs: 'matList',
   templateUrl: 'list.html',
   host: {'class': 'mat-list'},