Show shortcut keys in pop-up menus
[geeqie.git] / src / search.c
index 196c7bb..0aaf658 100644 (file)
 #define SEARCH_BUFFER_MATCH_MISS 1
 #define SEARCH_BUFFER_FLUSH_SIZE 99
 
+#define FORMAT_CLASS_BROKEN FILE_FORMAT_CLASSES + 1
+
 typedef enum {
        SEARCH_MATCH_NONE,
        SEARCH_MATCH_EQUAL,
        SEARCH_MATCH_CONTAINS,
+       SEARCH_MATCH_NAME_EQUAL,
+       SEARCH_MATCH_NAME_CONTAINS,
+       SEARCH_MATCH_PATH_CONTAINS,
        SEARCH_MATCH_UNDER,
        SEARCH_MATCH_OVER,
        SEARCH_MATCH_BETWEEN,
        SEARCH_MATCH_ALL,
-       SEARCH_MATCH_ANY
+       SEARCH_MATCH_ANY,
+       SEARCH_MATCH_COLLECTION
 } MatchType;
 
 enum {
@@ -96,8 +102,14 @@ struct _SearchData
        GtkWidget *label_progress;
        GtkWidget *button_start;
        GtkWidget *button_stop;
+       GtkWidget *button_close;
+       GtkWidget *button_help;
        GtkWidget *spinner;
 
+       GtkWidget *collection;
+       GtkWidget *fd_button;
+       GtkWidget *collection_entry;
+
        GtkWidget *box_search;
 
        GtkWidget *menu_path;
@@ -202,6 +214,7 @@ struct _SearchData
        gboolean match_rating_enable;
        gboolean match_class_enable;
        gboolean match_marks_enable;
+       gboolean match_broken_enable;
 
        GList *search_folder_list;
        GList *search_done_list;
@@ -256,12 +269,14 @@ struct _MatchList
 static const MatchList text_search_menu_path[] = {
        { N_("folder"),         SEARCH_MATCH_NONE },
        { N_("comments"),       SEARCH_MATCH_ALL },
-       { N_("results"),        SEARCH_MATCH_CONTAINS }
+       { N_("results"),        SEARCH_MATCH_CONTAINS },
+       { N_("collection"),     SEARCH_MATCH_COLLECTION }
 };
 
 static const MatchList text_search_menu_name[] = {
-       { N_("contains"),       SEARCH_MATCH_CONTAINS },
-       { N_("is"),             SEARCH_MATCH_EQUAL }
+       { N_("name contains"),  SEARCH_MATCH_NAME_CONTAINS },
+       { N_("name is"),        SEARCH_MATCH_NAME_EQUAL },
+       { N_("path contains"),  SEARCH_MATCH_PATH_CONTAINS }
 };
 
 static const MatchList text_search_menu_size[] = {
@@ -322,7 +337,38 @@ static gint search_result_count(SearchData *sd, gint64 *bytes);
 static void search_window_close(SearchData *sd);
 
 static void search_notify_cb(FileData *fd, NotifyType type, gpointer data);
+static void search_start_cb(GtkWidget *widget, gpointer data);
+void mfd_list_free(GList *list);
 
+
+/**
+ * This array must be kept in sync with the contents of:\n
+ * @link search_result_press_cb @endlink \n
+ * @link search_window_keypress_cb @endlink \n
+ * @link search_result_menu @endlink
+ * 
+ * See also @link hard_coded_window_keys @endlink
+ **/
+
+hard_coded_window_keys search_window_keys[] = {
+       {GDK_CONTROL_MASK, 'C', N_("Copy")},
+       {GDK_CONTROL_MASK, 'M', N_("Move")},
+       {GDK_CONTROL_MASK, 'R', N_("Rename")},
+       {GDK_CONTROL_MASK, 'D', N_("Move to Trash")},
+       {GDK_SHIFT_MASK, GDK_KEY_Delete, N_("Delete")},
+       {0, GDK_KEY_Delete, N_("Remove")},
+       {GDK_CONTROL_MASK, 'A', N_("Select all")},
+       {GDK_CONTROL_MASK + GDK_SHIFT_MASK, 'A', N_("Select none")},
+       {GDK_CONTROL_MASK, GDK_KEY_Delete, N_("Clear")},
+       {GDK_CONTROL_MASK, 'T', N_("Toggle thumbs")},
+       {GDK_CONTROL_MASK, 'W', N_("Close window")},
+       {0, GDK_KEY_Return, N_("View")},
+       {0, 'V', N_("View in new window")},
+       {0, 'C', N_("Collection from selection")},
+       {GDK_CONTROL_MASK, GDK_KEY_Return, N_("Start/stop search")},
+       {0, GDK_KEY_F3, N_("Find duplicates")},
+       {0, 0, NULL}
+};
 /*
  *-------------------------------------------------------------------
  * utils
@@ -430,6 +476,7 @@ static gint search_result_find_row(SearchData *sd, FileData *fd, GtkTreeIter *it
        return -1;
 }
 
+
 static gboolean search_result_row_selected(SearchData *sd, FileData *fd)
 {
        GtkTreeModel *store;
@@ -452,7 +499,7 @@ static gboolean search_result_row_selected(SearchData *sd, FileData *fd)
                if (mfd_n->fd == fd) found = TRUE;
                work = work->next;
                }
-       g_list_foreach(slist, (GFunc)gtk_tree_path_free, NULL);
+       g_list_foreach(slist, (GFunc)tree_path_free_wrapper, NULL);
        g_list_free(slist);
 
        return found;
@@ -491,7 +538,7 @@ static gint search_result_selection_util(SearchData *sd, gint64 *bytes, GList **
 
                work = work->next;
                }
-       g_list_foreach(slist, (GFunc)gtk_tree_path_free, NULL);
+       g_list_foreach(slist, (GFunc)tree_path_free_wrapper, NULL);
        g_list_free(slist);
 
        if (bytes) *bytes = total;
@@ -709,7 +756,7 @@ static void search_result_remove_selection(SearchData *sd)
                flist = g_list_prepend(flist, mfd->fd);
                work = work->next;
                }
-       g_list_foreach(slist, (GFunc)gtk_tree_path_free, NULL);
+       g_list_foreach(slist, (GFunc)tree_path_free_wrapper, NULL);
        g_list_free(slist);
 
        work = flist;
@@ -904,6 +951,8 @@ static void search_result_thumb_height(SearchData *sd)
 
 static void search_result_thumb_enable(SearchData *sd, gboolean enable)
 {
+       GtkTreeViewColumn *column;
+
        if (sd->thumb_enable == enable) return;
 
        if (sd->thumb_enable)
@@ -925,6 +974,12 @@ static void search_result_thumb_enable(SearchData *sd, gboolean enable)
                search_progress_update(sd, TRUE, -1.0);
                }
 
+       column = gtk_tree_view_get_column(GTK_TREE_VIEW(sd->result_view), SEARCH_COLUMN_THUMB - 1);
+       if (column)
+               {
+               gtk_tree_view_column_set_visible(column, enable);
+               }
+
        sd->thumb_enable = enable;
 
        search_result_thumb_height(sd);
@@ -1042,20 +1097,6 @@ static void sr_menu_copy_path_unquoted_cb(GtkWidget *widget, gpointer data)
        file_util_copy_path_list_to_clipboard(search_result_selection_list(sd), FALSE);
 }
 
-static void sr_menu_remove_cb(GtkWidget *widget, gpointer data)
-{
-       SearchData *sd = data;
-
-       search_result_remove_selection(sd);
-}
-
-static void sr_menu_clear_cb(GtkWidget *widget, gpointer data)
-{
-       SearchData *sd = data;
-
-       search_result_clear(sd);
-}
-
 static void sr_menu_play_cb(GtkWidget *widget, gpointer data)
 {
        SearchData *sd = data;
@@ -1094,10 +1135,15 @@ static GtkWidget *search_result_menu(SearchData *sd, gboolean on_row, gboolean e
        GtkWidget *menu;
        GtkWidget *item;
        GList *editmenu_fd_list;
-       GtkWidget *submenu;
        gboolean video;
+       GtkAccelGroup *accel_group;
 
        menu = popup_menu_short_lived();
+       accel_group = gtk_accel_group_new();
+       gtk_menu_set_accel_group(GTK_MENU(menu), accel_group);
+
+       g_object_set_data(G_OBJECT(menu), "window_keys", search_window_keys);
+       g_object_set_data(G_OBJECT(menu), "accel_group", accel_group);
 
        video = (on_row && sd->click_fd && sd->click_fd->format_class == FORMAT_CLASS_VIDEO);
        menu_item_add_stock_sensitive(menu, _("_Play"), GTK_STOCK_MEDIA_PLAY, video,
@@ -1305,6 +1351,7 @@ static gboolean search_result_release_cb(GtkWidget *widget, GdkEventButton *beve
        return FALSE;
 }
 
+
 static gboolean search_result_keypress_cb(GtkWidget *widget, GdkEventKey *event, gpointer data)
 {
        SearchData *sd = data;
@@ -1329,7 +1376,7 @@ static gboolean search_result_keypress_cb(GtkWidget *widget, GdkEventKey *event,
                gtk_tree_model_get_iter(store, &iter, tpath);
                gtk_tree_model_get(store, &iter, SEARCH_COLUMN_POINTER, &mfd, -1);
                }
-       g_list_foreach(slist, (GFunc)gtk_tree_path_free, NULL);
+       g_list_foreach(slist, (GFunc)tree_path_free_wrapper, NULL);
        g_list_free(slist);
 
        if (event->state & GDK_CONTROL_MASK)
@@ -1443,6 +1490,9 @@ static gboolean search_window_keypress_cb(GtkWidget *widget, GdkEventKey *event,
                        case 'W': case 'w':
                                search_window_close(sd);
                                break;
+                       case GDK_KEY_Return: case GDK_KEY_KP_Enter:
+                               search_start_cb(NULL, sd);
+                               break;
                        default:
                                stop_signal = FALSE;
                                break;
@@ -1709,6 +1759,8 @@ static void search_stop(SearchData *sd)
        filelist_free(sd->search_file_list);
        sd->search_file_list = NULL;
 
+       sd->match_broken_enable = FALSE;
+
        gtk_widget_set_sensitive(sd->box_search, TRUE);
        spinner_set_interval(sd->spinner, -1);
        gtk_widget_set_sensitive(sd->button_start, TRUE);
@@ -1723,7 +1775,16 @@ static void search_file_load_process(SearchData *sd, CacheData *cd)
 
        pixbuf = image_loader_get_pixbuf(sd->img_loader);
 
-       if (cd && pixbuf)
+       /* Used to determine if image is broken
+        */
+       if (cd && !pixbuf)
+               {
+               if (!cd->dimensions)
+                       {
+                       cache_sim_data_set_dimensions(cd, -1, -1);
+                       }
+               }
+       else if (cd && pixbuf)
                {
                if (!cd->dimensions)
                        {
@@ -1802,8 +1863,7 @@ static gboolean search_file_do_extra(SearchData *sd, FileData *fd, gint *match,
 
        if (new_data)
                {
-               if ((sd->match_dimensions_enable && !sd->img_cd->dimensions) ||
-                   (sd->match_similarity_enable && !sd->img_cd->similarity))
+               if ((sd->match_dimensions_enable && !sd->img_cd->dimensions) || (sd->match_similarity_enable && !sd->img_cd->similarity) || sd->match_broken_enable)
                        {
                        sd->img_loader = image_loader_new(fd);
                        g_signal_connect(G_OBJECT(sd->img_loader), "error", (GCallback)search_file_load_done_cb, sd);
@@ -1820,6 +1880,20 @@ static gboolean search_file_do_extra(SearchData *sd, FileData *fd, gint *match,
                        }
                }
 
+       if (sd->match_broken_enable)
+               {
+               tested = TRUE;
+               tmatch = FALSE;
+               if (sd->match_class == SEARCH_MATCH_EQUAL && sd->img_cd->width == -1)
+                       {
+                       tmatch = TRUE;
+                       }
+               else if (sd->match_class == SEARCH_MATCH_NONE && sd->img_cd->width != -1)
+                       {
+                       tmatch = TRUE;
+                       }
+               }
+
        if (tmatch && sd->match_dimensions_enable && sd->img_cd->dimensions)
                {
                CacheData *cd = sd->img_cd;
@@ -1853,7 +1927,7 @@ static gboolean search_file_do_extra(SearchData *sd, FileData *fd, gint *match,
                tmatch = FALSE;
                tested = TRUE;
 
-               /* fixme: implement similarity checking */
+               /** @FIXME implement similarity checking */
                if (sd->search_similarity_cd && sd->search_similarity_cd->similarity)
                        {
                        gdouble result;
@@ -1916,7 +1990,7 @@ static gboolean search_file_next(SearchData *sd)
                tested = TRUE;
                match = FALSE;
 
-               if (sd->match_name == SEARCH_MATCH_EQUAL)
+               if (sd->match_name == SEARCH_MATCH_NAME_EQUAL)
                        {
                        if (sd->search_name_match_case)
                                {
@@ -1927,16 +2001,25 @@ static gboolean search_file_next(SearchData *sd)
                                match = (g_ascii_strcasecmp(fd->name, sd->search_name) == 0);
                                }
                        }
-               else if (sd->match_name == SEARCH_MATCH_CONTAINS)
+               else if (sd->match_name == SEARCH_MATCH_NAME_CONTAINS || sd->match_name == SEARCH_MATCH_PATH_CONTAINS)
                        {
+                       const gchar *fd_name_or_path;
+                       if (sd->match_name == SEARCH_MATCH_NAME_CONTAINS)
+                               {
+                               fd_name_or_path = fd->name;
+                               }
+                       else
+                               {
+                               fd_name_or_path = fd->path;
+                               }
                        if (sd->search_name_match_case)
                                {
-                               match = g_regex_match(sd->search_name_regex, fd->name, 0, NULL);
+                               match = g_regex_match(sd->search_name_regex, fd_name_or_path, 0, NULL);
                                }
                        else
                                {
                                /* sd->search_name is converted in search_start() */
-                               gchar *haystack = g_utf8_strdown(fd->name, -1);
+                               gchar *haystack = g_utf8_strdown(fd_name_or_path, -1);
                                match = g_regex_match(sd->search_name_regex, haystack, 0, NULL);
                                g_free(haystack);
                                }
@@ -2171,7 +2254,7 @@ static gboolean search_file_next(SearchData *sd)
                {
                tested = TRUE;
                match = FALSE;
-               gint class;
+               FileFormatClass class;
                FileFormatClass search_class;
 
                if (g_strcmp0(gtk_combo_box_text_get_active_text(
@@ -2189,24 +2272,49 @@ static gboolean search_file_next(SearchData *sd)
                        {
                        search_class = FORMAT_CLASS_VIDEO;
                        }
+               else if (g_strcmp0(gtk_combo_box_text_get_active_text(
+                                               GTK_COMBO_BOX_TEXT(sd->class_type)), _("Document")) == 0)
+                       {
+                       search_class = FORMAT_CLASS_DOCUMENT;
+                       }
                else if (g_strcmp0(gtk_combo_box_text_get_active_text(
                                                GTK_COMBO_BOX_TEXT(sd->class_type)), _("Metadata")) == 0)
                        {
                        search_class = FORMAT_CLASS_META;
                        }
-               else
+               else if (g_strcmp0(gtk_combo_box_text_get_active_text(
+                                               GTK_COMBO_BOX_TEXT(sd->class_type)), _("Unknown")) == 0)
                        {
                        search_class = FORMAT_CLASS_UNKNOWN;
                        }
+               else
+                       {
+                       search_class = FORMAT_CLASS_BROKEN;
+                       }
 
-               class = fd->format_class;
-               if (sd->match_class == SEARCH_MATCH_EQUAL)
+               if (search_class != FORMAT_CLASS_BROKEN)
                        {
-                       match = (class == search_class);
+                       class = fd->format_class;
+                       if (sd->match_class == SEARCH_MATCH_EQUAL)
+                               {
+                               match = (class == search_class);
+                               }
+                       else if (sd->match_class == SEARCH_MATCH_NONE)
+                               {
+                               match = (class != search_class);
+                               }
                        }
-               else if (sd->match_class == SEARCH_MATCH_NONE)
+               else
                        {
-                       match = (class != search_class);
+                       if (fd->format_class == FORMAT_CLASS_IMAGE || fd->format_class == FORMAT_CLASS_RAWIMAGE || fd->format_class == FORMAT_CLASS_VIDEO || fd->format_class == FORMAT_CLASS_DOCUMENT)
+                               {
+                               sd->match_broken_enable = TRUE;
+                               match = TRUE;
+                               }
+                       else
+                               {
+                               sd->match_broken_enable = FALSE;
+                               }
                        }
                }
 
@@ -2317,8 +2425,7 @@ static gboolean search_file_next(SearchData *sd)
                        }
                }
 
-       if ((match || extra_only) && (sd->match_dimensions_enable ||
-                                                               sd ->match_similarity_enable))
+       if ((match || extra_only) && (sd->match_dimensions_enable || sd->match_similarity_enable || sd->match_broken_enable))
                {
                tested = TRUE;
 
@@ -2580,6 +2687,7 @@ static void search_start_cb(GtkWidget *widget, gpointer data)
        GtkTreeViewColumn *column;
        gchar *path;
        gchar *entry_text;
+       gchar *collection;
 
        if (sd->search_folder_list)
                {
@@ -2705,6 +2813,29 @@ static void search_start_cb(GtkWidget *widget, gpointer data)
 
                sd->search_file_list = g_list_concat(sd->search_file_list, list);
                }
+       else if (sd->search_type == SEARCH_MATCH_COLLECTION)
+               {
+               collection = g_strdup(gtk_entry_get_text(GTK_ENTRY(sd->collection_entry)));
+
+               if (is_collection(collection))
+                       {
+                       GList *list = NULL;
+
+                       list = collection_contents_fd(collection);
+
+                       file_data_unref(sd->search_dir_fd);
+                       sd->search_dir_fd = NULL;
+
+                       search_start(sd);
+
+                       sd->search_file_list = g_list_concat(sd->search_file_list, list);
+                       }
+               else
+                       {
+                       file_util_warning_dialog(_("Collection not found"), _("Please enter an existing collection name."), GTK_STOCK_DIALOG_WARNING, sd->window);
+                       }
+               g_free(collection);
+               }
 }
 
 /*
@@ -2849,6 +2980,7 @@ static void menu_choice_path_cb(GtkWidget *combo, gpointer data)
 
        menu_choice_set_visible(gtk_widget_get_parent(sd->check_recurse),
                                (sd->search_type == SEARCH_MATCH_NONE));
+       menu_choice_set_visible(sd->collection, (sd->search_type == SEARCH_MATCH_COLLECTION));
 }
 
 static void menu_choice_name_cb(GtkWidget *combo, gpointer data)
@@ -3048,11 +3180,41 @@ static GtkWidget *menu_choice(GtkWidget *box, GtkWidget **check, GtkWidget **men
        return hbox;
 }
 
+static void search_window_get_geometry(SearchData *sd)
+{
+       GdkWindow *window;
+       LayoutWindow *lw = NULL;
+
+       layout_valid(&lw);
+
+       if (!sd || !lw) return;
+
+       window = gtk_widget_get_window(sd->window);
+       gdk_window_get_position(window, &lw->options.search_window.x, &lw->options.search_window.y);
+       lw->options.search_window.w = gdk_window_get_width(window);
+       lw->options.search_window.h = gdk_window_get_height(window);
+}
+
 static void search_window_close(SearchData *sd)
 {
+
+       search_window_get_geometry(sd);
+
        gtk_widget_destroy(sd->window);
 }
 
+static void search_window_close_cb(GtkWidget *widget, gpointer data)
+{
+       SearchData *sd = data;
+
+       search_window_close(sd);
+}
+
+static void search_window_help_cb(GtkWidget *widget, gpointer data)
+{
+       help_window_show("GuideImageSearchSearch.html");
+}
+
 static gboolean search_window_delete_cb(GtkWidget *widget, GdkEventAny *event, gpointer data)
 {
        SearchData *sd = data;
@@ -3069,7 +3231,7 @@ static void search_window_destroy_cb(GtkWidget *widget, gpointer data)
 
        search_result_update_idle_cancel(sd);
 
-       filelist_free(sd->search_buffer_list);
+       mfd_list_free(sd->search_buffer_list);
        sd->search_buffer_list = NULL;
 
        search_stop(sd);
@@ -3095,6 +3257,54 @@ static void search_window_destroy_cb(GtkWidget *widget, gpointer data)
        g_free(sd);
 }
 
+static void select_collection_dialog_close_cb(FileDialog *fdlg, gpointer data)
+{
+       file_dialog_close(fdlg);
+}
+
+static void select_collection_dialog_ok_cb(FileDialog *fdlg, gpointer data)
+{
+       SearchData *sd = data;
+       gchar *path;
+       gchar *path_noext;
+       gchar *collection;
+
+       path = g_strdup(gtk_entry_get_text(GTK_ENTRY(fdlg->entry)));
+       path_noext = remove_extension_from_path(path);
+       collection = g_path_get_basename(path_noext);
+
+       gtk_entry_set_text(GTK_ENTRY(sd->collection_entry), collection);
+       file_dialog_close(fdlg);
+
+       g_free(path);
+       g_free(path_noext);
+       g_free(collection);
+}
+
+static void select_collection_clicked_cb(GtkWidget *widget, gpointer data)
+{
+       SearchData *sd = data;
+       FileDialog *fdlg;
+       const gchar *title;
+       const gchar *btntext;
+       gpointer btnfunc;
+       const gchar *stock_id;
+
+       title = _("Select collection");
+       btntext = NULL;
+       btnfunc = select_collection_dialog_ok_cb;
+       stock_id = GTK_STOCK_OK;
+
+       fdlg = file_util_file_dlg(title, "dlg_collection", sd->window, select_collection_dialog_close_cb, sd);
+
+       generic_dialog_add_message(GENERIC_DIALOG(fdlg), NULL, title, NULL, FALSE);
+       file_dialog_add_button(fdlg, stock_id, btntext, btnfunc, TRUE);
+
+       file_dialog_add_path_widgets(fdlg, get_collections_dir(), NULL, "search_collection", GQ_COLLECTION_EXT, _("Collection Files"));
+
+       gtk_widget_show(GENERIC_DIALOG(fdlg)->dialog);
+}
+
 void search_new(FileData *dir_fd, FileData *example_file)
 {
        SearchData *sd;
@@ -3107,10 +3317,14 @@ void search_new(FileData *dir_fd, FileData *example_file)
        GtkListStore *store;
        GtkTreeSortable *sortable;
        GtkTreeSelection *selection;
+       GtkTreeViewColumn *column;
        GtkWidget *combo;
        GdkGeometry geometry;
        gint i;
        gchar *marks_string;
+       LayoutWindow *lw = NULL;
+
+       layout_valid(&lw);
 
        sd = g_new0(SearchData, 1);
 
@@ -3124,7 +3338,7 @@ void search_new(FileData *dir_fd, FileData *example_file)
 
        sd->search_type = SEARCH_MATCH_NONE;
 
-       sd->match_name = SEARCH_MATCH_CONTAINS;
+       sd->match_name = SEARCH_MATCH_NAME_CONTAINS;
        sd->match_size = SEARCH_MATCH_EQUAL;
        sd->match_date = SEARCH_MATCH_EQUAL;
        sd->match_dimensions = SEARCH_MATCH_EQUAL;
@@ -3147,6 +3361,7 @@ void search_new(FileData *dir_fd, FileData *example_file)
                }
 
        sd->window = window_new(GTK_WINDOW_TOPLEVEL, "search", NULL, NULL, _("Image search"));
+       DEBUG_NAME(sd->window);
 
        gtk_window_set_resizable(GTK_WINDOW(sd->window), TRUE);
 
@@ -3157,7 +3372,15 @@ void search_new(FileData *dir_fd, FileData *example_file)
        gtk_window_set_geometry_hints(GTK_WINDOW(sd->window), NULL, &geometry,
                                      GDK_HINT_MIN_SIZE | GDK_HINT_BASE_SIZE);
 
-       gtk_window_set_default_size(GTK_WINDOW(sd->window), DEF_SEARCH_WIDTH, DEF_SEARCH_HEIGHT);
+       if (lw && options->save_window_positions)
+               {
+               gtk_window_set_default_size(GTK_WINDOW(sd->window), lw->options.search_window.w, lw->options.search_window.h);
+               gtk_window_move(GTK_WINDOW(sd->window), lw->options.search_window.x, lw->options.search_window.y);
+               }
+       else
+               {
+               gtk_window_set_default_size(GTK_WINDOW(sd->window), DEF_SEARCH_WIDTH, DEF_SEARCH_HEIGHT);
+               }
 
        g_signal_connect(G_OBJECT(sd->window), "delete_event",
                         G_CALLBACK(search_window_delete_cb), sd);
@@ -3193,9 +3416,22 @@ void search_new(FileData *dir_fd, FileData *example_file)
        sd->check_recurse = pref_checkbox_new_int(hbox2, _("Recurse"),
                                                  sd->search_path_recurse, &sd->search_path_recurse);
 
+       sd->collection = pref_box_new(hbox, TRUE, GTK_ORIENTATION_HORIZONTAL, PREF_PAD_SPACE);
+       sd->collection_entry = gtk_entry_new();
+       gtk_entry_set_text(GTK_ENTRY(sd->collection_entry), "");
+       gtk_box_pack_start(GTK_BOX(sd->collection), sd->collection_entry, TRUE, TRUE, 0);
+       gtk_widget_show(sd->collection_entry);
+
+       sd->fd_button = gtk_button_new_with_label("...");
+       g_signal_connect(G_OBJECT(sd->fd_button), "clicked", G_CALLBACK(select_collection_clicked_cb), sd);
+       gtk_box_pack_start(GTK_BOX(sd->collection), sd->fd_button, FALSE, FALSE, 0);
+       gtk_widget_show(sd->fd_button);
+
+       gtk_widget_hide(sd->collection);
+
        /* Search for file name */
        hbox = menu_choice(sd->box_search, &sd->check_name, &sd->menu_name,
-                          _("File name"), &sd->match_name_enable,
+                          _("File"), &sd->match_name_enable,
                           text_search_menu_name, sizeof(text_search_menu_name) / sizeof(MatchList),
                           G_CALLBACK(menu_choice_name_cb), sd);
        combo = history_combo_new(&sd->entry_name, "", "search_name", -1);
@@ -3203,7 +3439,7 @@ void search_new(FileData *dir_fd, FileData *example_file)
        gtk_widget_show(combo);
        pref_checkbox_new_int(hbox, _("Match case"),
                              sd->search_name_match_case, &sd->search_name_match_case);
-       gtk_widget_set_tooltip_text(GTK_WIDGET(combo), "When set to \"contains\", this field uses Perl Compatible Regular Expressions.\ne.g. use \n.*\\.jpg\n and not \n*.jpg\n\nSee the Help file.");
+       gtk_widget_set_tooltip_text(GTK_WIDGET(combo), "When set to \"contains\" or \"path contains\", this field uses Perl Compatible Regular Expressions.\ne.g. use \n.*\\.jpg\n and not \n*.jpg\n\nSee the Help file.");
 
        /* Search for file size */
        hbox = menu_choice(sd->box_search, &sd->check_size, &sd->menu_size,
@@ -3369,8 +3605,10 @@ void search_new(FileData *dir_fd, FileData *example_file)
        gtk_combo_box_text_append_text(GTK_COMBO_BOX_TEXT(sd->class_type), _("Image"));
        gtk_combo_box_text_append_text(GTK_COMBO_BOX_TEXT(sd->class_type), _("Raw Image"));
        gtk_combo_box_text_append_text(GTK_COMBO_BOX_TEXT(sd->class_type), _("Video"));
+       gtk_combo_box_text_append_text(GTK_COMBO_BOX_TEXT(sd->class_type), _("Document"));
        gtk_combo_box_text_append_text(GTK_COMBO_BOX_TEXT(sd->class_type), _("Metadata"));
        gtk_combo_box_text_append_text(GTK_COMBO_BOX_TEXT(sd->class_type), _("Unknown"));
+       gtk_combo_box_text_append_text(GTK_COMBO_BOX_TEXT(sd->class_type), _("Broken"));
        gtk_box_pack_start(GTK_BOX(hbox), sd->class_type, FALSE, FALSE, 0);
        gtk_combo_box_set_active(GTK_COMBO_BOX(sd->class_type), 0);
        gtk_widget_show(sd->class_type);
@@ -3451,7 +3689,7 @@ void search_new(FileData *dir_fd, FileData *example_file)
        gtk_tree_view_set_enable_search(GTK_TREE_VIEW(sd->result_view), FALSE);
 
        search_result_add_column(sd, SEARCH_COLUMN_RANK, _("Rank"), FALSE, FALSE);
-       search_result_add_column(sd, SEARCH_COLUMN_THUMB, "", TRUE, FALSE);
+       search_result_add_column(sd, SEARCH_COLUMN_THUMB, _("Thumb"), TRUE, FALSE);
        search_result_add_column(sd, SEARCH_COLUMN_NAME, _("Name"), FALSE, FALSE);
        search_result_add_column(sd, SEARCH_COLUMN_SIZE, _("Size"), FALSE, TRUE);
        search_result_add_column(sd, SEARCH_COLUMN_DATE, _("Date"), FALSE, TRUE);
@@ -3471,8 +3709,10 @@ void search_new(FileData *dir_fd, FileData *example_file)
 
        sd->button_thumbs = pref_checkbox_new(hbox, _("Thumbnails"), FALSE,
                                              G_CALLBACK(search_thumb_toggle_cb), sd);
+       gtk_widget_set_tooltip_text(GTK_WIDGET(sd->button_thumbs), "Ctrl-T");
 
        frame = gtk_frame_new(NULL);
+       DEBUG_NAME(frame);
        gtk_frame_set_shadow_type(GTK_FRAME(frame), GTK_SHADOW_IN);
        gtk_box_pack_start(GTK_BOX(hbox), frame, TRUE, TRUE, PREF_PAD_SPACE);
        gtk_widget_show(frame);
@@ -3495,12 +3735,27 @@ void search_new(FileData *dir_fd, FileData *example_file)
        gtk_box_pack_start(GTK_BOX(hbox), sd->spinner, FALSE, FALSE, 0);
        gtk_widget_show(sd->spinner);
 
+       sd->button_help = pref_button_new(hbox, GTK_STOCK_HELP, NULL, FALSE, G_CALLBACK(search_window_help_cb), sd);
+       gtk_widget_set_tooltip_text(GTK_WIDGET(sd->button_help), "F1");
+       gtk_widget_set_sensitive(sd->button_help, TRUE);
+       pref_spacer(hbox, PREF_PAD_BUTTON_GAP);
        sd->button_start = pref_button_new(hbox, GTK_STOCK_FIND, NULL, FALSE,
                                           G_CALLBACK(search_start_cb), sd);
+       gtk_widget_set_tooltip_text(GTK_WIDGET(sd->button_start), "Ctrl-Return");
        pref_spacer(hbox, PREF_PAD_BUTTON_GAP);
        sd->button_stop = pref_button_new(hbox, GTK_STOCK_STOP, NULL, FALSE,
                                          G_CALLBACK(search_start_cb), sd);
+       gtk_widget_set_tooltip_text(GTK_WIDGET(sd->button_stop), "Ctrl-Return");
        gtk_widget_set_sensitive(sd->button_stop, FALSE);
+       pref_spacer(hbox, PREF_PAD_BUTTON_GAP);
+       sd->button_close = pref_button_new(hbox, GTK_STOCK_CLOSE, NULL, FALSE, G_CALLBACK(search_window_close_cb), sd);
+       gtk_widget_set_tooltip_text(GTK_WIDGET(sd->button_close), "Ctrl-W");
+       gtk_widget_set_sensitive(sd->button_close, TRUE);
+
+       search_result_thumb_enable(sd, TRUE);
+       search_result_thumb_enable(sd, FALSE);
+       column = gtk_tree_view_get_column(GTK_TREE_VIEW(sd->result_view), SEARCH_COLUMN_RANK - 1);
+       gtk_tree_view_column_set_visible(column, FALSE);
 
        search_status_update(sd);
        search_progress_update(sd, FALSE, -1.0);
@@ -3572,4 +3827,20 @@ static void search_notify_cb(FileData *fd, NotifyType type, gpointer data)
                        break;
                }
 }
+
+void mfd_list_free(GList *list)
+{
+       GList *work;
+
+       work = list;
+       while (work)
+               {
+               MatchFileData *mfd = work->data;
+               file_data_unref((FileData *)mfd->fd);
+               work = work->next;
+               }
+
+       g_list_free(list);
+}
+
 /* vim: set shiftwidth=8 softtabstop=0 cindent cinoptions={1s: */