fixed that filtering did not work if there were no filter rules (closes bug #472003...
[claws.git] / src / summaryview.c
index 7a2cc68e58572c8a4234c6f901bd8e3e7fca61b1..59c2b5250bb2bfe941719f7221a7197ca485b9db 100644 (file)
@@ -60,6 +60,7 @@
 #include "headerwindow.h"
 #include "sourcewindow.h"
 #include "prefs_common.h"
+#include "prefs_summary_column.h"
 #include "account.h"
 #include "compose.h"
 #include "utils.h"
@@ -153,6 +154,12 @@ static void summary_set_menu_sensitive     (SummaryView            *summaryview);
 static guint summary_get_msgnum                (SummaryView            *summaryview,
                                         GtkCTreeNode           *node);
 
+static GtkCTreeNode *summary_find_prev_msg
+                                       (SummaryView            *summaryview,
+                                        GtkCTreeNode           *current_node);
+static GtkCTreeNode *summary_find_next_msg
+                                       (SummaryView            *summaryview,
+                                        GtkCTreeNode           *current_node);
 static GtkCTreeNode *summary_find_prev_unread_msg
                                        (SummaryView            *summaryview,
                                         GtkCTreeNode           *current_node);
@@ -181,7 +188,8 @@ static void summary_update_status   (SummaryView            *summaryview);
 static void summary_status_show                (SummaryView            *summaryview);
 static void summary_set_ctree_from_list        (SummaryView            *summaryview,
                                         GSList                 *mlist);
-static void summary_set_header         (gchar                  *text[],
+static void summary_set_header         (SummaryView            *summaryview,
+                                        gchar                  *text[],
                                         MsgInfo                *msginfo);
 static void summary_display_msg                (SummaryView            *summaryview,
                                         GtkCTreeNode           *row,
@@ -225,8 +233,10 @@ static void summary_execute_delete (SummaryView            *summaryview);
 static void summary_execute_delete_func        (GtkCTree               *ctree,
                                         GtkCTreeNode           *node,
                                         gpointer                data);
-static void summary_ignore_thread(SummaryView *summaryview);
-static void summary_unignore_thread(SummaryView *summaryview);
+
+static void summary_thread_init                (SummaryView            *summaryview);
+static void summary_ignore_thread      (SummaryView  *summaryview);
+static void summary_unignore_thread    (SummaryView *summaryview);
 
 static void summary_unthread_for_exec          (SummaryView    *summaryview);
 static void summary_unthread_for_exec_func     (GtkCTree       *ctree,
@@ -245,6 +255,8 @@ static void summary_colorlabel_menu_item_activate_item_cb
                                           gpointer      data);
 static void summary_colorlabel_menu_create(SummaryView *summaryview);
 
+static GtkWidget *summary_ctree_create (SummaryView    *summaryview);
+
 /* callback functions */
 static void summary_toggle_pressed     (GtkWidget              *eventbox,
                                         GdkEventButton         *event,
@@ -306,6 +318,8 @@ static void summary_from_clicked    (GtkWidget              *button,
                                         SummaryView            *summaryview);
 static void summary_subject_clicked    (GtkWidget              *button,
                                         SummaryView            *summaryview);
+static void summary_score_clicked      (GtkWidget              *button,
+                                        SummaryView            *summaryview);
 
 static void summary_start_drag         (GtkWidget        *widget, 
                                         int button,
@@ -404,10 +418,21 @@ static GtkItemFactoryEntry summary_popup_entries[] =
        {N_("/Select _all"),            NULL, summary_select_all, 0, NULL}
 };
 
+static const gchar *const col_label[N_SUMMARY_COLS] = {
+       N_("M"),        /* S_COL_MARK    */
+       N_("U"),        /* S_COL_UNREAD  */
+       "",             /* S_COL_MIME    */
+       N_("Subject"),  /* S_COL_SUBJECT */
+       N_("From"),     /* S_COL_FROM    */
+       N_("Date"),     /* S_COL_DATE    */
+       N_("Size"),     /* S_COL_SIZE    */
+       N_("No."),      /* S_COL_NUMBER  */
+       N_("Score")     /* S_COL_SCORE   */
+};
+
 SummaryView *summary_create(void)
 {
        SummaryView *summaryview;
-       gchar *titles[N_SUMMARY_COLS] = {_("M"), _("U")};
        GtkWidget *vbox;
        GtkWidget *scrolledwin;
        GtkWidget *ctree;
@@ -420,7 +445,6 @@ SummaryView *summary_create(void)
        GtkWidget *popupmenu;
        GtkItemFactory *popupfactory;
        gint n_entries;
-       gint i;
 
        debug_print(_("Creating summary view...\n"));
        summaryview = g_new0(SummaryView, 1);
@@ -436,92 +460,13 @@ SummaryView *summary_create(void)
                             prefs_common.summaryview_width,
                             prefs_common.summaryview_height);
 
-       if (prefs_common.trans_hdr) {
-               titles[S_COL_NUMBER]  = _("No.");
-               titles[S_COL_DATE]    = _("Date");
-               titles[S_COL_FROM]    = _("From");
-               titles[S_COL_SUBJECT] = _("Subject");
-       } else {
-               titles[S_COL_NUMBER]  = "No.";
-               titles[S_COL_DATE]    = "Date";
-               titles[S_COL_FROM]    = "From";
-               titles[S_COL_SUBJECT] = "Subject";
-       }
-       titles[S_COL_SIZE]  = _("Size");
-       titles[S_COL_SCORE] = _("Score");
+       ctree = summary_ctree_create(summaryview);
 
-       ctree = gtk_sctree_new_with_titles(N_SUMMARY_COLS, S_COL_SUBJECT, titles);
        gtk_scrolled_window_set_hadjustment(GTK_SCROLLED_WINDOW(scrolledwin),
                                            GTK_CLIST(ctree)->hadjustment);
        gtk_scrolled_window_set_vadjustment(GTK_SCROLLED_WINDOW(scrolledwin),
                                            GTK_CLIST(ctree)->vadjustment);
        gtk_container_add(GTK_CONTAINER(scrolledwin), ctree);
-       gtk_clist_set_selection_mode(GTK_CLIST(ctree), GTK_SELECTION_EXTENDED);
-       gtk_clist_set_column_justification(GTK_CLIST(ctree), S_COL_MARK,
-                                          GTK_JUSTIFY_CENTER);
-       gtk_clist_set_column_justification(GTK_CLIST(ctree), S_COL_UNREAD,
-                                          GTK_JUSTIFY_CENTER);
-       gtk_clist_set_column_justification(GTK_CLIST(ctree), S_COL_MIME,
-                                          GTK_JUSTIFY_CENTER);
-       gtk_clist_set_column_justification(GTK_CLIST(ctree), S_COL_NUMBER,
-                                          GTK_JUSTIFY_RIGHT);
-       gtk_clist_set_column_justification(GTK_CLIST(ctree), S_COL_SCORE,
-                                          GTK_JUSTIFY_RIGHT);
-       gtk_clist_set_column_justification(GTK_CLIST(ctree), S_COL_SIZE,
-                                          GTK_JUSTIFY_RIGHT);
-       gtk_clist_set_column_width(GTK_CLIST(ctree), S_COL_MARK,
-                                  SUMMARY_COL_MARK_WIDTH);
-       gtk_clist_set_column_width(GTK_CLIST(ctree), S_COL_UNREAD,
-                                  SUMMARY_COL_UNREAD_WIDTH);
-       gtk_clist_set_column_width(GTK_CLIST(ctree), S_COL_MIME,
-                                  SUMMARY_COL_MIME_WIDTH);
-       gtk_clist_set_column_width(GTK_CLIST(ctree), S_COL_NUMBER,
-                                  prefs_common.summary_col_number);
-       gtk_clist_set_column_width(GTK_CLIST(ctree), S_COL_SCORE,
-                                  prefs_common.summary_col_score);
-       gtk_clist_set_column_width(GTK_CLIST(ctree), S_COL_SIZE,
-                                  prefs_common.summary_col_size);
-       gtk_clist_set_column_width(GTK_CLIST(ctree), S_COL_DATE,
-                                  prefs_common.summary_col_date);
-       gtk_clist_set_column_width(GTK_CLIST(ctree), S_COL_FROM,
-                                  prefs_common.summary_col_from);
-       gtk_clist_set_column_width(GTK_CLIST(ctree), S_COL_SUBJECT,
-                                  prefs_common.summary_col_subject);
-       gtk_ctree_set_line_style(GTK_CTREE(ctree), GTK_CTREE_LINES_DOTTED);
-       gtk_ctree_set_expander_style(GTK_CTREE(ctree),
-                                    GTK_CTREE_EXPANDER_SQUARE);
-#if 0
-       gtk_ctree_set_line_style(GTK_CTREE(ctree), GTK_CTREE_LINES_NONE);
-       gtk_ctree_set_expander_style(GTK_CTREE(ctree),
-                                    GTK_CTREE_EXPANDER_TRIANGLE);
-#endif
-       gtk_ctree_set_indent(GTK_CTREE(ctree), 18);
-       gtk_object_set_user_data(GTK_OBJECT(ctree), summaryview);
-
-       /* don't let title buttons take key focus */
-       for (i = 0; i < N_SUMMARY_COLS; i++)
-               GTK_WIDGET_UNSET_FLAGS(GTK_CLIST(ctree)->column[i].button,
-                                      GTK_CAN_FOCUS);
-
-       /* connect signal to the buttons for sorting */
-#define CLIST_BUTTON_SIGNAL_CONNECT(col, func) \
-       gtk_signal_connect \
-               (GTK_OBJECT(GTK_CLIST(ctree)->column[col].button), \
-                "clicked", \
-                GTK_SIGNAL_FUNC(func), \
-                summaryview)
-
-       CLIST_BUTTON_SIGNAL_CONNECT(S_COL_MARK   , summary_mark_clicked);
-       CLIST_BUTTON_SIGNAL_CONNECT(S_COL_UNREAD , summary_unread_clicked);
-       CLIST_BUTTON_SIGNAL_CONNECT(S_COL_MIME   , summary_mime_clicked);
-       CLIST_BUTTON_SIGNAL_CONNECT(S_COL_NUMBER , summary_num_clicked);
-       CLIST_BUTTON_SIGNAL_CONNECT(S_COL_SCORE  , summary_score_clicked);
-       CLIST_BUTTON_SIGNAL_CONNECT(S_COL_SIZE   , summary_size_clicked);
-       CLIST_BUTTON_SIGNAL_CONNECT(S_COL_DATE   , summary_date_clicked);
-       CLIST_BUTTON_SIGNAL_CONNECT(S_COL_FROM   , summary_from_clicked);
-       CLIST_BUTTON_SIGNAL_CONNECT(S_COL_SUBJECT, summary_subject_clicked);
-
-#undef CLIST_BUTTON_SIGNAL_CONNECT
 
        /* create status label */
        hbox = gtk_hbox_new(FALSE, 0);
@@ -537,6 +482,9 @@ SummaryView *summary_create(void)
        gtk_box_pack_end(GTK_BOX(hbox), toggle_eventbox, FALSE, FALSE, 4);
        toggle_arrow = gtk_arrow_new(GTK_ARROW_DOWN, GTK_SHADOW_OUT);
        gtk_container_add(GTK_CONTAINER(toggle_eventbox), toggle_arrow);
+       gtk_signal_connect(GTK_OBJECT(toggle_eventbox), "button_press_event",
+                          GTK_SIGNAL_FUNC(summary_toggle_pressed),
+                          summaryview);
 
        statlabel_msgs = gtk_label_new("");
        gtk_box_pack_end(GTK_BOX(hbox), statlabel_msgs, FALSE, FALSE, 4);
@@ -548,40 +496,6 @@ SummaryView *summary_create(void)
                                      "<SummaryView>", &popupfactory,
                                      summaryview);
 
-       /* connect signals */
-       gtk_signal_connect(GTK_OBJECT(ctree), "tree_select_row",
-                          GTK_SIGNAL_FUNC(summary_selected), summaryview);
-       gtk_signal_connect(GTK_OBJECT(ctree), "button_press_event",
-                          GTK_SIGNAL_FUNC(summary_button_pressed),
-                          summaryview);
-       gtk_signal_connect(GTK_OBJECT(ctree), "button_release_event",
-                          GTK_SIGNAL_FUNC(summary_button_released),
-                          summaryview);
-       gtk_signal_connect(GTK_OBJECT(ctree), "key_press_event",
-                          GTK_SIGNAL_FUNC(summary_key_pressed), summaryview);
-       gtk_signal_connect(GTK_OBJECT(ctree), "resize_column",
-                          GTK_SIGNAL_FUNC(summary_col_resized), summaryview);
-        gtk_signal_connect(GTK_OBJECT(ctree), "open_row",
-                          GTK_SIGNAL_FUNC(summary_open_row), summaryview);
-
-       gtk_signal_connect_after(GTK_OBJECT(ctree), "tree_expand",
-                                GTK_SIGNAL_FUNC(summary_tree_expanded),
-                                summaryview);
-       gtk_signal_connect_after(GTK_OBJECT(ctree), "tree_collapse",
-                                GTK_SIGNAL_FUNC(summary_tree_collapsed),
-                                summaryview);
-
-       gtk_signal_connect(GTK_OBJECT(ctree), "start_drag",
-                          GTK_SIGNAL_FUNC(summary_start_drag),
-                          summaryview);
-       gtk_signal_connect(GTK_OBJECT(ctree), "drag_data_get",
-                          GTK_SIGNAL_FUNC(summary_drag_data_get),
-                          summaryview);
-
-       gtk_signal_connect(GTK_OBJECT(toggle_eventbox), "button_press_event",
-                          GTK_SIGNAL_FUNC(summary_toggle_pressed),
-                          summaryview);
-
        summaryview->vbox = vbox;
        summaryview->scrolledwin = scrolledwin;
        summaryview->ctree = ctree;
@@ -598,8 +512,6 @@ SummaryView *summary_create(void)
        summaryview->sort_mode = SORT_BY_NONE;
        summaryview->sort_type = GTK_SORT_ASCENDING;
 
-       summary_change_display_item(summaryview);
-
        gtk_widget_show_all(vbox);
 
        return summaryview;
@@ -627,7 +539,7 @@ void summary_init(SummaryView *summaryview)
 
        pixmap = gtk_pixmap_new(clipxpm, clipxpmmask);
        gtk_clist_set_column_widget(GTK_CLIST(summaryview->ctree),
-                                   S_COL_MIME, pixmap);
+                                   summaryview->col_pos[S_COL_MIME], pixmap);
        gtk_widget_show(pixmap);
 
        if (!small_style) {
@@ -910,8 +822,8 @@ gboolean summary_show(SummaryView *summaryview, FolderItem *item,
                           message, but do not display it */
                        node = summary_find_next_unread_msg(summaryview, NULL);
                        if (node == NULL && GTK_CLIST(ctree)->row_list != NULL)
-                               node = GTK_CTREE_NODE
-                                       (GTK_CLIST(ctree)->row_list_end);
+                               node = gtk_ctree_node_nth
+                                       (ctree, GTK_CLIST(ctree)->rows - 1);
                        summary_select_node(summaryview, node, FALSE);
                }
        } else {
@@ -922,10 +834,17 @@ gboolean summary_show(SummaryView *summaryview, FolderItem *item,
                else
                        node = summary_find_next_unread_msg(summaryview, NULL);
 
-               if (node == NULL && GTK_CLIST(ctree)->row_list != NULL)
-                       node = GTK_CTREE_NODE(GTK_CLIST(ctree)->row_list_end);
-               summary_select_node(summaryview, node,
-                                   prefs_common.open_unread_on_enter);
+               if (node == NULL && GTK_CLIST(ctree)->row_list != NULL) {
+                       /* Get the last visible node on screen */
+                       /* FIXME: huh, what happens if node is null? that allowed?? */
+                       node = gtk_ctree_node_nth(ctree, GTK_CLIST(ctree)->rows - 1);
+               }       
+               if (prefs_common.open_unread_on_enter) {
+                       summary_unlock(summaryview);
+                       summary_select_node(summaryview, node, TRUE);
+                       summary_lock(summaryview);
+               } else
+                       summary_select_node(summaryview, node, FALSE);
        }
 
        summary_status_show(summaryview);
@@ -991,8 +910,13 @@ void summary_clear_list(SummaryView *summaryview)
        summaryview->sort_type = GTK_SORT_ASCENDING;
 
        gtk_clist_clear(clist);
-       optimal_width = gtk_clist_optimal_column_width(clist, S_COL_SUBJECT);
-       gtk_clist_set_column_width(clist, S_COL_SUBJECT, optimal_width);
+       if (summaryview->col_pos[S_COL_SUBJECT] == N_SUMMARY_COLS - 1) {
+               optimal_width = gtk_clist_optimal_column_width
+                       (clist, summaryview->col_pos[S_COL_SUBJECT]);
+               gtk_clist_set_column_width
+                       (clist, summaryview->col_pos[S_COL_SUBJECT],
+                        optimal_width);
+       }
 
        gtk_clist_thaw(clist);
 }
@@ -1061,8 +985,7 @@ static void summary_set_menu_sensitive(SummaryView *summaryview)
        if (summaryview->folder_item->folder->type != F_NEWS) {
                menu_set_sensitive(ifactory, "/Move...", TRUE);
                menu_set_sensitive(ifactory, "/Copy...", TRUE);
-               if (summaryview->folder_item->stype != F_TRASH)
-                       menu_set_sensitive(ifactory, "/Delete", TRUE);
+               menu_set_sensitive(ifactory, "/Delete", TRUE);
        }
 
        menu_set_sensitive(ifactory, "/Execute", TRUE);
@@ -1092,9 +1015,9 @@ static void summary_set_menu_sensitive(SummaryView *summaryview)
        menu_set_sensitive(ifactory, "/View/Open in new window", sens);
        menu_set_sensitive(ifactory, "/View/Source", sens);
        menu_set_sensitive(ifactory, "/View/All header", sens);
-       if ((summaryview->folder_item->stype == F_DRAFT) ||
-           (summaryview->folder_item->stype == F_OUTBOX) ||
-           (summaryview->folder_item->stype == F_QUEUE))
+       if (summaryview->folder_item->stype == F_OUTBOX ||
+           summaryview->folder_item->stype == F_DRAFT  ||
+           summaryview->folder_item->stype == F_QUEUE)
                menu_set_sensitive(ifactory, "/Re-edit", sens);
 
        menu_set_sensitive(ifactory, "/Save as...", sens);
@@ -1364,6 +1287,46 @@ static guint summary_get_msgnum(SummaryView *summaryview, GtkCTreeNode *node)
        return msginfo->msgnum;
 }
 
+static GtkCTreeNode *summary_find_prev_msg(SummaryView *summaryview,
+                                          GtkCTreeNode *current_node)
+{
+       GtkCTree *ctree = GTK_CTREE(summaryview->ctree);
+       GtkCTreeNode *node;
+       MsgInfo *msginfo;
+
+       if (current_node)
+               node = current_node;
+       else
+               node = gtk_ctree_node_nth(ctree, GTK_CLIST(ctree)->rows - 1);
+
+       for (; node != NULL; node = GTK_CTREE_NODE_PREV(node)) {
+               msginfo = gtk_ctree_node_get_row_data(ctree, node);
+               if (!MSG_IS_DELETED(msginfo->flags)) break;
+       }
+
+       return node;
+}
+
+static GtkCTreeNode *summary_find_next_msg(SummaryView *summaryview,
+                                          GtkCTreeNode *current_node)
+{
+       GtkCTree *ctree = GTK_CTREE(summaryview->ctree);
+       GtkCTreeNode *node;
+       MsgInfo *msginfo;
+
+       if (current_node)
+               node = current_node;
+       else
+               node = GTK_CTREE_NODE(GTK_CLIST(ctree)->row_list);
+
+       for (; node != NULL; node = gtkut_ctree_node_next(ctree, node)) {
+               msginfo = gtk_ctree_node_get_row_data(ctree, node);
+               if (!MSG_IS_DELETED(msginfo->flags)) break;
+       }
+
+       return node;
+}
+
 static GtkCTreeNode *summary_find_prev_unread_msg(SummaryView *summaryview,
                                                  GtkCTreeNode *current_node)
 {
@@ -1374,7 +1337,7 @@ static GtkCTreeNode *summary_find_prev_unread_msg(SummaryView *summaryview,
        if (current_node)
                node = current_node;
        else
-               node = GTK_CTREE_NODE(GTK_CLIST(ctree)->row_list_end);
+               node = gtk_ctree_node_nth(ctree, GTK_CLIST(ctree)->rows - 1);
 
        for (; node != NULL; node = GTK_CTREE_NODE_PREV(node)) {
                msginfo = gtk_ctree_node_get_row_data(ctree, node);
@@ -1414,7 +1377,7 @@ static GtkCTreeNode *summary_find_prev_marked_msg(SummaryView *summaryview,
        if (current_node)
                node = GTK_CTREE_NODE_PREV(current_node);
        else
-               node = GTK_CTREE_NODE(GTK_CLIST(ctree)->row_list_end);
+               node = gtk_ctree_node_nth(ctree, GTK_CLIST(ctree)->rows - 1);
 
        for (; node != NULL; node = GTK_CTREE_NODE_PREV(node)) {
                msginfo = gtk_ctree_node_get_row_data(ctree, node);
@@ -1454,7 +1417,7 @@ static GtkCTreeNode *summary_find_prev_labeled_msg(SummaryView *summaryview,
        if (current_node)
                node = GTK_CTREE_NODE_PREV(current_node);
        else
-               node = GTK_CTREE_NODE(GTK_CLIST(ctree)->row_list_end);
+               node = gtk_ctree_node_nth(ctree, GTK_CLIST(ctree)->rows - 1);
 
        for (; node != NULL; node = GTK_CTREE_NODE_PREV(node)) {
                msginfo = gtk_ctree_node_get_row_data(ctree, node);
@@ -1817,6 +1780,43 @@ void summary_sort(SummaryView *summaryview, SummarySortType type)
        main_window_cursor_normal(summaryview->mainwin);
 }
 
+gboolean summary_insert_gnode_func(GtkCTree *ctree, guint depth, GNode *gnode,
+                                  GtkCTreeNode *cnode, gpointer data)
+{
+       SummaryView *summaryview = (SummaryView *)data;
+       MsgInfo *msginfo = (MsgInfo *)gnode->data;
+       gchar *text[N_SUMMARY_COLS];
+       gint *col_pos = summaryview->col_pos;
+       const gchar *msgid = msginfo->msgid;
+       GHashTable *msgid_table = summaryview->msgid_table;
+
+       summary_set_header(summaryview, text, msginfo);
+
+       gtk_ctree_set_node_info(ctree, cnode, text[col_pos[S_COL_SUBJECT]], 2,
+                               NULL, NULL, NULL, NULL, FALSE,
+                               gnode->parent->parent ? TRUE : FALSE);
+#define SET_TEXT(col) \
+       gtk_ctree_node_set_text(ctree, cnode, col_pos[col], \
+                               text[col_pos[col]])
+
+       SET_TEXT(S_COL_NUMBER);
+       SET_TEXT(S_COL_SCORE);
+       SET_TEXT(S_COL_SIZE);
+       SET_TEXT(S_COL_DATE);
+       SET_TEXT(S_COL_FROM);
+       SET_TEXT(S_COL_SUBJECT);
+
+#undef SET_TEXT
+
+       GTKUT_CTREE_NODE_SET_ROW_DATA(cnode, msginfo);
+       summary_set_marks_func(ctree, cnode, summaryview);
+
+       if (msgid)
+               g_hash_table_insert(msgid_table, (gchar *)msgid, cnode);
+
+       return TRUE;
+}
+
 static GtkCTreeNode * subject_table_lookup(GHashTable *subject_table,
                                           gchar * subject)
 {
@@ -1842,8 +1842,7 @@ static void summary_set_ctree_from_list(SummaryView *summaryview,
        MsgInfo *msginfo;
        MsgInfo *parentinfo;
        MsgInfo *cur_msginfo;
-       GtkCTreeNode *node, *parent;
-       gchar *text[N_SUMMARY_COLS];
+       GtkCTreeNode *node = NULL;
        GHashTable *msgid_table;
        GHashTable *subject_table;
        GSList * cur;
@@ -1878,124 +1877,50 @@ static void summary_set_ctree_from_list(SummaryView *summaryview,
        }
        
        if (summaryview->folder_item->threaded) {
-               for (; mlist != NULL; mlist = mlist->next) {
-                       msginfo = (MsgInfo *)mlist->data;
-                       parent = NULL;
-
-                       summary_set_header(text, msginfo);
+               GNode *root, *gnode;
 
-                       /* search parent node for threading */
-                       if (msginfo->inreplyto && *msginfo->inreplyto) {
-                               parent = g_hash_table_lookup
-                                       (msgid_table, msginfo->inreplyto);
-                       }
-                       if (parent == NULL && msginfo->subject &&
-                           g_strncasecmp(msginfo->subject, "Re: ", 4) == 0) {
-                               parent = subject_table_lookup
-                                       (subject_table, msginfo->subject);
-                       }
-                       if(parent) {
-                               parentinfo = gtk_ctree_node_get_row_data(ctree, parent);
-                               if(!MSG_IS_IGNORE_THREAD(msginfo->flags) && parentinfo && MSG_IS_IGNORE_THREAD(parentinfo->flags)) {
-                                       MSG_SET_PERM_FLAGS(msginfo->flags, MSG_IGNORE_THREAD);
-                               }
-                       }
-
-                       node = gtk_ctree_insert_node
-                               (ctree, parent, NULL, text, 2,
-                                NULL, NULL, NULL, NULL, FALSE,
-                                parent ? TRUE : FALSE);
-                       GTKUT_CTREE_NODE_SET_ROW_DATA(node, msginfo);
-                       summary_set_marks_func(ctree, node, summaryview);
-                       
-                       if (MSG_GET_COLORLABEL(msginfo->flags))
-                               summary_set_colorlabel_color(ctree, node, MSG_GET_COLORLABEL_VALUE(msginfo->flags));
-
-                       /* preserve previous node if the message is
-                          duplicated */
-                       if (msginfo->msgid && *msginfo->msgid &&
-                           g_hash_table_lookup(msgid_table, msginfo->msgid)
-                           == NULL)
-                               g_hash_table_insert(msgid_table,
-                                                   msginfo->msgid, node);
-                       if (msginfo->subject &&
-                           subject_table_lookup(subject_table,
-                                                msginfo->subject) == NULL) {
-                               subject_table_insert(subject_table,
-                                                    msginfo->subject, node);
-                       }
+               root = procmsg_get_thread_tree(mlist);
 
-                       cur_parent = parent;
-                       cur_msginfo = msginfo;
-                       while (cur_parent != NULL) {
-                               parentinfo = gtk_ctree_node_get_row_data(ctree, cur_parent);
-
-                               if (!parentinfo)
-                                       break;
-                               
-                               if (parentinfo->threadscore <
-                                   cur_msginfo->threadscore) {
-                                       gchar * s;
-                                       parentinfo->threadscore =
-                                               cur_msginfo->threadscore;
-#if 0
-                                       s = itos(parentinfo->threadscore);
-                                       gtk_ctree_node_set_text(ctree, cur_parent, S_COL_SCORE, s);
-#endif
-                               }
-                               else break;
-                               
-                               cur_msginfo = parentinfo;
-                               if (cur_msginfo->inreplyto &&
-                                   *cur_msginfo->inreplyto) {
-                                       cur_parent = g_hash_table_lookup(msgid_table, cur_msginfo->inreplyto);
-                               }
-                               if (cur_parent == NULL &&
-                                   cur_msginfo->subject &&
-                                   g_strncasecmp(cur_msginfo->subject,
-                                                 "Re: ", 4) == 0) {
-                                       cur_parent = subject_table_lookup(subject_table, cur_msginfo->subject);
-                               }
-                       }
+               for (gnode = root->children; gnode != NULL;
+                    gnode = gnode->next) {
+                       node = gtk_ctree_insert_gnode
+                               (ctree, NULL, node, gnode,
+                                summary_insert_gnode_func, summaryview);
                }
 
-               /* complete the thread */
-               summary_thread_build(summaryview, TRUE);
+               g_node_destroy(root);
+
+               summary_thread_init(summaryview);
        } else {
+               gchar *text[N_SUMMARY_COLS];
+
+               mlist = g_slist_reverse(mlist);
                for (; mlist != NULL; mlist = mlist->next) {
                        msginfo = (MsgInfo *)mlist->data;
 
-                       summary_set_header(text, msginfo);
+                       summary_set_header(summaryview, text, msginfo);
 
                        node = gtk_ctree_insert_node
-                               (ctree, NULL, NULL, text, 2,
+                               (ctree, NULL, node, text, 2,
                                 NULL, NULL, NULL, NULL, FALSE, FALSE);
                        GTKUT_CTREE_NODE_SET_ROW_DATA(node, msginfo);
                        summary_set_marks_func(ctree, node, summaryview);
 
-                       if ( MSG_GET_COLORLABEL(msginfo->flags) )
-                         summary_set_colorlabel_color(ctree, node, MSG_GET_COLORLABEL_VALUE(msginfo->flags));
-
-                       if (msginfo->msgid && *msginfo->msgid &&
-                           g_hash_table_lookup(msgid_table, msginfo->msgid)
-                           == NULL)
+                       if (msginfo->msgid)
                                g_hash_table_insert(msgid_table,
                                                    msginfo->msgid, node);
-
-                       if (msginfo->subject &&
-                           subject_table_lookup(subject_table,
-                                                msginfo->subject) == NULL)
-                               subject_table_insert(subject_table,
-                                                    msginfo->subject, node);
                }
+               mlist = g_slist_reverse(mlist);
        }
 
-       if (prefs_common.enable_hscrollbar) {
+       if (prefs_common.enable_hscrollbar &&
+           summaryview->col_pos[S_COL_SUBJECT] == N_SUMMARY_COLS - 1) {
                gint optimal_width;
 
                optimal_width = gtk_clist_optimal_column_width
-                       (GTK_CLIST(ctree), S_COL_SUBJECT);
-               gtk_clist_set_column_width(GTK_CLIST(ctree), S_COL_SUBJECT,
+                       (GTK_CLIST(ctree), summaryview->col_pos[S_COL_SUBJECT]);
+               gtk_clist_set_column_width(GTK_CLIST(ctree),
+                                          summaryview->col_pos[S_COL_SUBJECT],
                                           optimal_width);
        }
 
@@ -2120,36 +2045,39 @@ static void summary_write_cache_func(GtkCTree *ctree, GtkCTreeNode *node,
        procmsg_write_flags(msginfo, fps->mark_fp);
 }
 
-static void summary_set_header(gchar *text[], MsgInfo *msginfo)
+static void summary_set_header(SummaryView *summaryview, gchar *text[],
+                              MsgInfo *msginfo)
 {
        static gchar date_modified[80];
        static gchar *to = NULL;
        static gchar *from_name = NULL;
        static gchar col_number[11];
        static gchar col_score[11];
+       gint *col_pos = summaryview->col_pos;
+
+       text[col_pos[S_COL_MARK]]   = NULL;
+       text[col_pos[S_COL_UNREAD]] = NULL;
+       text[col_pos[S_COL_MIME]]   = NULL;
+       text[col_pos[S_COL_NUMBER]] = itos(msginfo->msgnum);
+       text[col_pos[S_COL_SIZE]]   = to_human_readable(msginfo->size);
 
-       text[S_COL_MARK]   = NULL;
-       text[S_COL_UNREAD] = NULL;
-       text[S_COL_MIME]   = NULL;
-       text[S_COL_NUMBER] = itos_buf(col_number, msginfo->msgnum);
-       text[S_COL_SIZE]   = to_human_readable(msginfo->size);
 #if 0
-       text[S_COL_SCORE]  = itos_buf(col_score, msginfo->threadscore);
+       text[col_pos[S_COL_SCORE]]  = itos_buf(col_score, msginfo->threadscore);
 #else
-       text[S_COL_SCORE]  = itos_buf(col_score, msginfo->score);
+       text[col_pos[S_COL_SCORE]]  = itos_buf(col_score, msginfo->score);
 #endif
 
        if (msginfo->date_t) {
                procheader_date_get_localtime(date_modified,
                                              sizeof(date_modified),
                                              msginfo->date_t);
-               text[S_COL_DATE] = date_modified;
+               text[col_pos[S_COL_DATE]] = date_modified;
        } else if (msginfo->date)
-               text[S_COL_DATE] = msginfo->date;
+               text[col_pos[S_COL_DATE]] = msginfo->date;
        else
-               text[S_COL_DATE] = _("(No Date)");
+               text[col_pos[S_COL_DATE]] = _("(No Date)");
 
-       text[S_COL_FROM] = msginfo->fromname ? msginfo->fromname :
+       text[col_pos[S_COL_FROM]] = msginfo->fromname ? msginfo->fromname :
                _("(No From)");
        if (prefs_common.swap_from && msginfo->from && msginfo->to &&
            !MSG_IS_NEWS(msginfo->flags)) {
@@ -2161,7 +2089,7 @@ static void summary_set_header(gchar *text[], MsgInfo *msginfo)
                if (account_find_from_address(from)) {
                        g_free(to);
                        to = g_strconcat("-->", msginfo->to, NULL);
-                       text[S_COL_FROM] = to;
+                       text[col_pos[S_COL_FROM]] = to;
                }
        }
 
@@ -2185,7 +2113,7 @@ static void summary_set_header(gchar *text[], MsgInfo *msginfo)
                }
        }
 
-       text[S_COL_SUBJECT] = msginfo->subject ? msginfo->subject :
+       text[col_pos[S_COL_SUBJECT]] = msginfo->subject ? msginfo->subject :
                _("(No Subject)");
 }
 
@@ -2299,10 +2227,10 @@ void summary_reedit(SummaryView *summaryview)
        MsgInfo *msginfo;
 
        if (!summaryview->selected) return;
-       if (!summaryview->folder_item ||
-           (summaryview->folder_item->stype != F_DRAFT &&
-            summaryview->folder_item->stype != F_OUTBOX &&
-            summaryview->folder_item->stype != F_QUEUE)) return;
+       if (!summaryview->folder_item) return;
+       if (summaryview->folder_item->stype != F_OUTBOX &&
+           summaryview->folder_item->stype != F_DRAFT  &&
+           summaryview->folder_item->stype != F_QUEUE) return;
 
        msginfo = gtk_ctree_node_get_row_data(GTK_CTREE(summaryview->ctree),
                                              summaryview->selected);
@@ -2324,7 +2252,7 @@ void summary_step(SummaryView *summaryview, GtkScrollType type)
 
        gtk_signal_emit_by_name(GTK_OBJECT(ctree), "scroll_vertical",
                                type, 0.0);
-
+       
        if (summaryview->msg_is_toggled_on)
                summary_display_msg(summaryview, summaryview->selected, FALSE);
 }
@@ -2416,6 +2344,7 @@ static void summary_set_row_marks(SummaryView *summaryview, GtkCTreeNode *row)
        GtkStyle *style = NULL;
        MsgInfo *msginfo;
        MsgFlags flags;
+       gint *col_pos = summaryview->col_pos;
 
        msginfo = gtk_ctree_node_get_row_data(ctree, row);
        if (!msginfo) return;
@@ -2426,22 +2355,23 @@ static void summary_set_row_marks(SummaryView *summaryview, GtkCTreeNode *row)
 
        /* set new/unread column */
        if (MSG_IS_IGNORE_THREAD(flags)) {
-               gtk_ctree_node_set_pixmap(ctree, row, S_COL_UNREAD,
+               gtk_ctree_node_set_pixmap(ctree, row, col_pos[S_COL_UNREAD],
                                          ignorethreadxpm, ignorethreadxpmmask);
        } else if (MSG_IS_NEW(flags)) {
-               gtk_ctree_node_set_pixmap(ctree, row, S_COL_UNREAD,
+               gtk_ctree_node_set_pixmap(ctree, row, col_pos[S_COL_UNREAD],
                                          newxpm, newxpmmask);
        } else if (MSG_IS_UNREAD(flags)) {
-               gtk_ctree_node_set_pixmap(ctree, row, S_COL_UNREAD,
+               gtk_ctree_node_set_pixmap(ctree, row, col_pos[S_COL_UNREAD],
                                          unreadxpm, unreadxpmmask);
        } else if (MSG_IS_REPLIED(flags)) {
-               gtk_ctree_node_set_pixmap(ctree, row, S_COL_UNREAD,
+               gtk_ctree_node_set_pixmap(ctree, row, col_pos[S_COL_UNREAD],
                                          repliedxpm, repliedxpmmask);
        } else if (MSG_IS_FORWARDED(flags)) {
-               gtk_ctree_node_set_pixmap(ctree, row, S_COL_UNREAD,
+               gtk_ctree_node_set_pixmap(ctree, row, col_pos[S_COL_UNREAD],
                                          forwardedxpm, forwardedxpmmask);
        } else {
-               gtk_ctree_node_set_text(ctree, row, S_COL_UNREAD, NULL);
+               gtk_ctree_node_set_text(ctree, row, col_pos[S_COL_UNREAD],
+                                       NULL);
        }
 
        if (prefs_common.bold_unread &&
@@ -2453,7 +2383,7 @@ static void summary_set_row_marks(SummaryView *summaryview, GtkCTreeNode *row)
 
        /* set mark column */
        if (MSG_IS_DELETED(flags)) {
-               gtk_ctree_node_set_pixmap(ctree, row, S_COL_MARK,
+               gtk_ctree_node_set_pixmap(ctree, row, col_pos[S_COL_MARK],
                                          deletedxpm, deletedxpmmask);
                if (style)
                        style = bold_deleted_style;
@@ -2463,10 +2393,10 @@ static void summary_set_row_marks(SummaryView *summaryview, GtkCTreeNode *row)
                        gtk_ctree_node_set_foreground
                                (ctree, row, &summaryview->color_dim);
        } else if (MSG_IS_MARKED(flags)) {
-               gtk_ctree_node_set_pixmap(ctree, row, S_COL_MARK,
+               gtk_ctree_node_set_pixmap(ctree, row, col_pos[S_COL_MARK],
                                          markxpm, markxpmmask);
        } else if (MSG_IS_MOVE(flags)) {
-               gtk_ctree_node_set_text(ctree, row, S_COL_MARK, "o");
+               gtk_ctree_node_set_text(ctree, row, col_pos[S_COL_MARK], "o");
                if (style)
                        style = bold_marked_style;
                else {
@@ -2475,8 +2405,8 @@ static void summary_set_row_marks(SummaryView *summaryview, GtkCTreeNode *row)
                        gtk_ctree_node_set_foreground
                                (ctree, row, &summaryview->color_marked);
        } else if (MSG_IS_COPY(flags)) {
-               gtk_ctree_node_set_text(ctree, row, S_COL_MARK, "O");
-                if (style)
+               gtk_ctree_node_set_text(ctree, row, col_pos[S_COL_MARK], "O");
+               if (style)
                        style = bold_marked_style;
                else {
                        style = small_marked_style;
@@ -2492,14 +2422,14 @@ static void summary_set_row_marks(SummaryView *summaryview, GtkCTreeNode *row)
                gtk_ctree_node_set_foreground(ctree, row,
                                              &summaryview->color_important);
        } else {
-               gtk_ctree_node_set_text(ctree, row, S_COL_MARK, NULL);
+               gtk_ctree_node_set_text(ctree, row, col_pos[S_COL_MARK], NULL);
        }
 
        if (MSG_IS_MIME(flags)) {
-               gtk_ctree_node_set_pixmap(ctree, row, S_COL_MIME,
+               gtk_ctree_node_set_pixmap(ctree, row, col_pos[S_COL_MIME],
                                          clipxpm, clipxpmmask);
        } else {
-               gtk_ctree_node_set_text(ctree, row, S_COL_MIME, NULL);
+               gtk_ctree_node_set_text(ctree, row, col_pos[S_COL_MIME], NULL);
        }
         if (!style)
                style = small_style;
@@ -2679,7 +2609,8 @@ static void summary_delete_row(SummaryView *summaryview, GtkCTreeNode *row)
        CHANGE_FLAGS(msginfo);
        summaryview->deleted++;
 
-       if (!prefs_common.immediate_exec)
+       if (!prefs_common.immediate_exec && 
+           summaryview->folder_item->stype != F_TRASH)
                summary_set_row_marks(summaryview, row);
 
        debug_print(_("Message %s/%d is set to delete\n"),
@@ -2689,25 +2620,47 @@ static void summary_delete_row(SummaryView *summaryview, GtkCTreeNode *row)
 void summary_delete(SummaryView *summaryview)
 {
        GtkCTree *ctree = GTK_CTREE(summaryview->ctree);
+       FolderItem *item = summaryview->folder_item;
        GList *cur;
+       GtkCTreeNode *sel_last = NULL;
+       GtkCTreeNode *node;
 
-       if (!summaryview->folder_item ||
-           summaryview->folder_item->folder->type == F_NEWS) return;
+       if (!item || item->folder->type == F_NEWS) return;
 
        if (summary_is_locked(summaryview)) return;
 
-       /* if current folder is trash, don't delete */
-       if (summaryview->folder_item->stype == F_TRASH) {
-               alertpanel_notice(_("Current folder is Trash."));
-               return;
+       /* if current folder is trash, ask for confirmation */
+       if (item->stype == F_TRASH) {
+               AlertValue aval;
+
+               aval = alertpanel(_("Delete message(s)"),
+                                 _("Do you really want to delete message(s) from the trash?"),
+                                 _("Yes"), _("No"), NULL);
+               if (aval != G_ALERTDEFAULT) return;
        }
 
-       for (cur = GTK_CLIST(ctree)->selection; cur != NULL; cur = cur->next)
-               summary_delete_row(summaryview, GTK_CTREE_NODE(cur->data));
+       /* next code sets current row focus right. We need to find a row
+        * that is not deleted. */
+       for (cur = GTK_CLIST(ctree)->selection; cur != NULL; cur = cur->next) {
+               sel_last = GTK_CTREE_NODE(cur->data);
+               summary_delete_row(summaryview, sel_last);
+       }
 
-       summary_step(summaryview, GTK_SCROLL_STEP_FORWARD);
+       node = summary_find_next_msg(summaryview, sel_last);
+       if (!node)
+               node = summary_find_prev_msg(summaryview, sel_last);
 
-       if (prefs_common.immediate_exec)
+       if (node) {
+               if (sel_last && node == gtkut_ctree_node_next(ctree, sel_last))
+                       summary_step(summaryview, GTK_SCROLL_STEP_FORWARD);
+               else if (sel_last && node == GTK_CTREE_NODE_PREV(sel_last))
+                       summary_step(summaryview, GTK_SCROLL_STEP_BACKWARD);
+               else
+                       summary_select_node(summaryview, node,
+                                           summaryview->msg_is_toggled_on);
+       }
+
+       if (prefs_common.immediate_exec || item->stype == F_TRASH)
                summary_execute(summaryview);
        else
                summary_status_show(summaryview);
@@ -3108,7 +3061,7 @@ gboolean summary_execute(SummaryView *summaryview)
        }
 
        if (summaryview->folder_item->threaded)
-               summary_thread_build(summaryview, FALSE);
+               summary_thread_build(summaryview);
 
        summaryview->selected = clist->selection ?
                GTK_CTREE_NODE(clist->selection->data) : NULL;
@@ -3238,7 +3191,6 @@ static void summary_execute_delete(SummaryView *summaryview)
        if (summaryview->folder_item->folder->type == F_MH) {
                g_return_if_fail(trash != NULL);
        }
-       if (summaryview->folder_item == trash) return;
 
        /* search deleting messages and execute */
        gtk_ctree_pre_recursive
@@ -3246,12 +3198,10 @@ static void summary_execute_delete(SummaryView *summaryview)
 
        if (!summaryview->mlist) return;
 
-       for(cur = summaryview->mlist ; cur != NULL ; cur = cur->next) {
-               MsgInfo * msginfo = cur->data;
-               MSG_UNSET_PERM_FLAGS(msginfo->flags, MSG_DELETED);
-       }
-
-       folder_item_move_msgs_with_dest(trash, summaryview->mlist);
+       if (summaryview->folder_item != trash)
+               folder_item_move_msgs_with_dest(trash, summaryview->mlist);
+       else
+               folder_item_remove_msgs(trash, summaryview->mlist);
 
        for (cur = summaryview->mlist; cur != NULL; cur = cur->next)
                procmsg_msginfo_free((MsgInfo *)cur->data);
@@ -3259,8 +3209,10 @@ static void summary_execute_delete(SummaryView *summaryview)
        g_slist_free(summaryview->mlist);
        summaryview->mlist = NULL;
 
-       folder_item_scan(trash);
-       folderview_update_item(trash, FALSE);
+       if (summaryview->folder_item != trash) {
+               folder_item_scan(trash);
+               folderview_update_item(trash, FALSE);
+       }
 }
 
 static void summary_execute_delete_func(GtkCTree *ctree, GtkCTreeNode *node,
@@ -3293,7 +3245,7 @@ static void summary_execute_delete_func(GtkCTree *ctree, GtkCTreeNode *node,
 
 /* thread functions */
 
-void summary_thread_build(SummaryView *summaryview, gboolean init)
+void summary_thread_build(SummaryView *summaryview)
 {
        GtkCTree *ctree = GTK_CTREE(summaryview->ctree);
        GtkCTreeNode *node;
@@ -3342,34 +3294,14 @@ void summary_thread_build(SummaryView *summaryview, gboolean init)
 
        node = GTK_CTREE_NODE(GTK_CLIST(ctree)->row_list);
 
-       /* for optimization */
-       if (init) {
-               if (prefs_common.expand_thread) {
-                       while (node) {
-                               next = GTK_CTREE_ROW(node)->sibling;
-                               if (GTK_CTREE_ROW(node)->children)
-                                       gtk_ctree_expand(ctree, node);
-                               node = next;
-                       }
-               } else if (prefs_common.bold_unread) {
-                       while (node) {
-                               next = GTK_CTREE_ROW(node)->sibling;
-                               if (GTK_CTREE_ROW(node)->children)
-                                       summary_set_row_marks
-                                               (summaryview, node);
-                               node = next;
-                       }
-               }
-       } else {
-               while (node) {
-                       next = GTK_CTREE_NODE_NEXT(node);
-                       if (prefs_common.expand_thread)
-                               gtk_ctree_expand(ctree, node);
-                       if (prefs_common.bold_unread &&
-                           GTK_CTREE_ROW(node)->children)
-                               summary_set_row_marks(summaryview, node);
-                       node = next;
-               }
+       while (node) {
+               next = GTK_CTREE_NODE_NEXT(node);
+               if (prefs_common.expand_thread)
+                       gtk_ctree_expand(ctree, node);
+               if (prefs_common.bold_unread &&
+                   GTK_CTREE_ROW(node)->children)
+                       summary_set_row_marks(summaryview, node);
+               node = next;
        }
 
        gtk_clist_thaw(GTK_CLIST(ctree));
@@ -3383,6 +3315,29 @@ void summary_thread_build(SummaryView *summaryview, gboolean init)
        summary_unlock(summaryview);
 }
 
+static void summary_thread_init(SummaryView *summaryview)
+{
+       GtkCTree *ctree = GTK_CTREE(summaryview->ctree);
+       GtkCTreeNode *node = GTK_CTREE_NODE(GTK_CLIST(ctree)->row_list);
+       GtkCTreeNode *next;
+
+       if (prefs_common.expand_thread) {
+               while (node) {
+                       next = GTK_CTREE_ROW(node)->sibling;
+                       if (GTK_CTREE_ROW(node)->children)
+                               gtk_ctree_expand(ctree, node);
+                       node = next;
+               }
+       } else if (prefs_common.bold_unread) {
+               while (node) {
+                       next = GTK_CTREE_ROW(node)->sibling;
+                       if (GTK_CTREE_ROW(node)->children)
+                               summary_set_row_marks(summaryview, node);
+                       node = next;
+               }
+       }
+}
+
 void summary_unthread(SummaryView *summaryview)
 {
        GtkCTree *ctree = GTK_CTREE(summaryview->ctree);
@@ -3481,7 +3436,10 @@ static void summary_unthread_for_exec_func(GtkCTree *ctree, GtkCTreeNode *node,
 
 void summary_filter(SummaryView *summaryview)
 {
-       if (!prefs_common.fltlist) return;
+       if (!prefs_common.fltlist && !prefs_filtering) {
+               alertpanel_error(_("No filter rules defined."));
+               return;
+       }
 
        summary_lock(summaryview);
 
@@ -3497,10 +3455,12 @@ void summary_filter(SummaryView *summaryview)
                                        summaryview);
                
                gtk_clist_thaw(GTK_CLIST(summaryview->ctree));
-
-               if (prefs_common.immediate_exec)
+               
+               if (prefs_common.immediate_exec) {
+                       summary_unlock(summaryview);
                        summary_execute(summaryview);
-               else
+                       summary_lock(summaryview);
+               } else
                        summary_status_show(summaryview);
        }
        else {
@@ -3736,6 +3696,169 @@ static void summary_colorlabel_menu_create(SummaryView *summaryview)
        summaryview->colorlabel_menu = menu;
 }
 
+static GtkWidget *summary_ctree_create(SummaryView *summaryview)
+{
+       GtkWidget *ctree;
+       gint *col_pos = summaryview->col_pos;
+       SummaryColumnState *col_state;
+       const gchar *titles[N_SUMMARY_COLS];
+       SummaryColumnType type;
+       gint pos;
+
+       col_state = prefs_summary_column_get_config();
+       for (pos = 0; pos < N_SUMMARY_COLS; pos++) {
+               summaryview->col_state[pos] = col_state[pos];
+               type = col_state[pos].type;
+               col_pos[type] = pos;
+
+               switch (type) {
+               case S_COL_NUMBER:
+               case S_COL_DATE:
+               case S_COL_FROM:
+               case S_COL_SUBJECT:
+               case S_COL_SCORE:
+                       if (prefs_common.trans_hdr)
+                               titles[pos] = gettext(col_label[type]);
+                       else
+                               titles[pos] = col_label[type];
+                       break;
+               default:
+                       titles[pos] = gettext(col_label[type]);
+               }
+       }
+       col_state = summaryview->col_state;
+
+       ctree = gtk_sctree_new_with_titles
+               (N_SUMMARY_COLS, col_pos[S_COL_SUBJECT], (gchar **)titles);
+
+       gtk_clist_set_selection_mode(GTK_CLIST(ctree), GTK_SELECTION_EXTENDED);
+       gtk_clist_set_column_justification(GTK_CLIST(ctree), col_pos[S_COL_MARK],
+                                          GTK_JUSTIFY_CENTER);
+       gtk_clist_set_column_justification(GTK_CLIST(ctree), col_pos[S_COL_UNREAD],
+                                          GTK_JUSTIFY_CENTER);
+       gtk_clist_set_column_justification(GTK_CLIST(ctree), col_pos[S_COL_MIME],
+                                          GTK_JUSTIFY_CENTER);
+       gtk_clist_set_column_justification(GTK_CLIST(ctree), col_pos[S_COL_SIZE],
+                                          GTK_JUSTIFY_RIGHT);
+       gtk_clist_set_column_justification(GTK_CLIST(ctree), col_pos[S_COL_NUMBER],
+                                          GTK_JUSTIFY_RIGHT);
+       gtk_clist_set_column_justification(GTK_CLIST(ctree), col_pos[S_COL_SCORE],
+                                          GTK_JUSTIFY_RIGHT);
+       gtk_clist_set_column_width(GTK_CLIST(ctree), col_pos[S_COL_MARK],
+                                  SUMMARY_COL_MARK_WIDTH);
+       gtk_clist_set_column_width(GTK_CLIST(ctree), col_pos[S_COL_UNREAD],
+                                  SUMMARY_COL_UNREAD_WIDTH);
+       gtk_clist_set_column_width(GTK_CLIST(ctree), col_pos[S_COL_MIME],
+                                  SUMMARY_COL_MIME_WIDTH);
+       gtk_clist_set_column_width(GTK_CLIST(ctree), col_pos[S_COL_SUBJECT],
+                                  prefs_common.summary_col_size[S_COL_SUBJECT]);
+       gtk_clist_set_column_width(GTK_CLIST(ctree), col_pos[S_COL_FROM],
+                                  prefs_common.summary_col_size[S_COL_FROM]);
+       gtk_clist_set_column_width(GTK_CLIST(ctree), col_pos[S_COL_DATE],
+                                  prefs_common.summary_col_size[S_COL_DATE]);
+       gtk_clist_set_column_width(GTK_CLIST(ctree), col_pos[S_COL_SIZE],
+                                  prefs_common.summary_col_size[S_COL_SIZE]);
+       gtk_clist_set_column_width(GTK_CLIST(ctree), col_pos[S_COL_NUMBER],
+                                  prefs_common.summary_col_size[S_COL_NUMBER]);
+       gtk_clist_set_column_width(GTK_CLIST(ctree), col_pos[S_COL_SCORE],
+                                  prefs_common.summary_col_size[S_COL_SCORE]);
+       gtk_ctree_set_line_style(GTK_CTREE(ctree), GTK_CTREE_LINES_DOTTED);
+       gtk_ctree_set_expander_style(GTK_CTREE(ctree),
+                                    GTK_CTREE_EXPANDER_SQUARE);
+#if 0
+       gtk_ctree_set_line_style(GTK_CTREE(ctree), GTK_CTREE_LINES_NONE);
+       gtk_ctree_set_expander_style(GTK_CTREE(ctree),
+                                    GTK_CTREE_EXPANDER_TRIANGLE);
+#endif
+       gtk_ctree_set_indent(GTK_CTREE(ctree), 18);
+       gtk_object_set_user_data(GTK_OBJECT(ctree), summaryview);
+
+       for (pos = 0; pos < N_SUMMARY_COLS; pos++) {
+               GTK_WIDGET_UNSET_FLAGS(GTK_CLIST(ctree)->column[pos].button,
+                                      GTK_CAN_FOCUS);
+               gtk_clist_set_column_visibility
+                       (GTK_CLIST(ctree), pos, col_state[pos].visible);
+       }
+
+       /* connect signal to the buttons for sorting */
+#define CLIST_BUTTON_SIGNAL_CONNECT(col, func) \
+       gtk_signal_connect \
+               (GTK_OBJECT(GTK_CLIST(ctree)->column[col_pos[col]].button), \
+                "clicked", \
+                GTK_SIGNAL_FUNC(func), \
+                summaryview)
+
+       CLIST_BUTTON_SIGNAL_CONNECT(S_COL_MARK   , summary_mark_clicked);
+       CLIST_BUTTON_SIGNAL_CONNECT(S_COL_UNREAD , summary_unread_clicked);
+       CLIST_BUTTON_SIGNAL_CONNECT(S_COL_MIME   , summary_mime_clicked);
+       CLIST_BUTTON_SIGNAL_CONNECT(S_COL_NUMBER , summary_num_clicked);
+       CLIST_BUTTON_SIGNAL_CONNECT(S_COL_SIZE   , summary_size_clicked);
+       CLIST_BUTTON_SIGNAL_CONNECT(S_COL_DATE   , summary_date_clicked);
+       CLIST_BUTTON_SIGNAL_CONNECT(S_COL_FROM   , summary_from_clicked);
+       CLIST_BUTTON_SIGNAL_CONNECT(S_COL_SUBJECT, summary_subject_clicked);
+       CLIST_BUTTON_SIGNAL_CONNECT(S_COL_SCORE,   summary_score_clicked);
+
+#undef CLIST_BUTTON_SIGNAL_CONNECT
+
+       gtk_signal_connect(GTK_OBJECT(ctree), "tree_select_row",
+                          GTK_SIGNAL_FUNC(summary_selected), summaryview);
+       gtk_signal_connect(GTK_OBJECT(ctree), "button_press_event",
+                          GTK_SIGNAL_FUNC(summary_button_pressed),
+                          summaryview);
+       gtk_signal_connect(GTK_OBJECT(ctree), "button_release_event",
+                          GTK_SIGNAL_FUNC(summary_button_released),
+                          summaryview);
+       gtk_signal_connect(GTK_OBJECT(ctree), "key_press_event",
+                          GTK_SIGNAL_FUNC(summary_key_pressed), summaryview);
+       gtk_signal_connect(GTK_OBJECT(ctree), "resize_column",
+                          GTK_SIGNAL_FUNC(summary_col_resized), summaryview);
+        gtk_signal_connect(GTK_OBJECT(ctree), "open_row",
+                          GTK_SIGNAL_FUNC(summary_open_row), summaryview);
+
+       gtk_signal_connect_after(GTK_OBJECT(ctree), "tree_expand",
+                                GTK_SIGNAL_FUNC(summary_tree_expanded),
+                                summaryview);
+       gtk_signal_connect_after(GTK_OBJECT(ctree), "tree_collapse",
+                                GTK_SIGNAL_FUNC(summary_tree_collapsed),
+                                summaryview);
+
+       gtk_signal_connect(GTK_OBJECT(ctree), "start_drag",
+                          GTK_SIGNAL_FUNC(summary_start_drag),
+                          summaryview);
+       gtk_signal_connect(GTK_OBJECT(ctree), "drag_data_get",
+                          GTK_SIGNAL_FUNC(summary_drag_data_get),
+                          summaryview);
+
+       return ctree;
+}
+
+void summary_set_column_order(SummaryView *summaryview)
+{
+       GtkWidget *ctree;
+       GtkWidget *scrolledwin = summaryview->scrolledwin;
+       GtkWidget *pixmap;
+       FolderItem *item;
+
+       item = summaryview->folder_item;
+       summary_clear_all(summaryview);
+       gtk_widget_destroy(summaryview->ctree);
+
+       summaryview->ctree = ctree = summary_ctree_create(summaryview);
+       pixmap = gtk_pixmap_new(clipxpm, clipxpmmask);
+       gtk_clist_set_column_widget(GTK_CLIST(ctree),
+                                   summaryview->col_pos[S_COL_MIME], pixmap);
+       gtk_widget_show(pixmap);
+       gtk_scrolled_window_set_hadjustment(GTK_SCROLLED_WINDOW(scrolledwin),
+                                           GTK_CLIST(ctree)->hadjustment);
+       gtk_scrolled_window_set_vadjustment(GTK_SCROLLED_WINDOW(scrolledwin),
+                                           GTK_CLIST(ctree)->vadjustment);
+       gtk_container_add(GTK_CONTAINER(scrolledwin), ctree);
+       gtk_widget_show(ctree);
+
+       summary_show(summaryview, item, FALSE);
+}
+
+
 /* callback functions */
 
 static void summary_toggle_pressed(GtkWidget *eventbox, GdkEventButton *event,
@@ -3756,8 +3879,7 @@ static void summary_button_pressed(GtkWidget *ctree, GdkEventButton *event,
        if (!event) return;
 
        if (event->button == 3) {
-               /* Right button clicked */
-               /* summary_set_add_sender_menu(summaryview); */
+               /* right clicked */
                gtk_menu_popup(GTK_MENU(summaryview->popupmenu), NULL, NULL,
                               NULL, NULL, event->button, event->time);
        } else if (event->button == 2) {
@@ -3974,9 +4096,9 @@ static void summary_key_pressed(GtkWidget *widget, GdkEventKey *event,
 
 static void summary_open_row(GtkSCTree *sctree, SummaryView *summaryview)
 {
-       if (summaryview->folder_item->stype == F_DRAFT  ||
-           summaryview->folder_item->stype == F_OUTBOX ||
-            summaryview->folder_item->stype == F_QUEUE)
+       if (summaryview->folder_item->stype == F_OUTBOX ||
+           summaryview->folder_item->stype == F_DRAFT  ||
+           summaryview->folder_item->stype == F_QUEUE)
                summary_reedit(summaryview);
        else
                summary_open_msg(summaryview);
@@ -4014,7 +4136,7 @@ static void summary_selected(GtkCTree *ctree, GtkCTreeNode *row,
 
        msginfo = gtk_ctree_node_get_row_data(ctree, row);
 
-       switch (column) {
+       switch (column < 0 ? column : summaryview->col_state[column].type) {
        case S_COL_MARK:
                if (MSG_IS_MARKED(msginfo->flags)) {
                        MSG_UNSET_PERM_FLAGS(msginfo->flags, MSG_MARKED);
@@ -4046,37 +4168,9 @@ static void summary_selected(GtkCTree *ctree, GtkCTreeNode *row,
 static void summary_col_resized(GtkCList *clist, gint column, gint width,
                                SummaryView *summaryview)
 {
-       switch (column) {
-       case S_COL_MARK:
-               prefs_common.summary_col_mark = width;
-               break;
-       case S_COL_UNREAD:
-               prefs_common.summary_col_unread = width;
-               break;
-       case S_COL_MIME:
-               prefs_common.summary_col_mime = width;
-               break;
-       case S_COL_NUMBER:
-               prefs_common.summary_col_number = width;
-               break;
-       case S_COL_SCORE:
-               prefs_common.summary_col_score = width;
-               break;
-       case S_COL_SIZE:
-               prefs_common.summary_col_size = width;
-               break;
-       case S_COL_DATE:
-               prefs_common.summary_col_date = width;
-               break;
-       case S_COL_FROM:
-               prefs_common.summary_col_from = width;
-               break;
-       case S_COL_SUBJECT:
-               prefs_common.summary_col_subject = width;
-               break;
-       default:
-               break;
-       }
+       SummaryColumnType type = summaryview->col_state[column].type;
+
+       prefs_common.summary_col_size[type] = width;
 }
 
 static void summary_reply_cb(SummaryView *summaryview, guint action,
@@ -4214,21 +4308,6 @@ static void summary_subject_clicked(GtkWidget *button,
        summary_sort(summaryview, SORT_BY_SUBJECT);
 }
 
-void summary_change_display_item(SummaryView *summaryview)
-{
-       GtkCList *clist = GTK_CLIST(summaryview->ctree);
-
-       gtk_clist_set_column_visibility(clist, S_COL_MARK, prefs_common.show_mark);
-       gtk_clist_set_column_visibility(clist, S_COL_UNREAD, prefs_common.show_unread);
-       gtk_clist_set_column_visibility(clist, S_COL_MIME, prefs_common.show_mime);
-       gtk_clist_set_column_visibility(clist, S_COL_NUMBER, prefs_common.show_number);
-       gtk_clist_set_column_visibility(clist, S_COL_SCORE, prefs_common.show_score);
-       gtk_clist_set_column_visibility(clist, S_COL_SIZE, prefs_common.show_size);
-       gtk_clist_set_column_visibility(clist, S_COL_DATE, prefs_common.show_date);
-       gtk_clist_set_column_visibility(clist, S_COL_FROM, prefs_common.show_from);
-       gtk_clist_set_column_visibility(clist, S_COL_SUBJECT, prefs_common.show_subject);
-}
-
 static void summary_start_drag(GtkWidget *widget, gint button, GdkEvent *event,
                               SummaryView *summaryview)
 {