launchpad-reviewers team mailing list archive

[Jonathan Riddell <jriddell@xxxxxxxxxx>]

Jonathan Riddell has proposed merging lp:~jr/launchpad/803016-assign-to-me into lp:launchpad.

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)
Related bugs:
  Bug #803016 in Launchpad itself: ""Assign me" should be "Assign to me""
  https://bugs.launchpad.net/launchpad/+bug/803016

For more details, see:
https://code.launchpad.net/~jr/launchpad/803016-assign-to-me/+merge/66164

Grammer fix

-- 
https://code.launchpad.net/~jr/launchpad/803016-assign-to-me/+merge/66164
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~jr/launchpad/803016-assign-to-me into lp:launchpad.

=== modified file 'lib/lp/app/doc/lazr-js-widgets.txt'
--- lib/lp/app/doc/lazr-js-widgets.txt	2011-04-15 11:02:24 +0000
+++ lib/lp/app/doc/lazr-js-widgets.txt	2011-06-28 15:13:27 +0000
@@ -278,7 +278,7 @@
 *****************
 
 If the logged in user is in the defined vocabulary (only occurs with people
-type vocabularies), a link is shown "Assign me'.
+type vocabularies), a link is shown "Assign to me'.
 
 If the field is optional, a "Remove" link is shown.  The "Remove" text is
 customizable thought the "remove_button_text" parameter.

=== modified file 'lib/lp/app/javascript/widgets.js'
--- lib/lp/app/javascript/widgets.js	2011-06-23 02:43:46 +0000
+++ lib/lp/app/javascript/widgets.js	2011-06-28 15:13:27 +0000
@@ -122,7 +122,7 @@
         this.constructor.superclass.renderUI.call(this);
         var remove_button, assign_me_button;
         var remove_button_text = "Remove assignee";
-        var assign_me_button_text = "Assign me";
+        var assign_me_button_text = "Assign to me";
 
         if (this._show_remove_button) {
             remove_button = Y.Node.create(