Addressed review issues

Author: dtaylor113

src/pagination/pagination.component.js

@@ -6,7 +6,7 @@
  * @param {number} pageNumber The current page number
  * @param {number} numTotalItems The total number of items in the data set.  When a filter is applied, update the <code>numTotalItems</code>
  * accordingly.
- * @param {number[]} pageSizeIncrements (optional) Page size increments for the 'per page' dropdown.  If not
+ * @param {Array<Number>} pageSizeIncrements (optional) Page size increments for the 'per page' dropdown.  If not
  * specified, the default values are: [5, 10, 20, 40, 80, 100]
  * @param {number} pageSize (optional) The initial page size to use.  If not specified, the default will be
  * the first increment in the <code>pageSizeIncrements</code> array.
@@ -41,7 +41,7 @@
    </div>
  </file>
  <file name="script.js">
- angular.module( 'patternfly.pagination').controller( 'PageCtrl', function( $scope, PaginationService ) {
+ angular.module( 'patternfly.pagination').controller( 'PageCtrl', function( $scope ) {
    $scope.pageSize = 10;
    $scope.pageNumber = 1;
 
@@ -94,6 +94,11 @@ angular.module('patternfly.pagination').component('pfPagination', {
 
     ctrl.onPageNumberChange = function () {
       ctrl.pageNumber = parseInt(ctrl.pageNumber, 10);
+      if (ctrl.pageNumber > ctrl.lastPageNumber) {
+        ctrl.pageNumber = ctrl.lastPageNumber;
+      } else if (ctrl.pageNumber < 1) {
+        ctrl.pageNumber = 1;
+      }
     };
 
     ctrl.gotoFirstPage = function () {

src/pagination/pagination.module.js

@@ -5,4 +5,11 @@
  *   Pagination module for patternfly.
  *
  */
-angular.module('patternfly.pagination', ['ui.bootstrap']);
+angular.module('patternfly.pagination', ['ui.bootstrap'])
+  .filter('startFrom', function () {
+    'use strict';
+    return function (input, start) {
+      start = parseInt(start, 10);
+      return input.slice(start);
+    };
+  });