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);
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. translations/ templates/ pofile- translate. pt' translations/ templates/ pofile- translate. pt 2010-03-18 17:01:11 +0000 translations/ templates/ pofile- translate. pt 2010-03-22 14:59:42 +0000
registerLaunchp adFunction( insertAllExpans ionButtons) ;
=== modified file 'lib/lp/
--- lib/lp/
+++ lib/lp/
@@ -18,31 +18,41 @@
- try {
- Y.on('domready', Y.lp.pofile.
- } catch (e) {
- Y.log(e, "error");
- }
- try {
- Y.on('domready', Y.lp.pofile.
- } catch (e) {
- Y.log(e, "error");
- }
- try {
- Y.on('domready', Y.lp.pofile.
- } catch (e) {
- Y.log(e, "error");
- }
- try {
- Y.on('domready', function(e) {
- Y.lp.pofile.
- });
- } catch (e) {
- Y.log(e, "error");
- }
+
+ Y.on('domready', function(e) {
+ try {
+ Y.lp.pofile.
+ } catch (e) {
+ Y.log(e, "error");
+ }
+ });
+
+ Y.on('domready', function(e) {
+ try {
+ Y.lp.pofile.
+ } catch (e) {
+ Y.log(e, "error");
+ }
+ });
+
+ Y.on('domready', function(e) {
+ try {
+ Y.lp.pofile.
+ } catch (e) {
+ Y.log(e, "error");
+ }
+ });
+
+ Y.on('domready', function(e) {
+ try {
+ Y.lp.pofile.
+ } catch (e) {
+ Y.log(e, "error");
+ }
+ });
+
});
- slot="main" >
</script>
</div>
<div metal:fill-
=== modified file 'lib/lp/ translations/ templates/ translationmess age-translate. pt' translations/ templates/ translationmess age-translate. pt 2010-03-18 17:01:11 +0000 translations/ templates/ translationmess age-translate. pt 2010-03-22 14:57:08 +0000 javascript" >
LPS.use( 'node', 'lp.pofile', function(Y) { setupSuggestion Dismissal) ; initializeKeyBi ndings) ; setFocus( autofocus_ field); setupSuggestion Dismissal( ); initializeKeyBi ndings( ); setFocus( autofocus_ field);
--- lib/lp/
+++ lib/lp/
@@ -15,23 +15,29 @@
</style>
<script type="text/
- try {
- Y.on('domready', Y.lp.pofile.
- } catch (e) {
- Y.log(e, "error");
- }
- try {
- Y.on('domready', Y.lp.pofile.
- } catch (e) {
- Y.log(e, "error");
- }
- try {
- Y.on('domready', function(e) {
- Y.lp.pofile.
- });
- } catch (e) {
- Y.log(e, "error");
- }
+ Y.on('domready', function(e) {
+ try {
+ Y.lp.pofile.
+ } catch (e) {
+ Y.log(e, "error");
+ }
+ });
+
+ Y.on('domready', function(e) {
+ try {
+ Y.lp.pofile.
+ } catch (e) {
+ Y.log(e, "error");
+ }
+ });
+
+ Y.on('domready', function(e) {
+ try {
+ Y.lp.pofile.
+ } catch (e) {
+ Y.log(e, "error");
+ }
+ });
});
</script>
</div>