Trim trailing white spaces.
[geeqie.git] / src / view_file_list.c
index ff2ba51..b0bd169 100644 (file)
@@ -1,7 +1,7 @@
 /*
  * Geeqie
  * (C) 2004 John Ellis
- * Copyright (C) 2008 The Geeqie Team
+ * Copyright (C) 2008 - 2012 The Geeqie Team
  *
  * Author: John Ellis
  *
 #include "main.h"
 #include "view_file_list.h"
 
-#include "bar_info.h"
+#include "bar.h"
 #include "cache_maint.h"
 #include "dnd.h"
 #include "editors.h"
 #include "img-view.h"
-#include "info.h"
 #include "layout.h"
 #include "layout_image.h"
 #include "menu.h"
@@ -38,10 +37,12 @@ enum {
        FILE_COLUMN_POINTER = 0,
        FILE_COLUMN_VERSION,
        FILE_COLUMN_THUMB,
+       FILE_COLUMN_FORMATTED,
        FILE_COLUMN_NAME,
-       FILE_COLUMN_MULTILINE,
+       FILE_COLUMN_SIDECARS,
        FILE_COLUMN_SIZE,
        FILE_COLUMN_DATE,
+       FILE_COLUMN_EXPANDED,
        FILE_COLUMN_COLOR,
        FILE_COLUMN_MARKS,
        FILE_COLUMN_MARKS_LAST = FILE_COLUMN_MARKS + FILEDATA_MARKS_SIZE - 1,
@@ -54,8 +55,7 @@ enum {
        FILE_VIEW_COLUMN_MARKS = 0,
        FILE_VIEW_COLUMN_MARKS_LAST = FILE_VIEW_COLUMN_MARKS + FILEDATA_MARKS_SIZE - 1,
        FILE_VIEW_COLUMN_THUMB,
-       FILE_VIEW_COLUMN_MULTILINE,
-       FILE_VIEW_COLUMN_NAME,
+       FILE_VIEW_COLUMN_FORMATTED,
        FILE_VIEW_COLUMN_SIZE,
        FILE_VIEW_COLUMN_DATE,
        FILE_VIEW_COLUMN_COUNT
@@ -63,9 +63,11 @@ enum {
 
 
 
-static gint vflist_row_is_selected(ViewFile *vf, FileData *fd);
-static gint vflist_row_rename_cb(TreeEditData *td, const gchar *old, const gchar *new, gpointer data);
-static void vflist_populate_view(ViewFile *vf);
+static gboolean vflist_row_is_selected(ViewFile *vf, FileData *fd);
+static gboolean vflist_row_rename_cb(TreeEditData *td, const gchar *old, const gchar *new, gpointer data);
+static void vflist_populate_view(ViewFile *vf, gboolean force);
+static gboolean vflist_is_multiline(ViewFile *vf);
+static void vflist_set_expanded(ViewFile *vf, GtkTreeIter *iter, gboolean expanded);
 
 
 /*
@@ -76,7 +78,7 @@ static void vflist_populate_view(ViewFile *vf);
 typedef struct {
        FileData *fd;
        GtkTreeIter *iter;
-       gint found;
+       gboolean found;
        gint row;
 } ViewFileFindRowData;
 
@@ -88,7 +90,7 @@ static gboolean vflist_find_row_cb(GtkTreeModel *model, GtkTreePath *path, GtkTr
        if (fd == find->fd)
                {
                *find->iter = *iter;
-               find->found = 1;
+               find->found = TRUE;
                return TRUE;
                }
        find->row++;
@@ -98,7 +100,7 @@ static gboolean vflist_find_row_cb(GtkTreeModel *model, GtkTreePath *path, GtkTr
 static gint vflist_find_row(ViewFile *vf, FileData *fd, GtkTreeIter *iter)
 {
        GtkTreeModel *store;
-       ViewFileFindRowData data = {fd, iter, 0, 0};
+       ViewFileFindRowData data = {fd, iter, FALSE, 0};
 
        store = gtk_tree_view_get_model(GTK_TREE_VIEW(vf->listview));
        gtk_tree_model_foreach(store, vflist_find_row_cb, &data);
@@ -134,55 +136,14 @@ static FileData *vflist_find_data_by_coord(ViewFile *vf, gint x, gint y, GtkTree
        return NULL;
 }
 
-#if 0
-static gint vflist_find_sidecar_list_idx(GList *work, FileData *fd)
-{
-       gint i = 0;
-       while (work)
-               {
-               FileData *fd_p = work->data;
-               if (fd == fd_p) return i;
-
-               i++;
-
-               GList *work2 = fd_p->sidecar_files;
-               while (work2)
-                       {
-                       fd_p = work2->data;
-                       if (fd == fd_p) return i;
-
-                       i++;
-                       work2 = work2->next;
-                       }
-               work = work->next;
-               }
-       return -1;
-}
-
-static gint vflist_sidecar_list_count(GList *work)
-{
-       gint i = 0;
-       while (work)
-               {
-               FileData *fd = work->data;
-               i++;
-
-               GList *work2 = fd->sidecar_files;
-               while (work2)
-                       {
-                       i++;
-                       work2 = work2->next;
-                       }
-               work = work->next;
-               }
-       return i;
-}
-#endif
-
 static gboolean vflist_store_clear_cb(GtkTreeModel *model, GtkTreePath *path, GtkTreeIter *iter, gpointer data)
 {
        FileData *fd;
        gtk_tree_model_get(model, iter, FILE_COLUMN_POINTER, &fd, -1);
+
+       /* it seems that gtk_tree_store_clear may call some callbacks
+          that use the column. Set the pointer to NULL to be safe. */
+       gtk_tree_store_set(GTK_TREE_STORE(model), iter, FILE_COLUMN_POINTER, NULL, -1);
        file_data_unref(fd);
        return FALSE;
 }
@@ -195,7 +156,7 @@ static void vflist_store_clear(ViewFile *vf)
        gtk_tree_store_clear(GTK_TREE_STORE(store));
 }
 
-void vflist_color_set(ViewFile *vf, FileData *fd, gint color_set)
+void vflist_color_set(ViewFile *vf, FileData *fd, gboolean color_set)
 {
        GtkTreeModel *store;
        GtkTreeIter iter;
@@ -218,27 +179,6 @@ static void vflist_move_cursor(ViewFile *vf, GtkTreeIter *iter)
 }
 
 
