Merge lp:~adiroiban/launchpad/bug-540105 into lp:launchpad

[Adi Roiban]

Move independent functionality in different JS sandboxes and fix
pofile.js updateNotificationBox() to not fail when there are no notification boxes on the page.

= Bug 540105 =
I noticed this behaviour in edge:

When entering text in the input box for a translation, this textbox does no longer get the focus by automatically selecting the checkbox dot on its left.

This leads to translators not noticing it, pressing "Save" and loosing all their translations.

== Proposed fix ==
Move independent functionality in different JS sandboxes and fix
pofile.js updateNotificationBox() to not fail when there are no notification boxes on the page.

== Pre-implementation notes ==
This branch is a hotfix for this problem.

I have opened bug 540216 and working on improving the general test coverage for pofile.js code.

== Implementation details ==
I have also fixes some other lint warnings.

== Tests ==
lp-test -t pofile_new_translation_autoselect

== Demo and Q/A ==
Login as admin and go to:
https://translations.launchpad.dev/evolution/trunk/+pots/evolution-2.2/es/+translate

In the "New translation" field, enter some text. The radio button before the text field should be now selected.

= Launchpad lint =

Checking for conflicts. and issues in doctests and templates.
Running jslint, xmllint, pyflakes, and pylint.
Using normal rules.

Linting changed files:
  lib/canonical/launchpad/javascript/translations/pofile.js

== JSLint notices ==
No handlers could be found for logger "bzr"
jslint: Lint found in '/home/adi/launchpad/lp-branches/bug-540105/lib/canonical/launchpad/javascript/translations/pofile.js':
Line 303 character 18: Be careful when making functions within a loop. Consider putting the function in a closure.
                },

Line 310 character 18: Be careful when making functions within a loop. Consider putting the function in a closure.
                }, '#' + fields[key], 'down:68+shift+alt', Y, original_stem);

Line 316 character 18: Be careful when making functions within a loop. Consider putting the function in a closure.
                }, '#' + fields[key], 'down:48+shift+alt', Y, fields[key]);

jslint: 1 file to lint.

[Edwin Grubbs (edwin-grubbs)]

Hi Adi,

This is a good improvement. I ran into an annoying problem running the tests.
The tests use the combined javascript file, but "make jsbuild" was not getting
run automatically. I also noticed that translationmessage-translate.pt and
pofile-translate.pt load pofile.js directly, when the <script> tag really
should be added to lib/lp/app/templates/base-layout-macros.pt next to all
the other js files in the ${lp_js} directory.

Since I was confused by some of your changes that I don't think were
necessary, I'm marking this
    needs-fixing

-Edwin

