Merge pull request #81 from spadgett/tile-click-button

Don't trigger tile-click target if user clicks a button inside the tile

dist/origin-web-common-ui.js

@@ -885,8 +885,9 @@ angular.module('openshiftCommonUI')
       restrict: 'AC',
       link: function($scope, element) {
         $(element).click(function (evt) {
+          // Don't trigger tile target if the user clicked directly on a link or button inside the tile.
           var t = $(evt.target);
-          if (t && t.closest("a", element).length) {
+          if (t && (t.closest("a", element).length || t.closest("button", element).length)) {
             return;
           }
           $('a.tile-target', element).trigger("click");

dist/origin-web-common.js

@@ -1056,8 +1056,9 @@ angular.module('openshiftCommonUI')
       restrict: 'AC',
       link: function($scope, element) {
         $(element).click(function (evt) {
+          // Don't trigger tile target if the user clicked directly on a link or button inside the tile.
           var t = $(evt.target);
-          if (t && t.closest("a", element).length) {
+          if (t && (t.closest("a", element).length || t.closest("button", element).length)) {
             return;
           }
           $('a.tile-target', element).trigger("click");

dist/origin-web-common.min.js

@@ -339,7 +339,7 @@ restrict:"AC",
 link:function($scope, element) {
 $(element).click(function(evt) {
 var t = $(evt.target);
-t && t.closest("a", element).length || $("a.tile-target", element).trigger("click");
+t && (t.closest("a", element).length || t.closest("button", element).length) || $("a.tile-target", element).trigger("click");
 });
 }
 };

src/components/tile-click/tileClick.js

@@ -6,8 +6,9 @@ angular.module('openshiftCommonUI')
       restrict: 'AC',
       link: function($scope, element) {
         $(element).click(function (evt) {
+          // Don't trigger tile target if the user clicked directly on a link or button inside the tile.
           var t = $(evt.target);
-          if (t && t.closest("a", element).length) {
+          if (t && (t.closest("a", element).length || t.closest("button", element).length)) {
             return;
           }
           $('a.tile-target', element).trigger("click");