From b5507cd0e96f6004d7bf0262971fc4d16e9c89ea Mon Sep 17 00:00:00 2001
From: Martin Belev <martinbelev93@gmail.com>
Date: Thu, 20 Apr 2017 20:02:50 +0300
Subject: [PATCH] fix(checkbox): fix pull request comments

 * remove redundant css class
 * add JSDoc to hasLabel and change it from getter to function
---
 src/lib/checkbox/checkbox.html |  2 +-
 src/lib/checkbox/checkbox.scss | 17 +++++------------
 src/lib/checkbox/checkbox.ts   |  7 ++++---
 3 files changed, 10 insertions(+), 16 deletions(-)

diff --git a/src/lib/checkbox/checkbox.html b/src/lib/checkbox/checkbox.html
index 48f144370829..277a9cb69185 100644
--- a/src/lib/checkbox/checkbox.html
+++ b/src/lib/checkbox/checkbox.html
@@ -1,6 +1,6 @@
 <label class="mat-checkbox-layout" #label>
   <div class="mat-checkbox-inner-container"
-       [class.mat-checkbox-inner-container-no-side-margin]="!hasLabel">
+       [class.mat-checkbox-inner-container-no-side-margin]="!_hasLabel()">
     <input #input
            class="mat-checkbox-input cdk-visually-hidden" type="checkbox"
            [id]="inputId"
diff --git a/src/lib/checkbox/checkbox.scss b/src/lib/checkbox/checkbox.scss
index 8cfc8eac2693..196cde2bd044 100644
--- a/src/lib/checkbox/checkbox.scss
+++ b/src/lib/checkbox/checkbox.scss
@@ -226,12 +226,12 @@ $_mat-checkbox-mark-stroke-size: 2 / 15 * $mat-checkbox-size !default;
       right: auto;
     }
   }
+}
 
-  &.mat-checkbox-inner-container-no-side-margin {
-    margin: {
-      left: 0;
-      right: 0;
-    }
+.mat-checkbox-inner-container-no-side-margin {
+  margin: {
+    left: 0;
+    right: 0;
   }
 }
 
@@ -300,13 +300,6 @@ $_mat-checkbox-mark-stroke-size: 2 / 15 * $mat-checkbox-size !default;
         right: $_mat-checkbox-item-spacing;
       }
     }
-
-    &.mat-checkbox-inner-container-no-side-margin {
-      margin: {
-        left: 0;
-        right: 0;
-      }
-    }
   }
 }
 
diff --git a/src/lib/checkbox/checkbox.ts b/src/lib/checkbox/checkbox.ts
index 4d5d4ed12f31..c3cf7ffc2979 100644
--- a/src/lib/checkbox/checkbox.ts
+++ b/src/lib/checkbox/checkbox.ts
@@ -166,9 +166,10 @@ export class MdCheckbox implements ControlValueAccessor, AfterViewInit, OnDestro
 
   @ViewChild('labelWrapper') _labelWrapper: ElementRef;
 
-  get hasLabel(): boolean {
-    const labelWrapperTextContent = this._labelWrapper.nativeElement.textContent;
-    return labelWrapperTextContent && labelWrapperTextContent.trim().length > 0;
+  /** Whether the checkbox has label */
+  _hasLabel(): boolean {
+    const labelText = this._labelWrapper.nativeElement.textContent || '';
+    return !!labelText.trim().length;
   }
 
   /**