pfTableView: refactor column 'colHeader' & 'colItemFld' to 'header' and 'itemField'

Author: dtaylor113

src/table/tableview/examples/table-view-basic.js

@@ -16,8 +16,8 @@
   * @param {array} items Array of items to display in the table view.
   * @param {array} columns Array of table column information to display in the table's header row
   * <ul style='list-style-type: none'>
-  *   <li>.colHeader     - (string) Text label for a column header
-  *   <li>.colItemFld    - (string) Item field to associate with a particular column.
+  *   <li>.header     - (string) Text label for a column header
+  *   <li>.itemField    - (string) Item field to associate with a particular column.
   * </ul>
   * @param {array} actionButtons List of action buttons in each row
   *   <ul style='list-style-type: none'>
@@ -62,10 +62,10 @@
         };
 
         $scope.colummns = [
-          { colHeader: "Name", colItemFld: "name" },
-          { colHeader: "Address", colItemFld: "address"},
-          { colHeader: "City", colItemFld: "city" },
-          { colHeader: "State", colItemFld: "state"}
+          { header: "Name", itemField: "name" },
+          { header: "Address", itemField: "address"},
+          { header: "City", itemField: "city" },
+          { header: "State", itemField: "state"}
         ];
 
         $scope.items = [

src/table/tableview/examples/table-view-with-toolbar.js

@@ -15,8 +15,8 @@
  * @param {array} items Array of items to display in the table view.
  * @param {array} columns Array of table column information to display in the table's header row
  * <ul style='list-style-type: none'>
- *   <li>.colHeader     - (string) Text label for a column header
- *   <li>.colItemFld    - (string) Item field to associate with a particular column.
+ *   <li>.header     - (string) Text label for a column header
+ *   <li>.itemField    - (string) Item field to associate with a particular column.
  * </ul>
  * @param {array} actionButtons List of action buttons in each row
  *   <ul style='list-style-type: none'>
@@ -76,10 +76,10 @@
       $scope.actionsText = "";
 
       $scope.colummns = [
-        { colHeader: "Name", colItemFld: "name" },
-        { colHeader: "Age", colItemFld: "age"},
-        { colHeader: "Address", colItemFld: "address" },
-        { colHeader: "BirthMonth", colItemFld: "birthMonth"}
+        { header: "Name", itemField: "name" },
+        { header: "Age", itemField: "age"},
+        { header: "Address", itemField: "address" },
+        { header: "BirthMonth", itemField: "birthMonth"}
       ];
 
       $scope.dtOptions = {

src/table/tableview/table-view.component.js

@@ -80,9 +80,9 @@ angular.module('patternfly.table').component('pfTableView', {
       if (!validSelectionMatchProp()) {
         angular.forEach(ctrl.colummns, function (col) {
           if (props.length === 0) {
-            props = col.colItemFld;
+            props = col.itemField;
           } else {
-            props += ", " + col.colItemFld;
+            props += ", " + col.itemField;
           }
         });
         throw new Error("pfTableView - " +
@@ -332,7 +332,7 @@ angular.module('patternfly.table').component('pfTableView', {
 
     ctrl.isColItemFld = function (key) {
       var retVal = false;
-      var tableCol = $filter('filter')(ctrl.colummns, {colItemFld: key});
+      var tableCol = $filter('filter')(ctrl.colummns, {itemField: key});
 
       if (tableCol && tableCol.length === 1) {
         retVal = true;

src/table/tableview/table-view.html

@@ -3,7 +3,7 @@
   <thead>
     <tr role="row">
       <th class="table-view-pf-select"><input type="checkbox" value="$ctrl.selectAll" ng-model="$ctrl.selectAll" ng-change="$ctrl.toggleAll()"/></th>
-      <th ng-repeat="col in $ctrl.colummns">{{col.colHeader}}</th>
+      <th ng-repeat="col in $ctrl.colummns">{{col.header}}</th>
       <th ng-if="$ctrl.areActions()" colspan="{{$ctrl.calcActionsColspan()}}">Actions</th>
     </tr>
   </thead>

src/toolbars/examples/toolbar.js

@@ -144,10 +144,10 @@
       $scope.filtersText = '';
 
       $scope.colummns = [
-        { colHeader: "Name", colItemFld: "name" },
-        { colHeader: "Age", colItemFld: "age"},
-        { colHeader: "Address", colItemFld: "address" },
-        { colHeader: "BirthMonth", colItemFld: "birthMonth"}
+        { header: "Name", itemField: "name" },
+        { header: "Age", itemField: "age"},
+        { header: "Address", itemField: "address" },
+        { header: "BirthMonth", itemField: "birthMonth"}
       ];
 
       // attempt to dyamically turn on/off pagination controls