Changeset 21168d2 in github

Timestamp:

Mar 31, 2008 10:49:54 AM (5 years ago)

Author:

svncommit <devs@…>

Branches:

master, HEAD, courier-fix, dev-browser-capabilities, pdo, release-0.6, release-0.7, release-0.8

Children:

65444ba

Parents:

1b4d732

Message:

Don't preview message if dragging or multi-selecting it, closes #1484316.

Location:

program/js

Files:

• app.js (modified) (2 diffs)
• list.js (modified) (3 diffs)

program/js/app.js

@@ -134 +134 @@
           this.message_list.addEventListener('select', function(o){ p.msglist_select(o); });
           this.message_list.addEventListener('dragstart', function(o){ p.drag_active = true; });
+          this.message_list.addEventListener('dragstart', function(o){ p.drag_active = true; if (p.preview_timer) clearTimeout(p.preview_timer); });
           this.message_list.addEventListener('dragend', function(o){ p.drag_active = false; });
 
@@ -1139 +1140 @@
 
     // start timer for message preview (wait for double click)
-    if (selected && this.env.contentframe)
+    if (selected && this.env.contentframe && !list.multi_selecting)
       this.preview_timer = setTimeout(function(){ ref.msglist_get_preview(); }, this.dblclick_time + 10);
     else if (this.env.contentframe)

program/js/list.js

@@ -36 +36 @@
   this.shiftkey = false;
   this.multiselect = false;
+  this.multi_selecting = false;
   this.draggable = false;
   this.keyboard = false;
@@ -320 +321 @@
     this.shift_start = id;
     this.highlight_row(id, false);
+    this.multi_selecting = false;
   }
   else
@@ -342 +344 @@
         break;
     }
+    this.multi_selecting = true;
   }