-static gint vflist_column_idx(ViewFile *vf, gint store_idx)
-{
-       GList *columns, *work;
-       gint i = 0;
-
-       columns = gtk_tree_view_get_columns(GTK_TREE_VIEW(vf->listview));
-       work = columns;
-       while (work)
-               {
-               GtkTreeViewColumn *column = work->data;
-               if (store_idx == GPOINTER_TO_INT(g_object_get_data (G_OBJECT(column), "column_store_idx")))
-                       break;
-               work = work->next;
-               i++;
-               }
-
-       g_list_free(columns);
-       return i;
-}
-
-
 /*
  *-----------------------------------------------------------------------------
  * dnd
@@ -251,8 +191,6 @@ static void vflist_dnd_get(GtkWidget *widget, GdkDragContext *context,
 {
        ViewFile *vf = data;
        GList *list = NULL;
-       gchar *uri_text = NULL;
-       gint total;
 
        if (!VFLIST(vf)->click_fd) return;
 
@@ -266,15 +204,8 @@ static void vflist_dnd_get(GtkWidget *widget, GdkDragContext *context,
                }
 
        if (!list) return;
-
-       uri_text = uri_text_from_filelist(list, &total, (info == TARGET_TEXT_PLAIN));
+       uri_selection_data_set_uris_from_filelist(selection_data, list);
        filelist_free(list);
-
-       DEBUG_1("%s", uri_text);
-
-       gtk_selection_data_set(selection_data, selection_data->target,
-                              8, (guchar *)uri_text, total);
-       g_free(uri_text);
 }
 
 static void vflist_dnd_begin(GtkWidget *widget, GdkDragContext *context, gpointer data)
@@ -303,7 +234,7 @@ static void vflist_dnd_end(GtkWidget *widget, GdkDragContext *context, gpointer
 
        vflist_color_set(vf, VFLIST(vf)->click_fd, FALSE);
 
-       if (context->action == GDK_ACTION_MOVE)
+       if (gdk_drag_context_get_selected_action(context) == GDK_ACTION_MOVE)
                {
                vf_refresh(vf);
                }
@@ -320,15 +251,12 @@ static void vflist_drag_data_received(GtkWidget *entry_widget, GdkDragContext *c
 
                if (fd) {
                        /* Add keywords to file */
-                       gchar *str = g_strndup(selection->data, selection->length);
+                       gchar *str = (gchar *) gtk_selection_data_get_text(selection);
                        GList *kw_list = string_to_keywords_list(str);
                        
-                       metadata_set(fd, kw_list, NULL, TRUE);
+                       metadata_append_list(fd, KEYWORD_KEY, kw_list);
                        string_list_free(kw_list);
                        g_free(str);
-                       if (vf->layout && vf->layout->bar_info) {
-                               bar_info_set(vf->layout->bar_info, fd);
-                       }
                }
        }
 }
@@ -358,6 +286,41 @@ void vflist_dnd_init(ViewFile *vf)
  *-----------------------------------------------------------------------------
  */
 
+GList *vflist_selection_get_one(ViewFile *vf, FileData *fd)
+{
+       GList *list = g_list_append(NULL, file_data_ref(fd));
+
+       if (fd->sidecar_files)
+               {
+               /* check if the row is expanded */
+               GtkTreeModel *store;
+               GtkTreeIter iter;
+               
+               store = gtk_tree_view_get_model(GTK_TREE_VIEW(vf->listview));
+               if (vflist_find_row(vf, fd, &iter) >= 0)
+                       {
+                       GtkTreePath *tpath;
+
+                       tpath = gtk_tree_model_get_path(store, &iter);
+                       if (!gtk_tree_view_row_expanded(GTK_TREE_VIEW(vf->listview), tpath))
+                               {
+                               /* unexpanded - add whole group */
+                               GList *work = fd->sidecar_files;
+                               while (work)
+                                       {
+                                       FileData *sfd = work->data;
+                                       list = g_list_prepend(list, file_data_ref(sfd));
+                                       work = work->next;
+                                       }
+                               }
+                       gtk_tree_path_free(tpath);
+                       }
+               list = g_list_reverse(list);
+               }
+
+       return list;
+}
+
 GList *vflist_pop_menu_file_list(ViewFile *vf)
 {
        if (!VFLIST(vf)->click_fd) return NULL;
@@ -366,10 +329,10 @@ GList *vflist_pop_menu_file_list(ViewFile *vf)
                {
                return vf_selection_get_list(vf);
                }
-
-       return g_list_append(NULL, file_data_ref(VFLIST(vf)->click_fd));
+       return vflist_selection_get_one(vf, VFLIST(vf)->click_fd);
 }
 
+
 void vflist_pop_menu_view_cb(GtkWidget *widget, gpointer data)
 {
        ViewFile *vf = data;
@@ -409,7 +372,7 @@ void vflist_pop_menu_rename_cb(GtkWidget *widget, gpointer data)
 
                        tpath = gtk_tree_model_get_path(store, &iter);
                        tree_edit_by_path(GTK_TREE_VIEW(vf->listview), tpath,
-                                         vflist_column_idx(vf, FILE_COLUMN_NAME), VFLIST(vf)->click_fd->name,
+                                         FILE_VIEW_COLUMN_FORMATTED, VFLIST(vf)->click_fd->name,
                                          vflist_row_rename_cb, vf);
                        gtk_tree_path_free(tpath);
                        }
@@ -440,6 +403,7 @@ void vflist_pop_menu_refresh_cb(GtkWidget *widget, gpointer data)
 
        vflist_color_set(vf, VFLIST(vf)->click_fd, FALSE);
        vf_refresh(vf);
+       gtk_tree_view_columns_autosize(GTK_TREE_VIEW(vf->listview));
 }
 
 void vflist_popup_destroy_cb(GtkWidget *widget, gpointer data)
@@ -457,15 +421,13 @@ void vflist_popup_destroy_cb(GtkWidget *widget, gpointer data)
  *-----------------------------------------------------------------------------
  */
 
-static gint vflist_row_rename_cb(TreeEditData *td, const gchar *old, const gchar *new, gpointer data)
+static gboolean vflist_row_rename_cb(TreeEditData *td, const gchar *old, const gchar *new, gpointer data)
 {
        ViewFile *vf = data;
-       gchar *old_path;
        gchar *new_path;
 
-       if (strlen(new) == 0) return FALSE;
+       if (!new || !new[0]) return FALSE;
 
-       old_path = g_build_filename(vf->dir_fd->path, old, NULL);
        new_path = g_build_filename(vf->dir_fd->path, new, NULL);
 
        if (strchr(new, G_DIR_SEPARATOR) != NULL)
@@ -476,11 +438,13 @@ static gint vflist_row_rename_cb(TreeEditData *td, const gchar *old, const gchar
                }
        else
                {
-               FileData *fd = file_data_new_simple(old_path); /* get the fd from cache */
+               gchar *old_path = g_build_filename(vf->dir_fd->path, old, NULL);
+               FileData *fd = file_data_new_group(old_path); /* get the fd from cache */
                file_util_rename_simple(fd, new_path, vf->listview);
                file_data_unref(fd);
+               g_free(old_path);
                }
-       g_free(old_path);
+       
        g_free(new_path);
 
        return FALSE;
@@ -503,12 +467,12 @@ static void vflist_menu_position_cb(GtkMenu *menu, gint *x, gint *y, gboolean *p
        popup_menu_position_clamp(menu, x, y, 0);
 }
 
-gint vflist_press_key_cb(GtkWidget *widget, GdkEventKey *event, gpointer data)
+gboolean vflist_press_key_cb(GtkWidget *widget, GdkEventKey *event, gpointer data)
 {
        ViewFile *vf = data;
        GtkTreePath *tpath;
 
-       if (event->keyval != GDK_Menu) return FALSE;
+       if (event->keyval != GDK_KEY_Menu) return FALSE;
 
        gtk_tree_view_get_cursor(GTK_TREE_VIEW(vf->listview), &tpath, NULL);
        if (tpath)
@@ -532,7 +496,7 @@ gint vflist_press_key_cb(GtkWidget *widget, GdkEventKey *event, gpointer data)
        return TRUE;
 }
 
-gint vflist_press_cb(GtkWidget *widget, GdkEventButton *bevent, gpointer data)
+gboolean vflist_press_cb(GtkWidget *widget, GdkEventButton *bevent, gpointer data)
 {
        ViewFile *vf = data;
        GtkTreePath *tpath;
@@ -559,9 +523,6 @@ gint vflist_press_cb(GtkWidget *widget, GdkEventButton *bevent, gpointer data)
 
                gtk_tree_model_get_iter(store, &iter, tpath);
                gtk_tree_model_get(store, &iter, FILE_COLUMN_POINTER, &fd, -1);
-#if 0
-               gtk_tree_view_set_cursor(GTK_TREE_VIEW(widget), tpath, NULL, FALSE);
-#endif
                gtk_tree_path_free(tpath);
                }
 
@@ -605,17 +566,15 @@ gint vflist_press_cb(GtkWidget *widget, GdkEventButton *bevent, gpointer data)
                return (gtk_tree_selection_count_selected_rows(selection) > 1);
                }
 
