Merge lp:~danilo/launchpad/bug-487601 into lp:launchpad/db-devel

[Данило Шеган]

Fix up JS for templates so the windmill test passes.

[Данило Шеган (danilo)]

= Bug #487601 =

Fix broken JS so the failing test passes: just s/template_table/templates_table/ and setting event on the correct object: TR.template_row vs. div.template_links (as it was before YUI3 migration: r8693).

= Tests =

bin/test --layer=TranslationsWindmillLayer -vvt productseries_templates

= Demo & QA =

https://translations.launchpad.dev/evolution/trunk/+templates
https://translations.staging.launchpad.net/openobject-addons/trunk/+templates

[Māris Fogels (mars)]

Hi Danilo,

Looks good to me, r=mars.

Maris

review: Approve

=== modified file 'lib/lp/translations/templates/object-templates.pt'
--- lib/lp/translations/templates/object-templates.pt	2009-11-24 06:08:55 +0000
+++ lib/lp/translations/templates/object-templates.pt	2009-11-24 19:33:13 +0000
@@ -38,15 +38,16 @@
       YUI().use('node-base', 'event-delegate', function(Y) {
           Y.on('domready', function(e) {
               Y.all('#templates_table .template_links').addClass(
-                  'inactive_links');
-
-              Y.delegate('mouseover', function(e) {
-                  this.removeClass('inactive_links');
-              }, '#templates_table tbody', '.template_links');
-
-              Y.delegate('mouseout', function(e) {
-                  this.addClass('inactive_links');
-              }, '#templates_table tbody', '.template_links');
+                    'inactive_links');
+              Y.all('#templates_table .template_row').each(function(row) {
+                  var template_links = row.one('.template_links');
+                  row.on('mouseover', function(e) {
+                      template_links.removeClass('inactive_links');
+                  });
+                  row.on('mouseout', function(e) {
+                      template_links.addClass('inactive_links');
+                  });
+              });
           });
       });
       </script>