Merge pull request #470 from bleathem/master

Set the patternfly version to 3.25.0

Author: dtaylor113

bower.json

@@ -43,10 +43,14 @@
     "angular-sanitize": "1.3.0 - 1.5.*",
     "angular-bootstrap": "0.14.x",
     "lodash": "3.x",
-    "patternfly": "~3.24.0"
+    "patternfly": "3.25.0",
+    "jquery": "2"
   },
   "devDependencies": {
     "angular-mocks": "1.3.0 - 1.5.*",
     "angular-ui-router": "1.0.0-beta.3"
+  },
+  "resolutions": {
+    "jquery": "2"
   }
 }

misc/angular-bootstrap-prettify.js

@@ -254,7 +254,7 @@ directive.ngEmbedApp = ['$templateCache', '$browser', '$rootScope', '$location',
         }
       });
 
-      element.bind('$destroy', function() {
+      element.on('$destroy', function() {
         deregisterEmbedRootScope();
         embedRootScope.$destroy();
       });

npm-shrinkwrap.json

@@ -4,7 +4,10 @@
   "version": "3.24.0",
   "license": "Apache-2.0",
   "description": "Angular extension of the PatternFly project.",
-  "keywords": ["angular", "patternfly"],
+  "keywords": [
+    "angular",
+    "patternfly"
+  ],
   "main": "index.js",
   "homepage": "https://github.com/patternfly/angular-patternfly",
   "dependencies": {
@@ -13,8 +16,9 @@
     "angular-drag-and-drop-lists": "2.0.0",
     "angular-sanitize": "1.3.0 - 1.5.*",
     "angular-ui-bootstrap": "0.14.x",
+    "jquery": "^2.2.4",
     "lodash": "3.x",
-    "patternfly": "~3.24.0"
+    "patternfly": "3.25.0"
   },
   "devDependencies": {
     "express": "3.4.4",

package.json

@@ -375,7 +375,7 @@ angular.module('patternfly.charts').directive('pfHeatmap', function ($compile, $
         setStyles();
       });
 
-      angular.element($window).bind('resize', function () {
+      angular.element($window).on('resize', function () {
         setSizes();
         redraw();
       });

src/charts/heatmap/heatmap.directive.js

@@ -164,7 +164,7 @@ angular.module('patternfly.form').directive('pfRemainingCharsCount', function ($
         checkCharactersRemaining();
       });
 
-      $element.bind('keypress', function (event) {
+      $element.on('keypress', function (event) {
         // Once the charsMaxLimit has been met or exceeded, prevent all keypresses from working
         if (blockInputAtMaxLimit && $element.val().length >= charsMaxLimit) {
           // Except backspace

src/form/remaining-chars-count/remaining-chars-count.directive.js

@@ -551,7 +551,7 @@ angular.module('patternfly.navigation').directive('pfVerticalNavigation', ['$loc
         initBodyElement();
         checkNavState();
 
-        angular.element($window).bind('resize', function () {
+        angular.element($window).on('resize', function () {
           checkNavState();
           $timeout(function () {
             try {

src/navigation/vertical-navigation-directive.js

@@ -142,7 +142,7 @@ angular.module('patternfly.utils').directive('pfFixedAccordion', function ($wind
       }, 100);
 
       // Update on window resizing
-      angular.element($window).bind('resize', function () {
+      angular.element($window).on('resize', function () {
         setCollapseHeights();
       });
     }

src/utils/fixed-accordion.directive.js

@@ -31,7 +31,7 @@ describe('Directive: pfAggregateStatusCard', function() {
       expect(angular.element(element).find('.card-pf-aggregate-status-count').html()).toBe("793");
 
       //Make sure a link renders in the title
-      expect(angular.element(element).find('.card-pf-title').find('a').size()).toBe(1);
+      expect(angular.element(element).find('.card-pf-title').find('a').length).toBe(1);
 
       //Make sure the class is getting set for the title icon
       expect(angular.element(element).find('.card-pf-title').find('.fa').hasClass('fa-shield')).toBeTruthy();
@@ -53,7 +53,7 @@ describe('Directive: pfAggregateStatusCard', function() {
       element = compileCard('<div pf-aggregate-status-card status="status"></div>', $scope);
 
       //Make sure a link renders in the title
-      expect(angular.element(element).find('.card-pf-title').find('a').size()).toBe(0);
+      expect(angular.element(element).find('.card-pf-title').find('a').length).toBe(0);
     });
 
     it("should set the notifications", function() {
@@ -81,13 +81,13 @@ describe('Directive: pfAggregateStatusCard', function() {
       notifications = angular.element(element).find('.card-pf-aggregate-status-notification');
 
       //Make sure two notifications render
-      expect(notifications.size()).toBe(2);
+      expect(notifications.length).toBe(2);
 
       //First notification should have a link
-      expect(notifications.eq(0).find('a').size()).toBe(1);
+      expect(notifications.eq(0).find('a').length).toBe(1);
 
       //Second notification should not have a link
-      expect(notifications.eq(1).find('a').size()).toBe(0);
+      expect(notifications.eq(1).find('a').length).toBe(0);
 
       //first notification should have the following class
       expect(notifications.eq(0).find('span')).toHaveClass('pficon pficon-error-circle-o');
@@ -135,7 +135,7 @@ describe('Directive: pfAggregateStatusCard', function() {
 
       // should show the main icon
       cardClass = angular.element(element).find('.fa-shield');
-      expect(cardClass.size()).toBe(1);
+      expect(cardClass.length).toBe(1);
 
       notifications = angular.element(element).find('.card-pf-aggregate-status-notification');
 
@@ -166,7 +166,7 @@ describe('Directive: pfAggregateStatusCard', function() {
 
       // should not show the main icon
       cardClass = angular.element(element).find('.fa-shield');
-      expect(cardClass.size()).toBe(0);
+      expect(cardClass.length).toBe(0);
 
       notifications = angular.element(element).find('.card-pf-aggregate-status-notification');