-#if 1
        if (bevent->button == MOUSE_BUTTON_LEFT && bevent->type == GDK_2BUTTON_PRESS)
                {
                if (vf->layout) layout_image_full_screen_start(vf->layout);
                }
-#endif
 
        return FALSE;
 }
 
-gint vflist_release_cb(GtkWidget *widget, GdkEventButton *bevent, gpointer data)
+gboolean vflist_release_cb(GtkWidget *widget, GdkEventButton *bevent, gpointer data)
 {
        ViewFile *vf = data;
        GtkTreePath *tpath;
@@ -674,7 +633,6 @@ gint vflist_release_cb(GtkWidget *widget, GdkEventButton *bevent, gpointer data)
                gtk_tree_selection_unselect_all(selection);
                gtk_tree_selection_select_iter(selection, &iter);
                vflist_move_cursor(vf, &iter);
-//             return TRUE;// FIXME - expand
                }
 
        return FALSE;
@@ -685,6 +643,7 @@ static void vflist_select_image(ViewFile *vf, FileData *sel_fd)
        FileData *read_ahead_fd = NULL;
        gint row;
        FileData *cur_fd;
+
        if (!sel_fd) return;
 
        cur_fd = layout_image_get_fd(vf->layout);
@@ -709,13 +668,13 @@ static void vflist_select_image(ViewFile *vf, FileData *sel_fd)
        layout_image_set_with_ahead(vf->layout, sel_fd, read_ahead_fd);
 }
 
-static gint vflist_select_idle_cb(gpointer data)
+static gboolean vflist_select_idle_cb(gpointer data)
 {
        ViewFile *vf = data;
 
        if (!vf->layout)
                {
-               VFLIST(vf)->select_idle_id = -1;
+               VFLIST(vf)->select_idle_id = 0;
                return FALSE;
                }
 
@@ -727,14 +686,17 @@ static gint vflist_select_idle_cb(gpointer data)
                VFLIST(vf)->select_fd = NULL;
                }
 
-       VFLIST(vf)->select_idle_id = -1;
+       VFLIST(vf)->select_idle_id = 0;
        return FALSE;
 }
 
 static void vflist_select_idle_cancel(ViewFile *vf)
 {
-       if (VFLIST(vf)->select_idle_id != -1) g_source_remove(VFLIST(vf)->select_idle_id);
-       VFLIST(vf)->select_idle_id = -1;
+       if (VFLIST(vf)->select_idle_id)
+               {
+               g_source_remove(VFLIST(vf)->select_idle_id);
+               VFLIST(vf)->select_idle_id = 0;
+               }
 }
 
 static gboolean vflist_select_cb(GtkTreeSelection *selection, GtkTreeModel *store, GtkTreePath *tpath,
@@ -754,7 +716,7 @@ static gboolean vflist_select_cb(GtkTreeSelection *selection, GtkTreeModel *stor
                }
 
        if (vf->layout &&
-           VFLIST(vf)->select_idle_id == -1)
+           !VFLIST(vf)->select_idle_id)
                {
                VFLIST(vf)->select_idle_id = g_idle_add(vflist_select_idle_cb, vf);
                }
@@ -762,43 +724,100 @@ static gboolean vflist_select_cb(GtkTreeSelection *selection, GtkTreeModel *stor
        return TRUE;
 }
 
+static void vflist_expand_cb(GtkTreeView *tree_view, GtkTreeIter *iter, GtkTreePath *path, gpointer data)
+{
+       ViewFile *vf = data;
+       vflist_set_expanded(vf, iter, TRUE);
+}
+
+static void vflist_collapse_cb(GtkTreeView *tree_view, GtkTreeIter *iter, GtkTreePath *path, gpointer data)
+{
+       ViewFile *vf = data;
+       vflist_set_expanded(vf, iter, FALSE);
+}
+
 /*
  *-----------------------------------------------------------------------------
  * misc
  *-----------------------------------------------------------------------------
  */
 
