Better to delete if (this.selectionList.disabled) {...} part

Author: sllethe

src/lib/list/list-option.ts

@@ -46,7 +46,7 @@ const FOCUSED_STYLE: string = 'mat-list-item-focus';
     'class': 'mat-list-item mat-list-option',
     '(focus)': '_handleFocus()',
     '(blur)': '_handleBlur()',
-    '(click)': 'toggle()',
+    '(click)': '_handleClick()',
     'tabindex': '-1',
     '[attr.aria-selected]': 'selected.toString()',
     '[attr.aria-disabled]': 'disabled.toString()',
@@ -125,11 +125,9 @@ export class MdListOption implements AfterContentInit, OnDestroy, Focusable {
   }
 
   toggle(): void {
-    if (!this._disabled) {
-      this.selected = !this.selected;
-      this.selectionList.selectedOptions.toggle(this);
-      this._changeDetector.markForCheck();
-    }
+    this.selected = !this.selected;
+    this.selectionList.selectedOptions.toggle(this);
+    this._changeDetector.markForCheck();
   }
 
   /** Allows for programmatic focusing of the option. */
@@ -143,6 +141,12 @@ export class MdListOption implements AfterContentInit, OnDestroy, Focusable {
     return !this.disableRipple && !this.selectionList.disableRipple;
   }
 
+  _handleClick() {
+    if (!this._disabled) {
+      this.toggle();
+    }
+  }
+
   _handleFocus() {
     this._hasFocus = true;
     this._renderer.addClass(this._element.nativeElement, FOCUSED_STYLE);