Changeset 4782 for branches/2.3/expressoMail1_2/js/common_functions.js

Timestamp:

07/18/11 11:39:34 (13 years ago)

Author:

rafaelraymundo

Message:

Ticket #2066 - Menu de contexto não funciona corretamente em todos resultados de pesquisa

File:

• branches/2.3/expressoMail1_2/js/common_functions.js (modified) (5 diffs)

branches/2.3/expressoMail1_2/js/common_functions.js

@@ -144 +144 @@
         remove_className(tr_message, 'selected_msg');
         add_className(tr_message, 'tr_msg_unread');
+        if( document.getElementById("td_message_unseen_"+msg_number) != null )
         Element("td_message_unseen_"+msg_number).innerHTML = "<img src ='templates/"+template+"/images/unseen.gif' title='"+get_lang('Unseen')+"'>";
 }
@@ -151 +152 @@
         remove_className(msg, 'selected_msg');
         add_className(msg, 'flagged_msg');
+        if( document.getElementById("td_message_important_"+msg_number) != null )
         Element("td_message_important_"+msg_number).innerHTML = "<img src ='templates/"+template+"/images/important.gif' title='"+get_lang('Important')+"'>";
 }
@@ -158 +160 @@
         remove_className(msg, 'selected_msg');
         remove_className(msg, 'flagged_msg');
+        if( document.getElementById("td_message_important_"+msg_number) != null )
         Element("td_message_important_"+msg_number).innerHTML = "&nbsp;&nbsp;&nbsp;";
 }
@@ -954 +957 @@
                 {
                         Element("check_box_message_"+main_list[i].id).checked = true;
-                        add_className(Element(main_list[i].id), 'selected_msg selected_shortcut_msg');
+                        add_className(Element(main_list[i].id), 'selected_msg'); // one by one to add only if does not exist yet
+                        add_className(Element(main_list[i].id), 'selected_shortcut_msg'); // one by one to add only if does not exist yet
                 }
         }
@@ -962 +966 @@
                 {
                         Element("check_box_message_"+main_list[i].id).checked = false;
-                        remove_className(Element(main_list[i].id), 'selected_msg selected_shortcut_msg');
+                        remove_className(Element(main_list[i].id), 'selected_msg'); // one by one to remove
+                        remove_className(Element(main_list[i].id), 'selected_shortcut_msg'); // one by one to remove
                 }
         }