-/*
-static gboolean vflist_dummy_select_cb(GtkTreeSelection *selection, GtkTreeModel *store, GtkTreePath *tpath,
-                                       gboolean path_currently_selected, gpointer data)
+
+static gchar* vflist_get_formatted(ViewFile *vf, const gchar *name, const gchar *sidecars, const gchar *size, const gchar *time, gboolean expanded)
+ {
+       gboolean multiline = vflist_is_multiline(vf);
+       gchar *text;
+
+       if (multiline)
+               {
+               text = g_strdup_printf("%s %s\n%s\n%s", name, expanded ? "" : sidecars, size, time);
+               }
+       else
+               {
+               text = g_strdup_printf("%s %s", name, expanded ? "" : sidecars);
+               }
+       return text;
+}
+
+static void vflist_set_expanded(ViewFile *vf, GtkTreeIter *iter, gboolean expanded)
 {
-       return TRUE;
+       GtkTreeStore *store;
+       gchar *name;
+       gchar *sidecars;
+       gchar *size;
+       gchar *time;
+       gchar *formatted;
+
+       store = GTK_TREE_STORE(gtk_tree_view_get_model(GTK_TREE_VIEW(vf->listview)));
+
+       gtk_tree_model_get(GTK_TREE_MODEL(store), iter,
+                                       FILE_COLUMN_NAME, &name,
+                                       FILE_COLUMN_SIDECARS, &sidecars,
+                                       FILE_COLUMN_SIZE, &size,
+                                       FILE_COLUMN_DATE, &time,
+                                       -1);
+       formatted = vflist_get_formatted(vf, name, sidecars, size, time, expanded);
+
+       gtk_tree_store_set(store, iter, FILE_COLUMN_FORMATTED, formatted,
+                                       FILE_COLUMN_EXPANDED, expanded,
+                                       -1);
+       g_free(time);
+       g_free(size);
+       g_free(sidecars);
+       g_free(name);
+       g_free(formatted);
 }
-*/
 
 static void vflist_setup_iter(ViewFile *vf, GtkTreeStore *store, GtkTreeIter *iter, FileData *fd)
 {
        gchar *size;
        gchar *sidecars = NULL;
-       gchar *name_sidecars;
-       gchar *multiline;
+       gchar *name;
        const gchar *time = text_from_time(fd->date);
-       name_sidecars = (gchar *)fd->name;
-
-       if (fd->sidecar_files)
+       gchar *link = islink(fd->path) ? GQ_LINK_STR : "";
+       const gchar *disabled_grouping;
+       gchar *formatted;
+       gboolean expanded = FALSE;
+       
+       if (fd->sidecar_files) /* expanded has no effect on files without sidecars */
                {
-               sidecars = file_data_sc_list_to_string(fd);
-               name_sidecars = g_strdup_printf("%s %s", fd->name, sidecars);
+               gtk_tree_model_get(GTK_TREE_MODEL(store), iter, FILE_COLUMN_EXPANDED, &expanded, -1);
                }
+
+       sidecars = file_data_sc_list_to_string(fd);
+
+       disabled_grouping = fd->disable_grouping ? _(" [NO GROUPING]") : "";
+       name = g_strdup_printf("%s%s%s", link, fd->name, disabled_grouping);
        size = text_from_size(fd->size);
        
-       multiline = g_strdup_printf("%s\n%s\n%s", name_sidecars, size, time);
-
+       formatted = vflist_get_formatted(vf, name, sidecars, size, time, expanded);
+       
        gtk_tree_store_set(store, iter, FILE_COLUMN_POINTER, fd,
                                        FILE_COLUMN_VERSION, fd->version,
                                        FILE_COLUMN_THUMB, fd->thumb_pixbuf,
-                                       FILE_COLUMN_MULTILINE, multiline,
-                                       FILE_COLUMN_NAME, name_sidecars,
+                                       FILE_COLUMN_FORMATTED, formatted,
+                                       FILE_COLUMN_SIDECARS, sidecars,
+                                       FILE_COLUMN_NAME, name,
                                        FILE_COLUMN_SIZE, size,
                                        FILE_COLUMN_DATE, time,
 #define STORE_SET_IS_SLOW 1
@@ -824,19 +843,16 @@ static void vflist_setup_iter(ViewFile *vf, GtkTreeStore *store, GtkTreeIter *it
        }
 #endif
        g_free(size);
-       if (sidecars)
-               {
-               g_free(sidecars);
-               g_free(name_sidecars);
-               }
-       g_free(multiline);
+       g_free(sidecars);
+       g_free(name);
+       g_free(formatted);
 }
 
-static void vflist_setup_iter_recursive(ViewFile *vf, GtkTreeStore *store, GtkTreeIter *parent_iter, GList *list, GList *selected)
+static void vflist_setup_iter_recursive(ViewFile *vf, GtkTreeStore *store, GtkTreeIter *parent_iter, GList *list, GList *selected, gboolean force)
 {
        GList *work;
        GtkTreeIter iter;
-       gint valid;
+       gboolean valid;
        gint num_ordered = 0;
        gint num_prepended = 0;
 
@@ -847,7 +863,7 @@ static void vflist_setup_iter_recursive(ViewFile *vf, GtkTreeStore *store, GtkTr
                {
                gint match;
                FileData *fd = work->data;
-               gint done = FALSE;
+               gboolean done = FALSE;
 
                while (!done)
                        {
@@ -856,7 +872,6 @@ static void vflist_setup_iter_recursive(ViewFile *vf, GtkTreeStore *store, GtkTr
 
                        if (valid)
                                {
-                               num_ordered++;
                                gtk_tree_model_get(GTK_TREE_MODEL(store), &iter,
                                                   FILE_COLUMN_POINTER, &old_fd,
                                                   FILE_COLUMN_VERSION, &old_version,
@@ -888,6 +903,7 @@ static void vflist_setup_iter_recursive(ViewFile *vf, GtkTreeStore *store, GtkTr
 
                                if (valid)
                                        {
+                                       num_ordered++;
                                        gtk_tree_store_insert_before(store, &new, parent_iter, &iter);
                                        }
                                else
@@ -901,7 +917,7 @@ static void vflist_setup_iter_recursive(ViewFile *vf, GtkTreeStore *store, GtkTr
                                        }
 
                                vflist_setup_iter(vf, store, &new, file_data_ref(fd));
-                               vflist_setup_iter_recursive(vf, store, &new, fd->sidecar_files, selected);
+                               vflist_setup_iter_recursive(vf, store, &new, fd->sidecar_files, selected, force);
                                
                                if (g_list_find(selected, fd))
                                        {
@@ -920,10 +936,11 @@ static void vflist_setup_iter_recursive(ViewFile *vf, GtkTreeStore *store, GtkTr
                                }
                        else
                                {
-                               if (fd->version != old_version)
+                               num_ordered++;
+                               if (fd->version != old_version || force)
                                        {
                                        vflist_setup_iter(vf, store, &iter, fd);
-                                       vflist_setup_iter_recursive(vf, store, &iter, fd->sidecar_files, selected);
+                                       vflist_setup_iter_recursive(vf, store, &iter, fd->sidecar_files, selected, force);
                                        }
 
                                if (valid) valid = gtk_tree_model_iter_next(GTK_TREE_MODEL(store), &iter);
@@ -963,7 +980,7 @@ static void vflist_setup_iter_recursive(ViewFile *vf, GtkTreeStore *store, GtkTr
                }
 }
 
-void vflist_sort_set(ViewFile *vf, SortType type, gint ascend)
+void vflist_sort_set(ViewFile *vf, SortType type, gboolean ascend)
 {
        gint i;
        GHashTable *fd_idx_hash = g_hash_table_new(NULL, NULL);
@@ -1014,9 +1031,8 @@ void vflist_sort_set(ViewFile *vf, SortType type, gint ascend)
  *-----------------------------------------------------------------------------
  */
 
-static gint vflist_thumb_next(ViewFile *vf);
 
-static void vflist_thumb_progress_count(GList *list, gint *count, gint *done)
+void vflist_thumb_progress_count(GList *list, gint *count, gint *done)
 {
        GList *work = list;
        while (work)
@@ -1034,44 +1050,7 @@ static void vflist_thumb_progress_count(GList *list, gint *count, gint *done)
                }
 }
 
-static gdouble vflist_thumb_progress(ViewFile *vf)
-{
-       gint count = 0;
-       gint done = 0;
-       
-       vflist_thumb_progress_count(vf->list, &count, &done);
-
-       DEBUG_1("thumb progress: %d of %d", done, count);
-       return (gdouble)done / count;
-}
-
-
-static void vflist_thumb_status(ViewFile *vf, gdouble val, const gchar *text)
-{
-       if (vf->func_thumb_status)
-               {
-               vf->func_thumb_status(vf, val, text, vf->data_thumb_status);
-               }
-}
-
-static void vflist_thumb_cleanup(ViewFile *vf)
-{
-       vflist_thumb_status(vf, 0.0, NULL);
-
-       vf->thumbs_running = FALSE;
-
-       thumb_loader_free(vf->thumbs_loader);
-       vf->thumbs_loader = NULL;
-
-       vf->thumbs_filedata = NULL;
-}
-
-static void vflist_thumb_stop(ViewFile *vf)
-{
-       if (vf->thumbs_running) vflist_thumb_cleanup(vf);
-}
-
-static void vflist_thumb_do(ViewFile *vf, ThumbLoader *tl, FileData *fd)
+void vflist_set_thumb_fd(ViewFile *vf, FileData *fd)
 {
        GtkTreeStore *store;
        GtkTreeIter iter;
@@ -1080,56 +1059,32 @@ static void vflist_thumb_do(ViewFile *vf, ThumbLoader *tl, FileData *fd)
 
        store = GTK_TREE_STORE(gtk_tree_view_get_model(GTK_TREE_VIEW(vf->listview)));
        gtk_tree_store_set(store, &iter, FILE_COLUMN_THUMB, fd->thumb_pixbuf, -1);
-
-       vflist_thumb_status(vf, vflist_thumb_progress(vf), _("Loading thumbs..."));
-}
-
-static void vflist_thumb_error_cb(ThumbLoader *tl, gpointer data)
-{
-       ViewFile *vf = data;
-
-       if (vf->thumbs_filedata && vf->thumbs_loader == tl)
-               {
-               vflist_thumb_do(vf, tl, vf->thumbs_filedata);
-               }
-
-       while (vflist_thumb_next(vf));
-}
-
-static void vflist_thumb_done_cb(ThumbLoader *tl, gpointer data)
-{
-       ViewFile *vf = data;
-
-       if (vf->thumbs_filedata && vf->thumbs_loader == tl)
-               {
-               vflist_thumb_do(vf, tl, vf->thumbs_filedata);
-               }
-
-       while (vflist_thumb_next(vf));
 }
 
-static gint vflist_thumb_next(ViewFile *vf)
+FileData *vflist_thumb_next_fd(ViewFile *vf)
 {
        GtkTreePath *tpath;
        FileData *fd = NULL;
 
        /* first check the visible files */
 
-       if (GTK_WIDGET_REALIZED(vf->listview) &&
-           gtk_tree_view_get_path_at_pos(GTK_TREE_VIEW(vf->listview), 0, 0, &tpath, NULL, NULL, NULL))
+       if (gtk_tree_view_get_path_at_pos(GTK_TREE_VIEW(vf->listview), 0, 0, &tpath, NULL, NULL, NULL))
                {
                GtkTreeModel *store;
                GtkTreeIter iter;
-               gint valid = TRUE;
-
+               gboolean valid = TRUE;
+       
                store = gtk_tree_view_get_model(GTK_TREE_VIEW(vf->listview));
                gtk_tree_model_get_iter(store, &iter, tpath);
                gtk_tree_path_free(tpath);
 
                while (!fd && valid && tree_view_row_get_visibility(GTK_TREE_VIEW(vf->listview), &iter, FALSE) == 0)
                        {
-                       gtk_tree_model_get(store, &iter, FILE_COLUMN_POINTER, &fd, -1);
-                       if (fd->thumb_pixbuf) fd = NULL;
+                       FileData *nfd;
+
+                       gtk_tree_model_get(store, &iter, FILE_COLUMN_POINTER, &nfd, -1);
+
+                       if (!nfd->thumb_pixbuf) fd = nfd;
 
                        valid = gtk_tree_model_iter_next(store, &iter);
                        }
@@ -1160,45 +1115,23 @@ static gint vflist_thumb_next(ViewFile *vf)
                        }
                }
 
-       if (!fd)
-               {
-               /* done */
-               vflist_thumb_cleanup(vf);
-               return FALSE;
-               }
-
-       vf->thumbs_filedata = fd;
-
-       thumb_loader_free(vf->thumbs_loader);
+       return fd;
+}
 
-       vf->thumbs_loader = thumb_loader_new(options->thumbnails.max_width, options->thumbnails.max_height);
-       thumb_loader_set_callbacks(vf->thumbs_loader,
-                                  vflist_thumb_done_cb,
-                                  vflist_thumb_error_cb,
-                                  NULL,
-                                  vf);
 
-       if (!thumb_loader_start(vf->thumbs_loader, fd))
+void vflist_thumb_reset_all(ViewFile *vf)
+{
+       GList *work = vf->list;
+       while (work)
                {
-               /* set icon to unknown, continue */
-               DEBUG_1("thumb loader start failed %s", fd->path);
-               vflist_thumb_do(vf, vf->thumbs_loader, fd);
-
-               return TRUE;
+               FileData *fd = work->data;
+               if (fd->thumb_pixbuf)
+                       {
+                       g_object_unref(fd->thumb_pixbuf);
+                       fd->thumb_pixbuf = NULL;
+                       }
+               work = work->next;
                }
-
-       return FALSE;
-}
-
-static void vflist_thumb_update(ViewFile *vf)
-{
-       vflist_thumb_stop(vf);
-       if (!VFLIST(vf)->thumbs_enabled) return;
-
-       vflist_thumb_status(vf, 0.0, _("Loading thumbs..."));
-       vf->thumbs_running = TRUE;
-
-       while (vflist_thumb_next(vf));
 }
 
 /*
@@ -1212,18 +1145,28 @@ FileData *vflist_index_get_data(ViewFile *vf, gint row)
        return g_list_nth_data(vf->list, row);
 }
 
-gint vflist_index_by_path(ViewFile *vf, const gchar *path)
+gint vflist_index_by_fd(ViewFile *vf, FileData *fd)
 {
        gint p = 0;
-       GList *work;
-
-       if (!path) return -1;
+       GList *work, *work2;
 
        work = vf->list;
        while (work)
                {
-               FileData *fd = work->data;
-               if (strcmp(path, fd->path) == 0) return p;
+               FileData *list_fd = work->data;
+               if (list_fd == fd) return p;
+               
+               work2 = list_fd->sidecar_files;
+               while (work2)
+                       {
+                       /* FIXME: return the same index also for sidecars
+                          it is sufficient for next/prev navigation but it should be rewritten
+                          without using indexes at all
+                       */
+                       FileData *sidecar_fd = work2->data;
+                       if (sidecar_fd == fd) return p;
+                       work2 = work2->next;
+                       }
                
                work = work->next;
                p++;
@@ -1276,13 +1219,13 @@ GList *vflist_get_list(ViewFile *vf)
  *-----------------------------------------------------------------------------
  */
 
-static gint vflist_row_is_selected(ViewFile *vf, FileData *fd)
+static gboolean vflist_row_is_selected(ViewFile *vf, FileData *fd)
 {
        GtkTreeModel *store;
        GtkTreeSelection *selection;
        GList *slist;
        GList *work;
-       gint found = FALSE;
+       gboolean found = FALSE;
 
        selection = gtk_tree_view_get_selection(GTK_TREE_VIEW(vf->listview));
        slist = gtk_tree_selection_get_selected_rows(selection, &store);
@@ -1304,7 +1247,7 @@ static gint vflist_row_is_selected(ViewFile *vf, FileData *fd)
        return found;
 }
 
-gint vflist_index_is_selected(ViewFile *vf, gint row)
+gboolean vflist_index_is_selected(ViewFile *vf, gint row)
 {
        FileData *fd;
 
@@ -1372,6 +1315,18 @@ GList *vflist_selection_get_list(ViewFile *vf)
                gtk_tree_model_get(store, &iter, FILE_COLUMN_POINTER, &fd, -1);
 
                list = g_list_prepend(list, file_data_ref(fd));
+               
+               if (!fd->parent && !gtk_tree_view_row_expanded(GTK_TREE_VIEW(vf->listview), tpath))
+                       {
+                       /* unexpanded - add whole group */
+                       GList *work2 = fd->sidecar_files;
+                       while (work2)
+                               {
+                               FileData *sfd = work2->data;
+                               list = g_list_prepend(list, file_data_ref(sfd));
+                               work2 = work2->next;
+                               }
+                       }
 
                work = work->next;
                }
@@ -1478,7 +1433,7 @@ void vflist_select_invert(ViewFile *vf)
 
        while (valid)
                {
-               gint selected = gtk_tree_selection_iter_is_selected(selection, &iter);
+               gboolean selected = gtk_tree_selection_iter_is_selected(selection, &iter);
 
                if (selected)
                        gtk_tree_selection_unselect_iter(selection, &iter);
@@ -1518,6 +1473,7 @@ void vflist_select_by_fd(ViewFile *vf, FileData *fd)
 static void vflist_select_closest(ViewFile *vf, FileData *sel_fd)
 {
        GList *work;
+       FileData *fd = NULL;
        
        if (sel_fd->parent) sel_fd = sel_fd->parent;
        work = vf->list;
@@ -1525,19 +1481,16 @@ static void vflist_select_closest(ViewFile *vf, FileData *sel_fd)
        while (work)
                {
                gint match;
-               FileData *fd = work->data;
+               fd = work->data;
                work = work->next;
-               
 
                match = filelist_sort_compare_filedata_full(fd, sel_fd, vf->sort_method, vf->sort_ascend);
                
-               if (match >= 0)
-                       {
-                       vflist_select_by_fd(vf, fd);
-                       break;
-                       }
+               if (match >= 0) break;
                }
 
+       if (fd) vflist_select_by_fd(vf, fd);
+
 }
 
 void vflist_mark_to_selection(ViewFile *vf, gint mark, MarkToSelectionMode mode)
@@ -1545,7 +1498,7 @@ void vflist_mark_to_selection(ViewFile *vf, gint mark, MarkToSelectionMode mode)
        GtkTreeModel *store;
        GtkTreeIter iter;
        GtkTreeSelection *selection;
-       gint valid;
+       gboolean valid;
        gint n = mark - 1;
 
        g_assert(mark >= 1 && mark <= FILEDATA_MARKS_SIZE);
@@ -1567,11 +1520,11 @@ void vflist_mark_to_selection(ViewFile *vf, gint mark, MarkToSelectionMode mode)
                        {
                        case MTS_MODE_SET: selected = mark_val;
                                break;
-                       case MTS_MODE_OR: selected = mark_val | selected;
+                       case MTS_MODE_OR: selected = mark_val || selected;
                                break;
-                       case MTS_MODE_AND: selected = mark_val & selected;
+                       case MTS_MODE_AND: selected = mark_val && selected;
                                break;
-                       case MTS_MODE_MINUS: selected = !mark_val & selected;
+                       case MTS_MODE_MINUS: selected = !mark_val && selected;
                                break;
                        }
 
@@ -1606,6 +1559,8 @@ void vflist_selection_to_mark(ViewFile *vf, gint mark, SelectionToMarkMode mode)
                gtk_tree_model_get_iter(store, &iter, tpath);
                gtk_tree_model_get(store, &iter, FILE_COLUMN_POINTER, &fd, -1);
 
+               /* the change has a very limited range and the standard notification would trigger
+                  complete re-read of the directory - try to do only minimal update instead */
                file_data_unregister_notify_func(vf_notify_cb, vf); /* we don't need the notification */
 
                switch (mode)
@@ -1622,12 +1577,17 @@ void vflist_selection_to_mark(ViewFile *vf, gint mark, SelectionToMarkMode mode)
                        {
                        vf_refresh_idle(vf);
                        }
+               else
+                       {
+                       /* mark functions can have various side effects - update all columns to be sure */
+                       vflist_setup_iter(vf, GTK_TREE_STORE(store), &iter, fd);
+                       /* mark functions can change sidecars too */
+                       vflist_setup_iter_recursive(vf, GTK_TREE_STORE(store), &iter, fd->sidecar_files, NULL, FALSE);
+                       }
 
                
                file_data_register_notify_func(vf_notify_cb, vf, NOTIFY_PRIORITY_MEDIUM);
 
-               gtk_tree_store_set(GTK_TREE_STORE(store), &iter, FILE_COLUMN_MARKS + n, file_data_get_mark(fd, n), -1);
-
                work = work->next;
                }
        g_list_foreach(slist, (GFunc)gtk_tree_path_free, NULL);
@@ -1640,19 +1600,18 @@ void vflist_selection_to_mark(ViewFile *vf, gint mark, SelectionToMarkMode mode)
  *-----------------------------------------------------------------------------
  */
 
-static void vflist_listview_set_columns(GtkWidget *listview, gint thumb)
+static void vflist_listview_set_columns(GtkWidget *listview, gboolean thumb, gboolean multiline)
 {
        GtkTreeViewColumn *column;
        GtkCellRenderer *cell;
        GList *list;
-       gboolean multiline;
 
        column = gtk_tree_view_get_column(GTK_TREE_VIEW(listview), FILE_VIEW_COLUMN_THUMB);
        if (!column) return;
 
        gtk_tree_view_column_set_fixed_width(column, options->thumbnails.max_width + 4);
 
-       list = gtk_tree_view_column_get_cell_renderers(column);
+       list = gtk_cell_layout_get_cells(GTK_CELL_LAYOUT(column));
        if (!list) return;
        cell = list->data;
        g_list_free(list);
@@ -1660,17 +1619,9 @@ static void vflist_listview_set_columns(GtkWidget *listview, gint thumb)
        g_object_set(G_OBJECT(cell), "height", options->thumbnails.max_height, NULL);
        gtk_tree_view_column_set_visible(column, thumb);
 
-       multiline = (thumb && options->thumbnails.max_height >= 48);
-       
-       column = gtk_tree_view_get_column(GTK_TREE_VIEW(listview), FILE_VIEW_COLUMN_MULTILINE);
+       column = gtk_tree_view_get_column(GTK_TREE_VIEW(listview), FILE_VIEW_COLUMN_FORMATTED);
        if (!column) return;
-       gtk_tree_view_column_set_visible(column, multiline);
-       if (multiline) gtk_tree_view_set_expander_column(GTK_TREE_VIEW(listview), column);
-
-       column = gtk_tree_view_get_column(GTK_TREE_VIEW(listview), FILE_VIEW_COLUMN_NAME);
-       if (!column) return;
-       gtk_tree_view_column_set_visible(column, !multiline);
-       if (!multiline) gtk_tree_view_set_expander_column(GTK_TREE_VIEW(listview), column);
+       gtk_tree_view_set_expander_column(GTK_TREE_VIEW(listview), column);
 
        column = gtk_tree_view_get_column(GTK_TREE_VIEW(listview), FILE_VIEW_COLUMN_SIZE);
        if (!column) return;
@@ -1679,20 +1630,22 @@ static void vflist_listview_set_columns(GtkWidget *listview, gint thumb)
        column = gtk_tree_view_get_column(GTK_TREE_VIEW(listview), FILE_VIEW_COLUMN_DATE);
        if (!column) return;
        gtk_tree_view_column_set_visible(column, !multiline);
+}
 
-       gtk_tree_view_columns_autosize(GTK_TREE_VIEW(listview));
+static gboolean vflist_is_multiline(ViewFile *vf)
+{
+       return (VFLIST(vf)->thumbs_enabled && options->thumbnails.max_height >= 48);
 }
 
-static void vflist_populate_view(ViewFile *vf)
+
+static void vflist_populate_view(ViewFile *vf, gboolean force)
 {
        GtkTreeStore *store;
-       gint thumbs;
        GList *selected;
 
        store = GTK_TREE_STORE(gtk_tree_view_get_model(GTK_TREE_VIEW(vf->listview)));
-       thumbs = VFLIST(vf)->thumbs_enabled;
 
-       vflist_thumb_stop(vf);
+       vf_thumb_stop(vf);
 
        if (!vf->list)
                {
@@ -1701,11 +1654,11 @@ static void vflist_populate_view(ViewFile *vf)
                return;
                }
 
-       vflist_listview_set_columns(vf->listview, thumbs);
+       vflist_listview_set_columns(vf->listview, VFLIST(vf)->thumbs_enabled, vflist_is_multiline(vf));
 
        selected = vflist_selection_get_list(vf);
        
-       vflist_setup_iter_recursive(vf, store, NULL, vf->list, selected);
+       vflist_setup_iter_recursive(vf, store, NULL, vf->list, selected, force);
 
        if (selected && vflist_selection_count(vf, NULL) == 0)
                {
@@ -1716,13 +1669,13 @@ static void vflist_populate_view(ViewFile *vf)
        filelist_free(selected);
        
        vf_send_update(vf);
-       vflist_thumb_update(vf);
+       vf_thumb_update(vf);
 }
 
-gint vflist_refresh(ViewFile *vf)
+gboolean vflist_refresh(ViewFile *vf)
 {
        GList *old_list;
-       gint ret = TRUE;
+       gboolean ret = TRUE;
 
        old_list = vf->list;
        vf->list = NULL;
@@ -1742,7 +1695,7 @@ gint vflist_refresh(ViewFile *vf)
 
        DEBUG_1("%s vflist_refresh: populate view", get_exec_time());
 
-       vflist_populate_view(vf);
+       vflist_populate_view(vf, FALSE);
 
        filelist_free(old_list);
        DEBUG_1("%s vflist_refresh: done", get_exec_time());
@@ -1800,7 +1753,7 @@ static void vflist_listview_color_cb(GtkTreeViewColumn *tree_column, GtkCellRend
                     "cell-background-set", set, NULL);
 }
 
-static void vflist_listview_add_column(ViewFile *vf, gint n, const gchar *title, gint image, gint right_justify, gint expand)
+static void vflist_listview_add_column(ViewFile *vf, gint n, const gchar *title, gboolean image, gboolean right_justify, gboolean expand)
 {
        GtkTreeViewColumn *column;
        GtkCellRenderer *renderer;
@@ -1845,7 +1798,7 @@ static void vflist_listview_mark_toggled_cb(GtkCellRendererToggle *cell, gchar *
        GtkTreePath *path = gtk_tree_path_new_from_string(path_str);
        GtkTreeIter iter;
        FileData *fd;
-       gboolean mark;
+       gboolean marked;
        guint col_idx;
 
        store = GTK_TREE_STORE(gtk_tree_view_get_model(GTK_TREE_VIEW(vf->listview)));
@@ -1856,17 +1809,26 @@ static void vflist_listview_mark_toggled_cb(GtkCellRendererToggle *cell, gchar *
 
        g_assert(col_idx >= FILE_COLUMN_MARKS && col_idx <= FILE_COLUMN_MARKS_LAST);
 
-       gtk_tree_model_get(GTK_TREE_MODEL(store), &iter, FILE_COLUMN_POINTER, &fd, col_idx, &mark, -1);
-       mark = !mark;
-       file_data_unregister_notify_func(vf_notify_cb, vf); /* we don't need the notification */
-       file_data_set_mark(fd, col_idx - FILE_COLUMN_MARKS, mark);
+       gtk_tree_model_get(GTK_TREE_MODEL(store), &iter, FILE_COLUMN_POINTER, &fd, col_idx, &marked, -1);
+       marked = !marked;
+       
+       /* the change has a very limited range and the standard notification would trigger
+          complete re-read of the directory - try to do only minimal update instead */
+       file_data_unregister_notify_func(vf_notify_cb, vf);
+       file_data_set_mark(fd, col_idx - FILE_COLUMN_MARKS, marked);
        if (!file_data_filter_marks(fd, vf_marks_get_filter(vf))) /* file no longer matches the filter -> remove it */
                {
                vf_refresh_idle(vf);
                }
+       else
+               {
+               /* mark functions can have various side effects - update all columns to be sure */
+               vflist_setup_iter(vf, GTK_TREE_STORE(store), &iter, fd);
+               /* mark functions can change sidecars too */
+               vflist_setup_iter_recursive(vf, GTK_TREE_STORE(store), &iter, fd->sidecar_files, NULL, FALSE);
+               }
        file_data_register_notify_func(vf_notify_cb, vf, NOTIFY_PRIORITY_MEDIUM);
 
-       gtk_tree_store_set(store, &iter, col_idx, mark, -1);
        gtk_tree_path_free(path);
 }
 
@@ -1874,10 +1836,6 @@ static void vflist_listview_add_column_toggle(ViewFile *vf, gint n, const gchar
 {
        GtkTreeViewColumn *column;
        GtkCellRenderer *renderer;
-       GtkTreeStore *store;
-       gint index;
-
-       store = GTK_TREE_STORE(gtk_tree_view_get_model(GTK_TREE_VIEW(vf->listview)));
 
        renderer = gtk_cell_renderer_toggle_new();
        column = gtk_tree_view_column_new_with_attributes(title, renderer, "active", n, NULL);
@@ -1886,8 +1844,8 @@ static void vflist_listview_add_column_toggle(ViewFile *vf, gint n, const gchar
        g_object_set_data(G_OBJECT(column), "column_store_idx", GUINT_TO_POINTER(n));
        g_object_set_data(G_OBJECT(renderer), "column_store_idx", GUINT_TO_POINTER(n));
 
-       index = gtk_tree_view_append_column(GTK_TREE_VIEW(vf->listview), column);
-       gtk_tree_view_column_set_fixed_width(column, 18);
+       gtk_tree_view_append_column(GTK_TREE_VIEW(vf->listview), column);
+       gtk_tree_view_column_set_fixed_width(column, 22);
        gtk_tree_view_column_set_visible(column, vf->marks_enabled);
 
 
@@ -1900,8 +1858,9 @@ static void vflist_listview_add_column_toggle(ViewFile *vf, gint n, const gchar
  *-----------------------------------------------------------------------------
  */
 
-gint vflist_set_fd(ViewFile *vf, FileData *dir_fd)
+gboolean vflist_set_fd(ViewFile *vf, FileData *dir_fd)
 {
+       gboolean ret;
        if (!dir_fd) return FALSE;
        if (vf->dir_fd == dir_fd) return TRUE;
 
@@ -1914,7 +1873,9 @@ gint vflist_set_fd(ViewFile *vf, FileData *dir_fd)
        filelist_free(vf->list);
        vf->list = NULL;
 
-       return vf_refresh(vf);
+       ret = vf_refresh(vf);
+       gtk_tree_view_columns_autosize(GTK_TREE_VIEW(vf->listview));
+       return ret;
 }
 
 void vflist_destroy_cb(GtkWidget *widget, gpointer data)
@@ -1925,7 +1886,7 @@ void vflist_destroy_cb(GtkWidget *widget, gpointer data)
 
        vflist_select_idle_cancel(vf);
        vf_refresh_idle_cancel(vf);
-       vflist_thumb_stop(vf);
+       vf_thumb_stop(vf);
 
        filelist_free(vf->list);
 }
@@ -1934,26 +1895,21 @@ ViewFile *vflist_new(ViewFile *vf, FileData *dir_fd)
 {
        GtkTreeStore *store;
        GtkTreeSelection *selection;
-
        GType flist_types[FILE_COLUMN_COUNT];
        gint i;
        gint column;
 
        vf->info = g_new0(ViewFileInfoList, 1);
        
-       VFLIST(vf)->click_fd = NULL;
-       VFLIST(vf)->select_fd = NULL;
-       VFLIST(vf)->thumbs_enabled = FALSE;
-
-       VFLIST(vf)->select_idle_id = -1;
-
        flist_types[FILE_COLUMN_POINTER] = G_TYPE_POINTER;
        flist_types[FILE_COLUMN_VERSION] = G_TYPE_INT;
        flist_types[FILE_COLUMN_THUMB] = GDK_TYPE_PIXBUF;
+       flist_types[FILE_COLUMN_FORMATTED] = G_TYPE_STRING;
        flist_types[FILE_COLUMN_NAME] = G_TYPE_STRING;
-       flist_types[FILE_COLUMN_MULTILINE] = G_TYPE_STRING;
+       flist_types[FILE_COLUMN_SIDECARS] = G_TYPE_STRING;
        flist_types[FILE_COLUMN_SIZE] = G_TYPE_STRING;
        flist_types[FILE_COLUMN_DATE] = G_TYPE_STRING;
+       flist_types[FILE_COLUMN_EXPANDED] = G_TYPE_BOOLEAN;
        flist_types[FILE_COLUMN_COLOR] = G_TYPE_BOOLEAN;
        for (i = FILE_COLUMN_MARKS; i < FILE_COLUMN_MARKS + FILEDATA_MARKS_SIZE; i++)
                flist_types[i] = G_TYPE_BOOLEAN;
@@ -1963,6 +1919,12 @@ ViewFile *vflist_new(ViewFile *vf, FileData *dir_fd)
        vf->listview = gtk_tree_view_new_with_model(GTK_TREE_MODEL(store));
        g_object_unref(store);
 
+       g_signal_connect(G_OBJECT(vf->listview), "row-expanded",
+                        G_CALLBACK(vflist_expand_cb), vf);
+
+       g_signal_connect(G_OBJECT(vf->listview), "row-collapsed",
+                        G_CALLBACK(vflist_collapse_cb), vf);
+
        selection = gtk_tree_view_get_selection(GTK_TREE_VIEW(vf->listview));
        gtk_tree_selection_set_mode(GTK_TREE_SELECTION(selection), GTK_SELECTION_MULTIPLE);
        gtk_tree_selection_set_select_function(selection, vflist_select_cb, vf, NULL);
@@ -1983,12 +1945,8 @@ ViewFile *vflist_new(ViewFile *vf, FileData *dir_fd)
        g_assert(column == FILE_VIEW_COLUMN_THUMB);
        column++;
        
-       vflist_listview_add_column(vf, FILE_COLUMN_MULTILINE, _("Name"), FALSE, FALSE, TRUE);
-       g_assert(column == FILE_VIEW_COLUMN_MULTILINE);
-       column++;
-
-       vflist_listview_add_column(vf, FILE_COLUMN_NAME, _("Name"), FALSE, FALSE, TRUE);
-       g_assert(column == FILE_VIEW_COLUMN_NAME);
+       vflist_listview_add_column(vf, FILE_COLUMN_FORMATTED, _("Name"), FALSE, FALSE, TRUE);
+       g_assert(column == FILE_VIEW_COLUMN_FORMATTED);
        column++;
 
        vflist_listview_add_column(vf, FILE_COLUMN_SIZE, _("Size"), FALSE, TRUE, FALSE);
@@ -2003,15 +1961,24 @@ ViewFile *vflist_new(ViewFile *vf, FileData *dir_fd)
        return vf;
 }
 
-void vflist_thumb_set(ViewFile *vf, gint enable)
+void vflist_thumb_set(ViewFile *vf, gboolean enable)
 {
        if (VFLIST(vf)->thumbs_enabled == enable) return;
 
        VFLIST(vf)->thumbs_enabled = enable;
-       if (vf->layout) vf_refresh(vf);
+       
+       /* vflist_populate_view is better than vf_refresh:
+          - no need to re-read the directory
+          - force update because the formatted string has changed
+       */
+       if (vf->layout)
+               {
+               vflist_populate_view(vf, TRUE);
+               gtk_tree_view_columns_autosize(GTK_TREE_VIEW(vf->listview));
+               }
 }
 
-void vflist_marks_set(ViewFile *vf, gint enable)
+void vflist_marks_set(ViewFile *vf, gboolean enable)
 {
        GList *columns, *work;
 
@@ -2029,7 +1996,6 @@ void vflist_marks_set(ViewFile *vf, gint enable)
                }
 
        g_list_free(columns);
-       //vf_refresh(vf);
 }
 
 /* vim: set shiftwidth=8 softtabstop=0 cindent cinoptions={1s: */