>=== modified file 'lib/canonical/launchpad/javascript/translations/pofile.js'
>--- lib/canonical/launchpad/javascript/translations/pofile.js 2010-02-25 12:56:45 +0000
>+++ lib/canonical/launchpad/javascript/translations/pofile.js 2010-03-17 17:36:08 +0000
>@@ -55,10 +55,15 @@
> node.setStyle('display', 'none');
> });
> hide_anim.run();
>-}
>+};
>
> self.updateNotificationBox = function(e) {
> var notice = Y.one('.important-notice-container');
>+ if (!notice) {

Since Y.one() is expected to return null when the element is not found,
the preferred if-statement is:
    if (notice !== null)

>+ // We have no notice container on this page, this is why there is
>+ // nothing more to be done by this function.
>+ return;
>+ }
> var balloon = notice.one('.important-notice-balloon');
> var dismiss_notice_cookie = ('translation-docs-for-' +
> documentation_cookie);
>@@ -74,6 +79,10 @@
> '.important-notice-cancel-button');
> // Cancel button starts out hidden. If user has JavaScript,
> // then we want to show it.
>+ if (!cancel_button) {

Same here.

>+ // No cancel button was found to attach the action.
>+ return;
>+ }
> cancel_button.setStyle('visibility', 'visible');
> cancel_button.on('click', function(e) {
> e.halt();
>@@ -202,30 +211,34 @@
> }
> // Shift+Alt+n - Go to next page in batch
> if (e.shiftKey && e.altKey && e.keyCode == 78) {
>- if (link = Y.one('#batchnav_next')){
>- window.location.assign(link.get('href'))
>+ link = Y.one('#batchnav_next');
>+ if (link){

Same for these if-statements.

>+ window.location.assign(link.get('href'));
> }
> }
> // Shift+Alt+p - Go to previous page in batch
> if (e.shiftKey && e.altKey && e.keyCode == 80) {
>- if (link = Y.one('#batchnav_previous')){
>- window.location.assign(link.get('href'))
>+ link = Y.one('#batchnav_previous');
>+ if (link){
>+ window.location.assign(link.get('href'));
> }
> }
> // Shift+Alt+a - Go to first page in batch
> if (e.shiftKey && e.altKey && e.keyCode == 65) {
>- if (link = Y.one('#batchnav_first')){
>- window.location.assign(link.get('href'))
>+ link = Y.one('#batchnav_first');
>+ if (link){
>+ window.location.assign(link.get('href'));
> }
> }
> // Shift+Alt+l - Go to last page in batch
> if (e.shiftKey && e.altKey && e.keyCode == 76) {
>- if (link = Y.one('#batchnav_last')){
>- ...

review: Needs Fixing (code)

[Adi Roiban (adiroiban)]

Hi,
> Hi Adi,
>
> This is a good improvement. I ran into an annoying problem running the tests.
> The tests use the combined javascript file, but "make jsbuild" was not getting
> run automatically. I also noticed that translationmessage-translate.pt and
> pofile-translate.pt load pofile.js directly, when the <script> tag really
> should be added to lib/lp/app/templates/base-layout-macros.pt next to all
> the other js files in the ${lp_js} directory.

I have moved the pofile.js inclusing in base-layout-macros.pt.

> Since I was confused by some of your changes that I don't think were
> necessary, I'm marking this
> needs-fixing

Thanks for the review

[snip]

> > var notice = Y.one('.important-notice-container');
> >+ if (!notice) {
>
>
> Since Y.one() is expected to return null when the element is not found,
> the preferred if-statement is:
> if (notice !== null)
Thanks for this remark.

I have changed the if statement to explicitly test agains null.

Same for the other cases.

[snip]
> >+ LPS.use('lp.pofile', function(Y) {
> > Y.on('domready', Y.lp.pofile.setupSuggestionDismissal);
> >- Y.on('domready', Y.lp.pofile.updateNotificationBox);
> >+ });
> >+
> >+ LPS.use('lp.pofile', function(Y) {
>
>
> Why are you calling LPS.use('lp.pofile', ...) repeatedly? This
> should have no effect unless there is something seriously wrong with
> lp.pofile.
I moved them back in the same sandbox.
I split independent functions in different sandboxes so that the failure of one function to not affect the others.

Here is the latest diff:

=== modified file 'lib/canonical/launchpad/javascript/translations/pofile.js'
--- lib/canonical/launchpad/javascript/translations/pofile.js 2010-03-17 13:11:23 +0000
+++ lib/canonical/launchpad/javascript/translations/pofile.js 2010-03-17 22:18:24 +0000
@@ -59,7 +59,7 @@

 self.updateNotificationBox = function(e) {
   var notice = Y.one('.important-notice-container');
- if (!notice) {
+ if (notice == null) {
     // We have no notice container on this page, this is why there is
     // nothing more to be done by this function.
     return;
@@ -71,7 +71,7 @@
   // Check the cookie to see if the user has already dismissed
   // the notification box for this session.
   var already_seen = Y.Cookie.get(dismiss_notice_cookie, Boolean);
- if (already_seen) {
+ if (already_seen !== null) {
      notice.setStyle('display', 'none');
   }

@@ -79,7 +79,7 @@
       '.important-notice-cancel-button');
   // Cancel button starts out hidden. If user has JavaScript,
   // then we want to show it.
- if (!cancel_button) {
+ if (cancel_button == null) {
     // No cancel button was found to attach the action.
     return;
   }
@@ -94,7 +94,7 @@

 self.setFocus = function(field) {
     // if there is nofield, do nothing
- if (Y.one('#' + field)) {
+ if (Y.one('#' + field) !== null) {
         Y.one('#' + field).focus();
     }
 };
@@ -151,7 +151,7 @@
         original_singular, translation_singular, singular_select);

     // Copy plural text if needed
- if (Y.one('#' + translation_plural + '1')) {
+ if (Y.one('#' + translation_plural + '1') !== null) {
         copyOrigi...

[Edwin Grubbs (edwin-grubbs)]

> > Why are you calling LPS.use('lp.pofile', ...) repeatedly? This
> > should have no effect unless there is something seriously wrong with
> > lp.pofile.
> I moved them back in the same sandbox.
> I split independent functions in different sandboxes so that the failure of
> one function to not affect the others.

Hi Adi,

All your changes look good. Now I understand what you were trying to do with the separate LPS calls. Since I wasn't completely certain how far up the stack an exception goes before it continues running the rest of the javascript in the page, I played around with it a little bit. Apparently, an exception halts execution of a <script> block, so you would have to have multiple <script> blocks around each LPS.use() in order to ensure that the following code is run despite an exception above it. A much clearer way to handle that would be:

LPS.use('lp.pofile', function(Y) {
  try {
    do_something();
  } catch (e) {
    Y.log(e, "error");
  }
  do_something_else();
});

At least in firefox, if you call Y.log() with the second argument as "error", you will get a full stack trace in the console just like a normal uncaught exception.

Using try/catch is completely up to you.

 merge-approved

BTW, calling LPS.use() multiple times does not give you separate sandboxes, since you are re-using LPS instead of creating a new YUI() object.

-Edwin

review: Approve (code)

[Adi Roiban (adiroiban)]

I added try/catch statements.

Many thanks for your review.

Do you have time to send this branch to ec2 test and land, or should I ask the ORC?

Here is the latest diff.
=== modified file 'lib/lp/translations/templates/pofile-translate.pt'
--- lib/lp/translations/templates/pofile-translate.pt 2010-03-17 23:17:46 +0000
+++ lib/lp/translations/templates/pofile-translate.pt 2010-03-18 16:26:48 +0000
@@ -18,12 +18,28 @@
       registerLaunchpadFunction(insertAllExpansionButtons);

       LPS.use('lp.pofile', function(Y) {
+ try {
           Y.on('domready', Y.lp.pofile.updateNotificationBox);
+ } catch (e) {
+ Y.log(e, "error");
+ }
+ try {
           Y.on('domready', Y.lp.pofile.setupSuggestionDismissal);
+ } catch (e) {
+ Y.log(e, "error");
+ }
+ try {
           Y.on('domready', Y.lp.pofile.initializeKeyBindings);
+ } catch (e) {
+ Y.log(e, "error");
+ }
+ try {
           Y.on('domready', function(e) {
             Y.lp.pofile.setFocus(autofocus_field);
           });
+ } catch (e) {
+ Y.log(e, "error");
+ }
       });

=== modified file 'lib/lp/translations/templates/translationmessage-translate.pt'
--- lib/lp/translations/templates/translationmessage-translate.pt 2010-03-17 23:17:46 +0000
+++ lib/lp/translations/templates/translationmessage-translate.pt 2010-03-18 16:25:45 +0000
@@ -15,11 +15,23 @@
     </style>
     <script type="text/javascript">
       LPS.use('node', 'lp.pofile', function(Y) {
+ try {
           Y.on('domready', Y.lp.pofile.setupSuggestionDismissal);
+ } catch (e) {
+ Y.log(e, "error");
+ }
+ try {
           Y.on('domready', Y.lp.pofile.initializeKeyBindings);
+ } catch (e) {
+ Y.log(e, "error");
+ }
+ try {
           Y.on('domready', function(e) {
             Y.lp.pofile.setFocus(autofocus_field);
           });
+ } catch (e) {
+ Y.log(e, "error");
+ }
       });
     </script>
     </div>

[Adi Roiban (adiroiban)]

Hi Edwin,

I moved the try/catch inside the domready function call.

I did some testing on Firefox and Chromium on Ubuntu Karmic, and without the try/catch, a failing domready function call will prevent the loading of all other domready functions.

Here is the latest review.
=== modified file 'lib/lp/translations/templates/pofile-translate.pt'
--- lib/lp/translations/templates/pofile-translate.pt 2010-03-18 17:01:11 +0000
+++ lib/lp/translations/templates/pofile-translate.pt 2010-03-22 14:59:42 +0000
@@ -18,31 +18,41 @@
       registerLaunchpadFunction(insertAllExpansionButtons);

       LPS.use('lp.pofile', function(Y) {
- try {
- Y.on('domready', Y.lp.pofile.updateNotificationBox);
- } catch (e) {
- Y.log(e, "error");
- }
- try {
- Y.on('domready', Y.lp.pofile.setupSuggestionDismissal);
- } catch (e) {
- Y.log(e, "error");
- }
- try {
- Y.on('domready', Y.lp.pofile.initializeKeyBindings);
- } catch (e) {
- Y.log(e, "error");
- }
- try {
- Y.on('domready', function(e) {
- Y.lp.pofile.setFocus(autofocus_field);
- });
- } catch (e) {
- Y.log(e, "error");
- }
+
+ Y.on('domready', function(e) {
+ try {
+ Y.lp.pofile.updateNotificationBox();
+ } catch (e) {
+ Y.log(e, "error");
+ }
+ });
+
+ Y.on('domready', function(e) {
+ try {
+ Y.lp.pofile.setupSuggestionDismissal();
+ } catch (e) {
+ Y.log(e, "error");
+ }
+ });
+
+ Y.on('domready', function(e) {
+ try {
+ Y.lp.pofile.initializeKeyBindings();
+ } catch (e) {
+ Y.log(e, "error");
+ }
+ });
+
+ Y.on('domready', function(e) {
+ try {
+ Y.lp.pofile.setFocus(autofocus_field);
+ } catch (e) {
+ Y.log(e, "error");
+ }
+ });
+
       });

-
     </script>
     </div>
     <div metal:fill-slot="main">

=== modified file 'lib/lp/translations/templates/translationmessage-translate.pt'
--- lib/lp/translations/templates/translationmessage-translate.pt 2010-03-18 17:01:11 +0000
+++ lib/lp/translations/templates/translationmessage-translate.pt 2010-03-22 14:57:08 +0000
@@ -15,23 +15,29 @@
     </style>
     <script type="text/javascript">
       LPS.use('node', 'lp.pofile', function(Y) {
- try {
- Y.on('domready', Y.lp.pofile.setupSuggestionDismissal);
- } catch (e) {
- Y.log(e, "error");
- }
- try {
- Y.on('domready', Y.lp.pofile.initializeKeyBindings);
- } catch (e) {
- Y.log(e, "error");
- }
- try {
- Y.on('domready', function(e) {
- Y.lp.pofile.setFocus(autofocus_field);
- });
- } catch (e) {
- Y.log(e, "error");
- }
+ Y.on('domready', function(e) {
+ try {
+ Y.lp.pofile.setupSuggestionDismissal();
+ } catch (e) {
+ Y.log(e, "error");
...

[Adi Roiban (adiroiban)]

Hi Edwin and thanks for the review!

I moved the try/catch in a single domready block, declared link as a local varibaled and used === to compare for null.

Here is the latests diff:
=== modified file 'lib/canonical/launchpad/javascript/translations/pofile.js'
--- lib/canonical/launchpad/javascript/translations/pofile.js 2010-03-22 14:59:40 +0000
+++ lib/canonical/launchpad/javascript/translations/pofile.js 2010-03-22 17:43:04 +0000
@@ -59,7 +59,7 @@

 self.updateNotificationBox = function(e) {
   var notice = Y.one('.important-notice-container');
- if (notice == null) {
+ if (notice === null) {
     // We have no notice container on this page, this is why there is
     // nothing more to be done by this function.
     return;
@@ -79,7 +79,7 @@
       '.important-notice-cancel-button');
   // Cancel button starts out hidden. If user has JavaScript,
   // then we want to show it.
- if (cancel_button == null) {
+ if (cancel_button === null) {
     // No cancel button was found to attach the action.
     return;
   }
@@ -197,6 +197,7 @@
 var initializeGlobalKeyBindings = function(fields) {

     Y.get('document').on("keyup", function(e) {
+ var link;
         // Shift+Alt+s - Save form
         if (e.shiftKey && e.altKey && e.keyCode == 83) {
             Y.one('#save_and_continue_button').invoke('click');

=== modified file 'lib/lp/translations/templates/pofile-translate.pt'
--- lib/lp/translations/templates/pofile-translate.pt 2010-03-22 14:59:44 +0000
+++ lib/lp/translations/templates/pofile-translate.pt 2010-03-22 17:34:27 +0000
@@ -25,25 +25,19 @@
             } catch (e) {
               Y.log(e, "error");
             }
- });

- Y.on('domready', function(e) {
             try {
               Y.lp.pofile.setupSuggestionDismissal();
             } catch (e) {
               Y.log(e, "error");
             }
- });

- Y.on('domready', function(e) {
             try {
               Y.lp.pofile.initializeKeyBindings();
             } catch (e) {
               Y.log(e, "error");
             }
- });

- Y.on('domready', function(e) {
             try {
               Y.lp.pofile.setFocus(autofocus_field);
             } catch (e) {

=== modified file 'lib/lp/translations/templates/translationmessage-translate.pt'
--- lib/lp/translations/templates/translationmessage-translate.pt 2010-03-22 14:50:08 +0000
+++ lib/lp/translations/templates/translationmessage-translate.pt 2010-03-22 17:35:02 +0000
@@ -21,17 +21,13 @@
             } catch (e) {
               Y.log(e, "error");
             }
- });

- Y.on('domready', function(e) {
             try {
               Y.lp.pofile.initializeKeyBindings();
             } catch (e) {
               Y.log(e, "error");
             }
- });

- Y.on('domready', function(e) {
             try {
               Y.lp.pofile.setFocus(autofocus_field);
             } catch (e) {

=== modified file 'lib/canonical/launchpad/javascript/translations/pofile.js'
--- lib/canonical/launchpad/javascript/translations/pofile.js	2010-02-25 12:56:45 +0000
+++ lib/canonical/launchpad/javascript/translations/pofile.js	2010-03-22 17:49:29 +0000
@@ -55,10 +55,15 @@
       node.setStyle('display', 'none');
   });
   hide_anim.run();
-}
+};
 
 self.updateNotificationBox = function(e) {
   var notice = Y.one('.important-notice-container');
+  if (notice === null) {
+    // We have no notice container on this page, this is why there is
+    // nothing more to be done by this function.
+    return;
+  }
   var balloon = notice.one('.important-notice-balloon');
   var dismiss_notice_cookie = ('translation-docs-for-' +
                                documentation_cookie);
@@ -66,7 +71,7 @@
   // Check the cookie to see if the user has already dismissed
   // the notification box for this session.
   var already_seen = Y.Cookie.get(dismiss_notice_cookie, Boolean);
-  if (already_seen) {
+  if (already_seen !== null) {
      notice.setStyle('display', 'none');
   }
 
@@ -74,6 +79,10 @@
       '.important-notice-cancel-button');
   // Cancel button starts out hidden.  If user has JavaScript,
   // then we want to show it.
+  if (cancel_button === null) {
+    // No cancel button was found to attach the action.
+    return;
+  }
   cancel_button.setStyle('visibility', 'visible');
   cancel_button.on('click', function(e) {
       e.halt();
@@ -85,7 +94,7 @@
 
 self.setFocus = function(field) {
     // if there is nofield, do nothing
-    if (Y.one('#' + field)) {
+    if (Y.one('#' + field) !== null) {
         Y.one('#' + field).focus();
     }
 };
@@ -142,7 +151,7 @@
         original_singular, translation_singular, singular_select);
 
     // Copy plural text if needed
-    if (Y.one('#' + translation_plural + '1')) {
+    if (Y.one('#' + translation_plural + '1') !== null) {
         copyOriginalTextPlural(
             original_plural, translation_plural, plural_forms);
     }
@@ -153,7 +162,7 @@
 
 var selectWidgetByID = function(widget) {
     var node = Y.one('#' + widget);
-    if (node) {
+    if (node !== null) {
         node.set('checked', true);
     }
 };
@@ -161,7 +170,7 @@
 
 var toggleWidget = function(widget) {
     var node = Y.one('#' + widget);
-    if (node) {
+    if (node !== null) {
         if (node.get('checked')) {
             node.set('checked', false);
         } else {
@@ -187,45 +196,50 @@
 
 var initializeGlobalKeyBindings = function(fields) {
 
-	Y.get('document').on("keyup", function(e) {
+    Y.get('document').on("keyup", function(e) {
+        var link;
         // Shift+Alt+s - Save form
-	    if (e.shiftKey && e.altKey && e.keyCode == 83) {
-	        Y.one('#save_and_continue_button').invoke('click');
-	    }
+        if (e.shiftKey && e.altKey && e.keyCode == 83) {
+            Y.one('#save_and_continue_button').invoke('click');
+        }
         // Shift+Alt+f - Go to search field
-	    if (e.shiftKey && e.altKey && e.keyCode == 70) {
-	        self.setFocus('search_box');
-	    }
+        if (e.shiftKey && e.altKey && e.keyCode == 70) {
+            self.setFocus('search_box');
+        }
         // Shift+Alt+b - Go to first translation field
-	    if (e.shiftKey && e.altKey && e.keyCode == 66) {
-	        self.setFocus(fields[0]);
-	    }
+        if (e.shiftKey && e.altKey && e.keyCode == 66) {
+            self.setFocus(fields[0]);
+        }
         // Shift+Alt+n - Go to next page in batch
-	    if (e.shiftKey && e.altKey && e.keyCode == 78) {
-	        if (link = Y.one('#batchnav_next')){
-	            window.location.assign(link.get('href'))
-	        }
-	    }
+        if (e.shiftKey && e.altKey && e.keyCode == 78) {
+            link = Y.one('#batchnav_next');
+            if (link !== null){
+                window.location.assign(link.get('href'));
+            }
+        }
         // Shift+Alt+p - Go to previous page in batch
-	    if (e.shiftKey && e.altKey && e.keyCode == 80) {
-	        if (link = Y.one('#batchnav_previous')){
-	            window.location.assign(link.get('href'))
-	        }
-	    }
+        if (e.shiftKey && e.altKey && e.keyCode == 80) {
+            link = Y.one('#batchnav_previous');
+            if (link !== null){
+                window.location.assign(link.get('href'));
+            }
+        }
         // Shift+Alt+a - Go to first page in batch
-	    if (e.shiftKey && e.altKey && e.keyCode == 65) {
-	        if (link = Y.one('#batchnav_first')){
-	            window.location.assign(link.get('href'))
-	        }
-	    }
+        if (e.shiftKey && e.altKey && e.keyCode == 65) {
+            link = Y.one('#batchnav_first');
+            if (link !== null){
+                window.location.assign(link.get('href'));
+            }
+        }
         // Shift+Alt+l - Go to last page in batch
-	    if (e.shiftKey && e.altKey && e.keyCode == 76) {
-	        if (link = Y.one('#batchnav_last')){
-	            window.location.assign(link.get('href'))
-	        }
-	    }
-	});
-}
+        if (e.shiftKey && e.altKey && e.keyCode == 76) {
+            link = Y.one('#batchnav_last');
+            if (link !== null){
+                window.location.assign(link.get('href'));
+            }
+        }
+    });
+};
 
 
 var initializeSuggestionsKeyBindings = function(stem) {
@@ -243,7 +257,7 @@
                 Y, node.get('id'));
         }
     });
-}
+};
 
 
 var initializeFieldsKeyBindings = function (fields) {
@@ -321,7 +335,7 @@
                 singular_copy_text);
         }
     }
-}
+};
 
 
 /**

=== modified file 'lib/lp/app/templates/base-layout-macros.pt'
--- lib/lp/app/templates/base-layout-macros.pt	2010-03-15 17:17:05 +0000
+++ lib/lp/app/templates/base-layout-macros.pt	2010-03-22 17:49:29 +0000
@@ -211,6 +211,9 @@
     <script type="text/javascript"
             tal:attributes="src string:${lp_js}/registry/team.js"></script>
 
+    <script type="text/javascript"
+            tal:attributes="src string:${lp_js}/translations/pofile.js"></script>
+
   </tal:devmode>
   <tal:production condition="not:devmode">
     <script type="text/javascript"

=== modified file 'lib/lp/translations/templates/pofile-translate.pt'
--- lib/lp/translations/templates/pofile-translate.pt	2010-02-25 00:47:17 +0000
+++ lib/lp/translations/templates/pofile-translate.pt	2010-03-22 17:49:29 +0000
@@ -14,21 +14,39 @@
     }
     </style>
 
-    <script type="text/javascript"
-      tal:condition="devmode"
-      tal:define="lp_js string:${icingroot}/build"
-      tal:attributes="src string:${lp_js}/translations/pofile.js"></script>
     <script type="text/javascript">
       registerLaunchpadFunction(insertAllExpansionButtons);
 
       LPS.use('lp.pofile', function(Y) {
-          Y.on('domready', Y.lp.pofile.setupSuggestionDismissal);
-          Y.on('domready', Y.lp.pofile.updateNotificationBox);
-          Y.on('domready', Y.lp.pofile.initializeKeyBindings);
+
           Y.on('domready', function(e) {
-            Y.lp.pofile.setFocus(autofocus_field);
+            try {
+              Y.lp.pofile.updateNotificationBox();
+            } catch (e) {
+              Y.log(e, "error");
+            }
+
+            try {
+              Y.lp.pofile.setupSuggestionDismissal();
+            } catch (e) {
+              Y.log(e, "error");
+            }
+
+            try {
+              Y.lp.pofile.initializeKeyBindings();
+            } catch (e) {
+              Y.log(e, "error");
+            }
+
+            try {
+              Y.lp.pofile.setFocus(autofocus_field);
+            } catch (e) {
+              Y.log(e, "error");
+            }
           });
+
       });
+
     </script>
     </div>
     <div metal:fill-slot="main">

=== modified file 'lib/lp/translations/templates/translationmessage-translate.pt'
--- lib/lp/translations/templates/translationmessage-translate.pt	2010-02-25 00:47:17 +0000
+++ lib/lp/translations/templates/translationmessage-translate.pt	2010-03-22 17:49:29 +0000
@@ -13,16 +13,26 @@
         color: lightgray;
     }
     </style>
-    <script type="text/javascript"
-      tal:condition="devmode"
-      tal:define="lp_js string:${icingroot}/build"
-      tal:attributes="src string:${lp_js}/translations/pofile.js"></script>
     <script type="text/javascript">
       LPS.use('node', 'lp.pofile', function(Y) {
-          Y.on('domready', Y.lp.pofile.setupSuggestionDismissal);
-          Y.on('domready', Y.lp.pofile.initializeKeyBindings);
           Y.on('domready', function(e) {
-            Y.lp.pofile.setFocus(autofocus_field);
+            try {
+              Y.lp.pofile.setupSuggestionDismissal();
+            } catch (e) {
+              Y.log(e, "error");
+            }
+
+            try {
+              Y.lp.pofile.initializeKeyBindings();
+            } catch (e) {
+              Y.log(e, "error");
+            }
+
+            try {
+              Y.lp.pofile.setFocus(autofocus_field);
+            } catch (e) {
+              Y.log(e, "error");
+            }
           });
       });
     </script>

=== added file 'lib/lp/translations/windmill/tests/test_pofile_translate.py'
--- lib/lp/translations/windmill/tests/test_pofile_translate.py	1970-01-01 00:00:00 +0000
+++ lib/lp/translations/windmill/tests/test_pofile_translate.py	2010-03-22 17:49:29 +0000
@@ -0,0 +1,52 @@
+# Copyright 2010 Canonical Ltd.  This software is licensed under the
+# GNU Affero General Public License version 3 (see the file LICENSE).
+
+"""Tests for pofile translate pages."""
+
+__metaclass__ = type
+__all__ = []
+
+from canonical.launchpad.windmill.testing import constants, lpuser
+from lp.translations.windmill.testing import TranslationsWindmillLayer
+from lp.testing import WindmillTestCase
+
+
+class POFileNewTranslationFieldKeybindings(WindmillTestCase):
+    """Tests for keybinding actions associated to the translation field."""
+
+    layer = TranslationsWindmillLayer
+    suite_name = 'POFile Translate'
+
+    def test_pofile_new_translation_autoselect(self):
+        """Test for automatically selecting new translation on text input.
+
+        When new text is typed into the new translation text fields, the
+        associated radio button should be automatically selected.
+        """
+        client = self.client
+        start_url = ('http://translations.launchpad.dev:8085/'
+                        'evolution/trunk/+pots/evolution-2.2/es/+translate')
+        user = lpuser.TRANSLATIONS_ADMIN
+        new_translation_id = u'msgset_1_es_translation_0_new'
+        radiobutton_id = u'msgset_1_es_translation_0_new_select'
+
+        # Go to the translation page.
+        self.client.open(url=start_url)
+        self.client.waits.forPageLoad(timeout=constants.PAGE_LOAD)
+        user.ensure_login(self.client)
+
+        # Wait for the new translation field and it's associated radio button.
+        client.waits.forElement(
+            id=new_translation_id, timeout=constants.FOR_ELEMENT)
+        client.waits.forElement(
+            id=radiobutton_id, timeout=constants.FOR_ELEMENT)
+
+        # Check that the associated radio button is not selected.
+        client.asserts.assertNotChecked(id=radiobutton_id)
+
+        # Type a new translation.
+        client.type(
+            id=new_translation_id, text=u'New translation')
+
+        # Check that the associated radio button is selected.
+        client.asserts.assertChecked(id=radiobutton_id)