fix($location): rewrite links with nested elements

For example:
<a href="some/link">inner <span>text</span></a>

If you click on "text", then the span element is event.target, so we need to traverse the DOM.

1 files changed, 8 insertions, 4 deletions

diff --git a/src/service/location.js b/src/service/location.js
index 2ffc5587..d1d34e67 100644
--- a/src/service/location.js
+++ b/src/service/location.js
@@ -442,12 +442,16 @@ angularServiceInject('$location', function($browser, $sniffer, $locationConfig,
       // TODO(vojta): rewrite link when opening in new tab/window (in legacy browser)
       // currently we open nice url link and redirect then
 
-      if (uppercase(event.target.nodeName) != 'A' || event.ctrlKey || event.metaKey ||
-          event.which == 2) return;
+      if (event.ctrlKey || event.metaKey || event.which == 2) return;
 
-      var elm = jqLite(event.target),
-          href = elm.attr('href');
+      var elm = jqLite(event.target);
 
+      // traverse the DOM up to find first A tag
+      while (elm.length && lowercase(elm[0].nodeName) !== 'a') {
+        elm = elm.parent();
+      }
+
+      var href = elm.attr('href');
       if (!href || isDefined(elm.attr('ng:ext-link')) || elm.attr('target')) return;
 
       // remove same domain from full url links (IE7 always returns full hrefs)