launchpad-reviewers team mailing list archive

[Colin Watson <cjwatson@xxxxxxxxxxxxx>]

Colin Watson has proposed merging ~cjwatson/launchpad:js-edit-comment-whitespace into launchpad:master.

Commit message:
Tweak whitespace when editing a comment via JS

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~cjwatson/launchpad/+git/launchpad/+merge/403394
-- 
Your team Launchpad code reviewers is requested to review the proposed merge of ~cjwatson/launchpad:js-edit-comment-whitespace into launchpad:master.

diff --git a/lib/lp/services/messages/javascript/messages.edit.js b/lib/lp/services/messages/javascript/messages.edit.js
index f84bf72..c6e79b6 100644
--- a/lib/lp/services/messages/javascript/messages.edit.js
+++ b/lib/lp/services/messages/javascript/messages.edit.js
@@ -155,7 +155,7 @@ YUI.add('lp.services.messages.edit', function(Y) {
         elements.update_btn.getDOMNode().disabled = false;
         module.hideLoading(elements.container);
         elements.last_edit.getDOMNode().innerHTML = (
-            '<a href="#" class="editable-message-last-edit-link">' +
+            ' <a href="#" class="editable-message-last-edit-link">' +
             '(last edit a moment ago):' +
             '</a>');
 
diff --git a/lib/lp/services/messages/javascript/tests/test_messages.edit.js b/lib/lp/services/messages/javascript/tests/test_messages.edit.js
index 5d12dab..f5cbd39 100644
--- a/lib/lp/services/messages/javascript/tests/test_messages.edit.js
+++ b/lib/lp/services/messages/javascript/tests/test_messages.edit.js
@@ -169,7 +169,7 @@ YUI.add('lp.services.messages.edit.test', function(Y) {
             // Check that the "last edit" header changed.
             Y.Assert.areSame(":", this.last_edit[0].getDOMNode().innerHTML);
             Y.Assert.areSame(
-                '<a href="#" class="editable-message-last-edit-link">' +
+                ' <a href="#" class="editable-message-last-edit-link">' +
                 '(last edit a moment ago):</a>',
                 this.last_edit[1].getDOMNode().innerHTML);
         },