Addressed review issues

Author: dtaylor113

src/views/cardview/card-view.component.js

@@ -90,8 +90,8 @@
              <input type="checkbox" ng-model="showDisabled">Show Disabled Cards</input>
            </label>
            <label class="checkbox-inline">
- +           <input type="checkbox" ng-model="config.itemsAvailable">Items Available</input>
- +         </label>
+             <input type="checkbox" ng-model="config.itemsAvailable">Items Available</input>
+           </label>
          </div>
        </form>
      </div>
@@ -217,7 +217,7 @@ angular.module('patternfly.views').component('pfCardView', {
   },
   transclude: true,
   templateUrl: 'views/cardview/card-view.html',
-  controller: function (pfUtils, $log) {
+  controller: function () {
     'use strict';
     var ctrl = this;
     ctrl.defaultConfig = {

src/views/empty-state.component.js

@@ -116,7 +116,7 @@ angular.module('patternfly.views').component('pfEmptyState', {
       }
     };
 
-    ctrl.areMainActions = function () {
+    ctrl.hasMainActions = function () {
       var mainActions;
 
       if (ctrl.actionButtons) {
@@ -127,7 +127,7 @@ angular.module('patternfly.views').component('pfEmptyState', {
       return false;
     };
 
-    ctrl.areSecondaryActions = function () {
+    ctrl.hasSecondaryActions = function () {
       var secondaryActions;
 
       if (ctrl.actionButtons) {

src/views/empty-state.html

@@ -11,15 +11,15 @@ <h1 id="title">
   <p id="helpLink" ng-if="$ctrl.config.helpLink">
     {{$ctrl.config.helpLink.label}} <a href="{{$ctrl.config.helpLink.url}}">{{$ctrl.config.helpLink.urlLabel}}</a>.
   </p>
-  <div ng-if="$ctrl.areMainActions()" class="blank-slate-pf-main-action">
+  <div ng-if="$ctrl.hasMainActions()" class="blank-slate-pf-main-action">
     <button class="btn btn-primary btn-lg"
             ng-repeat="actionButton in $ctrl.actionButtons | filter:$ctrl.filterMainActions"
             title="{{actionButton.title}}"
             ng-click="$ctrl.handleButtonAction(actionButton)">
       {{actionButton.name}}
     </button>
   </div>
-  <div ng-if="$ctrl.areSecondaryActions()" class="blank-slate-pf-secondary-action">
+  <div ng-if="$ctrl.hasSecondaryActions()" class="blank-slate-pf-secondary-action">
     <button class="btn btn-default"
             ng-repeat="actionButton in $ctrl.actionButtons | filter:$ctrl.filterSecondaryActions"
             title="{{actionButton.title}}"

src/views/listview/list-view.component.js

@@ -420,7 +420,7 @@ angular.module('patternfly.views').component('pfListView', {
     expandedContent: '?listExpandedContent'
   },
   templateUrl: 'views/listview/list-view.html',
-  controller: function ($timeout, $window, $element, pfUtils) {
+  controller: function ($timeout, $window, $element) {
     'use strict';
     var ctrl = this;
 

styles/angular-patternfly.css

@@ -513,11 +513,13 @@ table.dataTable tbody th, table.dataTable tbody td {
   width: 13px;
 }
 
+/* overriding pf base so that blank slate fills entire parent container */
 .blank-slate-pf {
   height:100%;
   margin-bottom: 0px;
 }
 
+/* overriding pf base so that buttons have spaces between them */
 .blank-slate-pf button {
   margin-right: 4px;
 }