- remove offline message list generation on IE, this doesn't realy improve performance now

release-0.6
alecpl 15 years ago
parent a4a270be84
commit e99d21ba05

@ -1690,13 +1690,6 @@ function rcube_webmail()
} }
}; };
// messages list handling in background (for performance)
this.offline_message_list = function(flag)
{
if (this.message_list)
this.message_list.set_background_mode(flag);
};
this.set_list_sorting = function(sort_col, sort_order) this.set_list_sorting = function(sort_col, sort_order)
{ {
// set table header class // set table header class

@ -72,7 +72,7 @@ foreach ($a_mailboxes as $mbox_name) {
if ($all_count) { if ($all_count) {
$a_headers = $IMAP->list_headers($mbox_name, null, $_SESSION['sort_col'], $_SESSION['sort_order']); $a_headers = $IMAP->list_headers($mbox_name, null, $_SESSION['sort_col'], $_SESSION['sort_order']);
// add message rows // add message rows
rcmail_js_message_list($a_headers, false, false); rcmail_js_message_list($a_headers, false);
// remove messages that don't exists from list selection array // remove messages that don't exists from list selection array
$OUTPUT->command('update_selection'); $OUTPUT->command('update_selection');
} }

@ -221,7 +221,7 @@ function rcmail_message_list($attrib)
* return javascript commands to add rows to the message list * return javascript commands to add rows to the message list
* or to replace the whole list (IE only) * or to replace the whole list (IE only)
*/ */
function rcmail_js_message_list($a_headers, $insert_top=FALSE, $replace=TRUE, $head_replace=FALSE) function rcmail_js_message_list($a_headers, $insert_top=FALSE, $head_replace=FALSE)
{ {
global $CONFIG, $IMAP, $OUTPUT; global $CONFIG, $IMAP, $OUTPUT;
@ -251,9 +251,6 @@ function rcmail_js_message_list($a_headers, $insert_top=FALSE, $replace=TRUE, $h
if(($key = array_search('flag', $a_show_cols)) !== FALSE) if(($key = array_search('flag', $a_show_cols)) !== FALSE)
unset($a_show_cols[$key]); unset($a_show_cols[$key]);
if ($OUTPUT->browser->ie && $replace)
$OUTPUT->command('offline_message_list', true);
// loop through message headers // loop through message headers
foreach ($a_headers as $n => $header) foreach ($a_headers as $n => $header)
{ {
@ -314,9 +311,6 @@ function rcmail_js_message_list($a_headers, $insert_top=FALSE, $replace=TRUE, $h
$a_msg_flags, $a_msg_flags,
$insert_top); $insert_top);
} }
if ($browser->ie && $replace)
$OUTPUT->command('offline_message_list', false);
} }

@ -83,7 +83,7 @@ $OUTPUT->command('set_rowcount', rcmail_get_messagecount_text($count));
$OUTPUT->command('set_mailboxname', rcmail_get_mailbox_name_text()); $OUTPUT->command('set_mailboxname', rcmail_get_mailbox_name_text());
// add message rows // add message rows
rcmail_js_message_list($a_headers, FALSE, TRUE, (bool) $cols); rcmail_js_message_list($a_headers, FALSE, (bool) $cols);
if (isset($a_headers) && count($a_headers)) if (isset($a_headers) && count($a_headers))
{ {
if ($search_request) if ($search_request)

@ -114,7 +114,7 @@ if (($uids = get_input_value('_uid', RCUBE_INPUT_POST)) && ($flag = get_input_va
$a_headers = $IMAP->list_headers($mbox, NULL, $sort_col, $sort_order, $a_headers = $IMAP->list_headers($mbox, NULL, $sort_col, $sort_order,
$jump_back ? NULL : $count); $jump_back ? NULL : $count);
rcmail_js_message_list($a_headers, false, false); rcmail_js_message_list($a_headers, false);
} }
} }
} }

@ -128,7 +128,7 @@ else
$a_headers = $IMAP->list_headers($mbox, NULL, $sort_col, $sort_order, $a_headers = $IMAP->list_headers($mbox, NULL, $sort_col, $sort_order,
$jump_back ? NULL : $count); $jump_back ? NULL : $count);
rcmail_js_message_list($a_headers, false, false); rcmail_js_message_list($a_headers, false);
} }
} }

Loading…
Cancel
Save