Fix double http request regression (#5633)

Removed 'click' event handler on messages list which was used only
to allow message re-selection. As this feature was broken since 0.9
it looks like we don't really need it. When you're in ctrl-selection
state you can easily ctrl-unselect the message.
pull/5639/head
Aleksander Machniak 8 years ago
parent 8efd2003aa
commit 3e427b4a42

@ -230,7 +230,6 @@ function rcube_webmail()
this.message_list this.message_list
.addEventListener('initrow', function(o) { ref.init_message_row(o); }) .addEventListener('initrow', function(o) { ref.init_message_row(o); })
.addEventListener('dblclick', function(o) { ref.msglist_dbl_click(o); }) .addEventListener('dblclick', function(o) { ref.msglist_dbl_click(o); })
.addEventListener('click', function(o) { ref.msglist_click(o); })
.addEventListener('keypress', function(o) { ref.msglist_keypress(o); }) .addEventListener('keypress', function(o) { ref.msglist_keypress(o); })
.addEventListener('select', function(o) { ref.msglist_select(o); }) .addEventListener('select', function(o) { ref.msglist_select(o); })
.addEventListener('dragstart', function(o) { ref.drag_start(o); }) .addEventListener('dragstart', function(o) { ref.drag_start(o); })
@ -1853,25 +1852,6 @@ function rcube_webmail()
this.show_contentframe(false); this.show_contentframe(false);
}; };
// This allow as to re-select selected message and display it in preview frame
this.msglist_click = function(list)
{
if (list.multi_selecting || !this.env.contentframe)
return;
if (!list.get_single_selection())
return;
var win = this.get_frame_window(this.env.contentframe);
if (win && win.location.href.indexOf(this.env.blankpage) >= 0) {
if (this.preview_timer)
clearTimeout(this.preview_timer);
this.preview_timer = setTimeout(function() { ref.msglist_get_preview(); }, list.dblclick_time);
}
};
this.msglist_dbl_click = function(list) this.msglist_dbl_click = function(list)
{ {
if (this.preview_timer) if (this.preview_timer)

Loading…
Cancel
Save