From 1dc16ebf67b058663d0423f0eef10d19918584f5 Mon Sep 17 00:00:00 2001
From: "Yogesh (OpenERP)" <ysa@tinyerp.com>
Date: Thu, 21 Jul 2011 15:25:08 +0530
Subject: [PATCH] [IMP] remove double click on a tag and add double click
 method of tr tag.

bzr revid: ysa@tinyerp.com-20110721095508-v5hdi94amw8o8w2n
---
 addons/base_export/static/src/css/base_export.css |  1 -
 addons/base_export/static/src/js/base_export.js   | 11 +++++++----
 addons/base_export/static/src/xml/base_export.xml |  2 +-
 3 files changed, 8 insertions(+), 6 deletions(-)

diff --git a/addons/base_export/static/src/css/base_export.css b/addons/base_export/static/src/css/base_export.css
index 198db30b5d0..93d2fce7dec 100644
--- a/addons/base_export/static/src/css/base_export.css
+++ b/addons/base_export/static/src/css/base_export.css
@@ -10,7 +10,6 @@ tr.ui-selected td {
 
 .row:hover{
     background-color: #F3F3F3;
-    color : blue;
 }
 
 .fields-selector-export {
diff --git a/addons/base_export/static/src/js/base_export.js b/addons/base_export/static/src/js/base_export.js
index b00da47e924..da78d3fe5bd 100644
--- a/addons/base_export/static/src/js/base_export.js
+++ b/addons/base_export/static/src/js/base_export.js
@@ -230,6 +230,7 @@ openerp.base_export.Export = openerp.base.Dialog.extend({
                     $(this).find('a').focus();
                     $(this).addClass("ui-selected");
                 }
+                return false;
             });
 
             $("tr[id^='treerow-" + record.id + "']").keydown(function (e) {
@@ -266,11 +267,13 @@ openerp.base_export.Export = openerp.base.Dialog.extend({
                     break;
                 }
             });
+	        $("tr[id^='treerow-" + record.id + "']").dblclick(function (e) {
+                var field_id =  $(this).find("a").attr("id");
+                if(field_id){
+	               self.add_field(field_id.split('-')[1], $(this).find("a").attr("string"))
+	            }
+	        });
         });
-        $('[id^=export-]').dblclick(function(){
-            self.add_field(this.id.split('-')[1], this.text)
-        });
-
         $('#fields_list').mouseover(function(event){
             if(event.relatedTarget){
                 if (event.relatedTarget.attributes['id'] && event.relatedTarget.attributes['string']){
diff --git a/addons/base_export/static/src/xml/base_export.xml b/addons/base_export/static/src/xml/base_export.xml
index 84bdf683529..9b63b8d944a 100644
--- a/addons/base_export/static/src/xml/base_export.xml
+++ b/addons/base_export/static/src/xml/base_export.xml
@@ -99,7 +99,7 @@
                                 </t>
                             </td>
                             <td valign="top" align="left" style="cursor: pointer;">
-                                <a t-att-id="'export-' + field.id"  t-att-string="field.string" href="javascript: void(0)" style="text-decoration: none;">
+                                <a t-att-id="'export-' + field.id"  t-att-string="field.string" href="javascript: void(0);" style="text-decoration: none;">
                                     <t t-esc="field.string"/>
                                 </a>
                             </td>