Merge lp:~allenap/launchpad/localpackagediffs-mouse-pointer-bug-815788 into lp:launchpad

[Gavin Panella]

[r=allenap][bug=815788] On DistroSeries:+localpackagediffs ensure that links dynamically added from JavaScript have a similar visual appearance and behaviour to others on Launchpad.

This branch does three small things in the
lp.registry.distroseries.differences JavaScript module:

- Changes linkify() to set href:

+ /* Set the href so that the visual display is consistent with
+ other links (cursor most notably). */

- Now that the links produced by linkify() have hrefs, the events
  produced by click have to be halted once we've done our thing.

- The test suite was failing because it was looking for the LP
  global. I've added this to the test case set up and tear down.

[Gavin Panella (allenap)]

review: Approve

=== modified file 'lib/lp/registry/javascript/distroseries/differences.js'
--- lib/lp/registry/javascript/distroseries/differences.js	2011-07-29 15:53:15 +0000
+++ lib/lp/registry/javascript/distroseries/differences.js	2011-07-29 15:53:15 +0000
@@ -89,7 +89,10 @@
  *
  */
 var linkify = function(node) {
-    var link = Y.Node.create('<a class="js-action sprite edit" />');
+    /* Set the href so that the visual display is consistent with
+       other links (cursor most notably). */
+    var link = Y.Node.create(
+        '<a href="#" class="js-action sprite edit" />');
     link.set("innerHTML", node.get("innerHTML"));
     node.empty().append(link);
     return link;
@@ -194,8 +197,10 @@
     overlay.after("visibleChange", initialize_picker);
 
     /* Linkify the origin and show the overlay when it's clicked. */
-    var link = linkify(origin);
-    link.on("click", function() { overlay.show(); });
+    linkify(origin).on("click", function(e) {
+        e.halt();
+        overlay.show();
+    });
 };
 
 
@@ -257,7 +262,10 @@
     });
 
     /* Linkify the origin and show the picker when it's clicked. */
-    linkify(origin).on("click", function(e) { picker.show(); });
+    linkify(origin).on("click", function(e) {
+        e.halt();
+        picker.show();
+    });
 
     return picker;
 };

=== modified file 'lib/lp/registry/javascript/distroseries/tests/test_differences.js'
--- lib/lp/registry/javascript/distroseries/tests/test_differences.js	2011-07-29 15:53:15 +0000
+++ lib/lp/registry/javascript/distroseries/tests/test_differences.js	2011-07-29 15:53:15 +0000
@@ -85,6 +85,9 @@
             Assert.areSame(link, target.one("a"));
             Assert.areSame("Foobar", link.get("text"));
             Assert.isTrue(link.hasClass("js-action"));
+            var href = link.get("href");
+            Assert.isString(href);
+            Assert.isTrue(href.length > 0);
         }
 
     };
@@ -95,6 +98,11 @@
         name: "TestLastChangedPicker",
 
         setUp: function() {
+            window.LP = {
+                links: {
+                    me: "~foobar"
+                }
+            };
             var body = Y.one(document.body);
             this.form = Y.Node.create("<form />").appendTo(body);
             this.origin = Y.Node.create("<div>Origin</div>").appendTo(body);
@@ -106,6 +114,7 @@
             this.picker.get("boundingBox").remove(true);
             this.origin.remove(true);
             this.form.remove(true);
+            delete window.LP;
         },
 
         test_linkify_click_activates_picker: function() {