Chris January wrote: > Assume imap_thread_algorithm='REFS'. > Sometimes you want to sort by the date of the last message received in a > thread (i.e. index_sort=true, message_sort_col='date') and sometimes you > want to sort by the date of the first message in a thread (i.e. > index_sort=false). At present the user must go into settings to turn > index_sort on/off which is unintuitive. This is the reason I argued for a > 4-state Date column.
I think 4-state switch isn't more intuitive. >> 5. In mail steps use short $IMAP instead of >> rcmail::get_instance()->imap->threading. Just for better readability. > > Agreed, but rcmail.php says: > * @todo Remove global $IMAP > and > // set global object for backward compatibility > $GLOBALS['IMAP'] = $this->imap; Yes, but I think it's for roundcube classes. In steps scripts we can still use shortest $IMAP or at least $RCMAIL->imap. -- Aleksander 'A.L.E.C' Machniak http://alec.pl gg:2275252 LAN Management System Developer http://lms.org.pl Roundcube Webmail Project Developer http://roundcube.net _______________________________________________ List info: http://lists.roundcube.net/dev/
