Trim trailing white spaces.
[geeqie.git] / src / img-view.c
index 861cbe0..06368d4 100644 (file)
@@ -1,7 +1,7 @@
 /*
  * Geeqie
  * (C) 2006 John Ellis
- * Copyright (C) 2008 The Geeqie Team
+ * Copyright (C) 2008 - 2012 The Geeqie Team
  *
  * Author: John Ellis
  *
 #include "fullscreen.h"
 #include "image.h"
 #include "image-overlay.h"
-#include "info.h"
 #include "layout.h"
 #include "layout_image.h"
 #include "menu.h"
-#include "pixbuf-renderer.h"
+#include "misc.h"
 #include "pixbuf_util.h"
+#include "pixbuf-renderer.h"
 #include "print.h"
 #include "slideshow.h"
-#include "utilops.h"
-#include "ui_bookmark.h"
 #include "ui_fileops.h"
 #include "ui_menu.h"
+#include "uri_utils.h"
+#include "utilops.h"
 #include "window.h"
 
 #include <gdk/gdkkeysyms.h> /* for keyboard values */
@@ -56,7 +56,7 @@ static GList *view_window_list = NULL;
 
 
 static GtkWidget *view_popup_menu(ViewWindow *vw);
-static void view_fullscreen_toggle(ViewWindow *vw, gint force_off);
+static void view_fullscreen_toggle(ViewWindow *vw, gboolean force_off);
 static void view_overlay_toggle(ViewWindow *vw);
 
 static void view_slideshow_next(ViewWindow *vw);
@@ -68,6 +68,7 @@ static void view_window_close(ViewWindow *vw);
 
 static void view_window_dnd_init(ViewWindow *vw);
 
+static void view_window_notify_cb(FileData *fd, NotifyType type, gpointer data);
 
 /*
  *-----------------------------------------------------------------------------
@@ -92,7 +93,7 @@ static void view_window_set_list(ViewWindow *vw, GList *list)
        vw->list = filelist_copy(list);
 }
 
-static gint view_window_contains_collection(ViewWindow *vw)
+static gboolean view_window_contains_collection(ViewWindow *vw)
 {
        CollectionData *cd;
        CollectInfo *info;
@@ -102,7 +103,7 @@ static gint view_window_contains_collection(ViewWindow *vw)
        return (cd && info);
 }
 
-static void view_collection_step(ViewWindow *vw, gint next)
+static void view_collection_step(ViewWindow *vw, gboolean next)
 {
        ImageWindow *imd = view_window_active_image(vw);
        CollectionData *cd;
@@ -134,14 +135,14 @@ static void view_collection_step(ViewWindow *vw, gint next)
 
        if (info)
                {
-               image_change_from_collection(imd, cd, info, image_zoom_get_default(imd, options->image.zoom_mode));
+               image_change_from_collection(imd, cd, info, image_zoom_get_default(imd));
 
                if (read_ahead_info) image_prebuffer_set(imd, read_ahead_info->fd);
                }
 
 }
 
-static void view_collection_step_to_end(ViewWindow *vw, gint last)
+static void view_collection_step_to_end(ViewWindow *vw, gboolean last)
 {
        ImageWindow *imd = view_window_active_image(vw);
        CollectionData *cd;
@@ -165,12 +166,12 @@ static void view_collection_step_to_end(ViewWindow *vw, gint last)
 
        if (info)
                {
-               image_change_from_collection(imd, cd, info, image_zoom_get_default(imd, options->image.zoom_mode));
+               image_change_from_collection(imd, cd, info, image_zoom_get_default(imd));
                if (read_ahead_info) image_prebuffer_set(imd, read_ahead_info->fd);
                }
 }
 
-static void view_list_step(ViewWindow *vw, gint next)
+static void view_list_step(ViewWindow *vw, gboolean next)
 {
        ImageWindow *imd = view_window_active_image(vw);
        FileData *fd;
@@ -188,7 +189,7 @@ static void view_list_step(ViewWindow *vw, gint next)
                }
        else
                {
-               gint found = FALSE;
+               gboolean found = FALSE;
 
                work = vw->list;
                while (work && !found)
@@ -225,7 +226,7 @@ static void view_list_step(ViewWindow *vw, gint next)
 
        vw->list_pointer = work;
        fd = work->data;
-       image_change_fd(imd, fd, image_zoom_get_default(imd, options->image.zoom_mode));
+       image_change_fd(imd, fd, image_zoom_get_default(imd));
 
        if (options->image.enable_read_ahead && work_ahead)
                {
@@ -234,7 +235,7 @@ static void view_list_step(ViewWindow *vw, gint next)
                }
 }
 
-static void view_list_step_to_end(ViewWindow *vw, gint last)
+static void view_list_step_to_end(ViewWindow *vw, gboolean last)
 {
        ImageWindow *imd = view_window_active_image(vw);
        FileData *fd;
@@ -256,7 +257,7 @@ static void view_list_step_to_end(ViewWindow *vw, gint last)
 
        vw->list_pointer = work;
        fd = work->data;
-       image_change_fd(imd, fd, image_zoom_get_default(imd, options->image.zoom_mode));
+       image_change_fd(imd, fd, image_zoom_get_default(imd));
 
        if (options->image.enable_read_ahead && work_ahead)
                {
@@ -297,7 +298,7 @@ static void view_step_prev(ViewWindow *vw)
                }
 }
 
-static void view_step_to_end(ViewWindow *vw, gint last)
+static void view_step_to_end(ViewWindow *vw, gboolean last)
 {
        if (vw->list)
                {
@@ -321,11 +322,11 @@ static void view_window_menu_pos_cb(GtkMenu *menu, gint *x, gint *y, gboolean *p
        ImageWindow *imd;
 
        imd = view_window_active_image(vw);
-       gdk_window_get_origin(imd->pr->window, x, y);
+       gdk_window_get_origin(gtk_widget_get_window(imd->pr), x, y);
        popup_menu_position_clamp(menu, x, y, 0);
 }
 
-static gint view_window_key_press_cb(GtkWidget *widget, GdkEventKey *event, gpointer data)
+static gboolean view_window_key_press_cb(GtkWidget *widget, GdkEventKey *event, gpointer data)
 {
        ViewWindow *vw = data;
        ImageWindow *imd;
@@ -339,16 +340,16 @@ static gint view_window_key_press_cb(GtkWidget *widget, GdkEventKey *event, gpoi
        stop_signal = TRUE;
        switch (event->keyval)
                {
-               case GDK_Left: case GDK_KP_Left:
+               case GDK_KEY_Left: case GDK_KEY_KP_Left:
                        x -= 1;
                        break;
-               case GDK_Right: case GDK_KP_Right:
+               case GDK_KEY_Right: case GDK_KEY_KP_Right:
                        x += 1;
                        break;
-               case GDK_Up: case GDK_KP_Up:
+               case GDK_KEY_Up: case GDK_KEY_KP_Up:
                        y -= 1;
                        break;
-               case GDK_Down: case GDK_KP_Down:
+               case GDK_KEY_Down: case GDK_KEY_KP_Down:
                        y += 1;
                        break;
                default:
@@ -372,40 +373,19 @@ static gint view_window_key_press_cb(GtkWidget *widget, GdkEventKey *event, gpoi
 
        if (event->state & GDK_CONTROL_MASK)
                {
-               gint n = -1;
-
                stop_signal = TRUE;
                switch (event->keyval)
                        {
                        case '1':
-                               n = 0;
-                               break;
                        case '2':
-                               n = 1;
-                               break;
                        case '3':
-                               n = 2;
-                               break;
                        case '4':
-                               n = 3;
-                               break;
                        case '5':
-                               n = 4;
-                               break;
                        case '6':
-                               n = 5;
-                               break;
                        case '7':
-                               n = 6;
-                               break;
                        case '8':
-                               n = 7;
-                               break;
                        case '9':
-                               n = 8;
-                               break;
                        case '0':
-                               n = 9;
                                break;
                        case 'C': case 'c':
                                file_util_copy(image_get_fd(imd), NULL, NULL, imd->widget);
@@ -419,9 +399,6 @@ static gint view_window_key_press_cb(GtkWidget *widget, GdkEventKey *event, gpoi
                        case 'D': case 'd':
                                file_util_delete(image_get_fd(imd), NULL, imd->widget);
                                break;
-                       case 'P': case 'p':
-                               info_window_new(image_get_fd(imd), NULL, vw->fs ? vw->fs->window : NULL);
-                               break;
                        case 'W': case 'w':
                                view_window_close(vw);
                                break;
@@ -429,15 +406,6 @@ static gint view_window_key_press_cb(GtkWidget *widget, GdkEventKey *event, gpoi
                                stop_signal = FALSE;
                                break;
                        }
-               if (n != -1)
-                       {
-                       if (!editor_window_flag_set(n))
-                               {
-                               view_fullscreen_toggle(vw, TRUE);
-                               }
-                       imd = view_window_active_image(vw);
-                       file_util_start_editor_from_file(n, image_get_fd(imd), imd->widget);
-                       }
                }
        else if (event->state & GDK_SHIFT_MASK)
                {
@@ -445,16 +413,16 @@ static gint view_window_key_press_cb(GtkWidget *widget, GdkEventKey *event, gpoi
                switch (event->keyval)
                        {
                        case 'R': case 'r':
-                               image_alter(imd, ALTER_ROTATE_180);
+                               image_alter_orientation(imd, ALTER_ROTATE_180);
                                break;
                        case 'M': case 'm':
-                               image_alter(imd, ALTER_MIRROR);
+                               image_alter_orientation(imd, ALTER_MIRROR);
                                break;
                        case 'F': case 'f':
-                               image_alter(imd, ALTER_FLIP);
+                               image_alter_orientation(imd, ALTER_FLIP);
                                break;
                        case 'G': case 'g':
-                               image_alter(imd, ALTER_DESATURATE);
+                               image_set_desaturate(imd, !image_get_desaturate(imd));
                                break;
                        case 'P': case 'p':
                                {
@@ -478,32 +446,32 @@ static gint view_window_key_press_cb(GtkWidget *widget, GdkEventKey *event, gpoi
                stop_signal = TRUE;
                switch (event->keyval)
                        {
-                       case GDK_Page_Up: case GDK_KP_Page_Up:
-                       case GDK_BackSpace:
+                       case GDK_KEY_Page_Up: case GDK_KEY_KP_Page_Up:
+                       case GDK_KEY_BackSpace:
                        case 'B': case 'b':
                                view_step_prev(vw);
                                break;
-                       case GDK_Page_Down: case GDK_KP_Page_Down:
-                       case GDK_space:
+                       case GDK_KEY_Page_Down: case GDK_KEY_KP_Page_Down:
+                       case GDK_KEY_space:
                        case 'N': case 'n':
                                view_step_next(vw);
                                break;
-                       case GDK_Home: case GDK_KP_Home:
+                       case GDK_KEY_Home: case GDK_KEY_KP_Home:
                                view_step_to_end(vw, FALSE);
                                break;
-                       case GDK_End: case GDK_KP_End:
+                       case GDK_KEY_End: case GDK_KEY_KP_End:
                                view_step_to_end(vw, TRUE);
                                break;
-                       case '+': case '=': case GDK_KP_Add:
+                       case '+': case '=': case GDK_KEY_KP_Add:
                                image_zoom_adjust(imd, get_zoom_increment());
                                break;
-                       case '-': case GDK_KP_Subtract:
+                       case '-': case GDK_KEY_KP_Subtract:
                                image_zoom_adjust(imd, -get_zoom_increment());
                                break;
-                       case 'X': case 'x': case GDK_KP_Multiply:
+                       case 'X': case 'x': case GDK_KEY_KP_Multiply:
                                image_zoom_set(imd, 0.0);
                                break;
-                       case 'Z': case 'z': case GDK_KP_Divide: case '1':
+                       case 'Z': case 'z': case GDK_KEY_KP_Divide: case '1':
                                image_zoom_set(imd, 1.0);
                                break;
                        case '2':
@@ -548,25 +516,25 @@ static gint view_window_key_press_cb(GtkWidget *widget, GdkEventKey *event, gpoi
                                break;
                        case 'F': case 'f':
                        case 'V': case 'v':
-                       case GDK_F11:
+                       case GDK_KEY_F11:
                                view_fullscreen_toggle(vw, FALSE);
                                break;
                        case 'I': case 'i':
                                view_overlay_toggle(vw);
                                break;
                        case ']':
-                               image_alter(imd, ALTER_ROTATE_90);
+                               image_alter_orientation(imd, ALTER_ROTATE_90);
                                break;
                        case '[':
-                               image_alter(imd, ALTER_ROTATE_90_CC);
+                               image_alter_orientation(imd, ALTER_ROTATE_90_CC);
                                break;
-                       case GDK_Delete: case GDK_KP_Delete:
+                       case GDK_KEY_Delete: case GDK_KEY_KP_Delete:
                                if (options->file_ops.enable_delete_key)
                                        {
                                        file_util_delete(image_get_fd(imd), NULL, imd->widget);
                                        }
                                break;
-                       case GDK_Escape:
+                       case GDK_KEY_Escape:
                                if (vw->fs)
                                        {
                                        view_fullscreen_toggle(vw, TRUE);
@@ -576,8 +544,8 @@ static gint view_window_key_press_cb(GtkWidget *widget, GdkEventKey *event, gpoi
                                        view_window_close(vw);
                                        }
                                break;
-                       case GDK_Menu:
-                       case GDK_F10:
+                       case GDK_KEY_Menu:
+                       case GDK_KEY_F10:
                                menu = view_popup_menu(vw);
                                gtk_menu_popup(GTK_MENU(menu), NULL, NULL,
                                               view_window_menu_pos_cb, vw, 0, GDK_CURRENT_TIME);
@@ -596,14 +564,12 @@ static gint view_window_key_press_cb(GtkWidget *widget, GdkEventKey *event, gpoi
  * view window main routines
  *-----------------------------------------------------------------------------
  */
-
-static void button_cb(ImageWindow *imd, gint button, guint32 time,
-                     gdouble x, gdouble y, guint state, gpointer data)
+static void button_cb(ImageWindow *imd, GdkEventButton *event, gpointer data)
 {
        ViewWindow *vw = data;
        GtkWidget *menu;
 
-       switch (button)
+       switch (event->button)
                {
                case MOUSE_BUTTON_LEFT:
                        view_step_next(vw);
@@ -613,35 +579,34 @@ static void button_cb(ImageWindow *imd, gint button, guint32 time,
                        break;
                case MOUSE_BUTTON_RIGHT:
                        menu = view_popup_menu(vw);
-                       gtk_menu_popup(GTK_MENU(menu), NULL, NULL, NULL, NULL, 3, time);
+                       gtk_menu_popup(GTK_MENU(menu), NULL, NULL, NULL, NULL, 3, event->time);
                        break;
                default:
                        break;
                }
 }
 
-static void scroll_cb(ImageWindow *imd, GdkScrollDirection direction, guint32 time,
-                     gdouble x, gdouble y, guint state, gpointer data)
+static void scroll_cb(ImageWindow *imd, GdkEventScroll *event, gpointer data)
 {
        ViewWindow *vw = data;
 
-       if (state & GDK_CONTROL_MASK)
+       if (event->state & GDK_CONTROL_MASK)
                {
-               switch (direction)
+               switch (event->direction)
                        {
                        case GDK_SCROLL_UP:
-                               image_zoom_adjust_at_point(imd, get_zoom_increment(), x, y);
+                               image_zoom_adjust_at_point(imd, get_zoom_increment(), event->x, event->y);
                                break;
                        case GDK_SCROLL_DOWN:
-                               image_zoom_adjust_at_point(imd, -get_zoom_increment(), x, y);
+                               image_zoom_adjust_at_point(imd, -get_zoom_increment(), event->x, event->y);
                                break;
                        default:
                                break;
                        }
                }
-       else if ( (state & GDK_SHIFT_MASK) != (guint) (options->mousewheel_scrolls))
+       else if ( (event->state & GDK_SHIFT_MASK) != (guint) (options->mousewheel_scrolls))
                {
-               switch (direction)
+               switch (event->direction)
                        {
                        case GDK_SCROLL_UP:
                                image_scroll(imd, 0, -MOUSEWHEEL_SCROLL_SIZE);
@@ -661,7 +626,7 @@ static void scroll_cb(ImageWindow *imd, GdkScrollDirection direction, guint32 ti
                }
        else
                {
-               switch (direction)
+               switch (event->direction)
                        {
                        case GDK_SCROLL_UP:
                                view_step_prev(vw);
@@ -690,7 +655,7 @@ static void view_fullscreen_stop_func(FullScreenData *fs, gpointer data)
        if (vw->ss) vw->ss->imd = vw->imd;
 }
 
-static void view_fullscreen_toggle(ViewWindow *vw, gint force_off)
+static void view_fullscreen_toggle(ViewWindow *vw, gboolean force_off)
 {
        if (force_off && !vw->fs) return;
 
@@ -774,7 +739,7 @@ static void view_slideshow_start(ViewWindow *vw)
 
                if (vw->list)
                        {
-                       vw->ss = slideshow_start_from_filelist(view_window_active_image(vw),
+                       vw->ss = slideshow_start_from_filelist(NULL, view_window_active_image(vw),
                                                                filelist_copy(vw->list),
                                                                view_slideshow_stop_func, vw);
                        vw->list_pointer = NULL;
@@ -784,7 +749,7 @@ static void view_slideshow_start(ViewWindow *vw)
                cd = image_get_collection(view_window_active_image(vw), &info);
                if (cd && info)
                        {
-                       vw->ss = slideshow_start_from_collection(view_window_active_image(vw), cd,
+                       vw->ss = slideshow_start_from_collection(NULL, view_window_active_image(vw), cd,
                                                                 view_slideshow_stop_func, vw, info);
                        }
                }
@@ -805,6 +770,9 @@ static void view_window_destroy_cb(GtkWidget *widget, gpointer data)
        fullscreen_stop(vw->fs);
 
        filelist_free(vw->list);
+
+       file_data_unregister_notify_func(view_window_notify_cb, vw);
+
        g_free(vw);
 }
 
@@ -815,7 +783,7 @@ static void view_window_close(ViewWindow *vw)
        gtk_widget_destroy(vw->window);
 }
 
-static gint view_window_delete_cb(GtkWidget *w, GdkEventAny *event, gpointer data)
+static gboolean view_window_delete_cb(GtkWidget *w, GdkEventAny *event, gpointer data)
 {
        ViewWindow *vw = data;
 
@@ -833,27 +801,27 @@ static ViewWindow *real_view_window_new(FileData *fd, GList *list, CollectionDat
        if (!fd && !list && (!cd || !info)) return NULL;
 
        vw = g_new0(ViewWindow, 1);
-       vw->fs = NULL;
-       vw->ss = NULL;
-       vw->list = NULL;
-       vw->list_pointer = NULL;
 
        vw->window = window_new(GTK_WINDOW_TOPLEVEL, "view", PIXBUF_INLINE_ICON_VIEW, NULL, NULL);
 
-       geometry.min_width = 8;
-       geometry.min_height = 8;
+       geometry.min_width = DEFAULT_MINIMAL_WINDOW_SIZE;
+       geometry.min_height = DEFAULT_MINIMAL_WINDOW_SIZE;
        gtk_window_set_geometry_hints(GTK_WINDOW(vw->window), NULL, &geometry, GDK_HINT_MIN_SIZE);
 
        gtk_window_set_resizable(GTK_WINDOW(vw->window), TRUE);
        gtk_container_set_border_width(GTK_CONTAINER(vw->window), 0);
 
        vw->imd = image_new(FALSE);
+       image_color_profile_set(vw->imd,
+                               options->color_profile.input_type,
+                               options->color_profile.use_image);
+       image_color_profile_set_use(vw->imd, options->color_profile.enabled);
 
-       image_background_set_color(vw->imd, options->image.use_custom_border_color ? &options->image.border_color : NULL);
+       image_background_set_color_from_options(vw->imd, FALSE);
 
        image_attach_window(vw->imd, vw->window, NULL, GQ_APPNAME, TRUE);
 
-       image_auto_refresh(vw->imd, 0);
+       image_auto_refresh_enable(vw->imd, TRUE);
        image_top_window_set_sync(vw->imd, TRUE);
 
        gtk_container_add(GTK_CONTAINER(vw->window), vw->imd->widget);
@@ -871,7 +839,7 @@ static ViewWindow *real_view_window_new(FileData *fd, GList *list, CollectionDat
                         G_CALLBACK(view_window_key_press_cb), vw);
        if (cd && info)
                {
-               image_change_from_collection(vw->imd, cd, info, image_zoom_get_default(NULL, options->image.zoom_mode));
+               image_change_from_collection(vw->imd, cd, info, image_zoom_get_default(NULL));
                if (options->image.enable_read_ahead)
                        {
                        CollectInfo * r_info = collection_next_by_info(cd, info);
@@ -883,7 +851,7 @@ static ViewWindow *real_view_window_new(FileData *fd, GList *list, CollectionDat
                {
                view_window_set_list(vw, list);
                vw->list_pointer = vw->list;
-               image_change_fd(vw->imd, (FileData *)vw->list->data, image_zoom_get_default(NULL, options->image.zoom_mode));
+               image_change_fd(vw->imd, (FileData *)vw->list->data, image_zoom_get_default(NULL));
 
                if (options->image.enable_read_ahead)
                        {
@@ -893,7 +861,15 @@ static ViewWindow *real_view_window_new(FileData *fd, GList *list, CollectionDat
                }
        else
                {
-               image_change_fd(vw->imd, fd, image_zoom_get_default(NULL, options->image.zoom_mode));
+               image_change_fd(vw->imd, fd, image_zoom_get_default(NULL));
+               }
+
+       /* Wait until image is loaded otherwise size is not defined */
+       int count;
+       for (count = 10; count && !w && !h; count++)
+               {
+               image_get_image_size(vw->imd, &w, &h);
+               usleep(100000);
                }
 
        if (image_zoom_get(vw->imd) == 0.0)
@@ -925,6 +901,8 @@ static ViewWindow *real_view_window_new(FileData *fd, GList *list, CollectionDat
 
        view_window_list = g_list_append(view_window_list, vw);
 
+       file_data_register_notify_func(view_window_notify_cb, vw, NOTIFY_PRIORITY_LOW);
+
        return vw;
 }
 
@@ -963,8 +941,8 @@ void view_window_new(FileData *fd)
                                         G_CALLBACK(view_window_collection_unref_cb), cd);
                        }
                }
-       else if (isdir(fd->path) && filelist_read(fd->path, &list, NULL))
-               {       
+       else if (isdir(fd->path) && filelist_read(fd, &list, NULL))
+               {
                list = filelist_sort_path(list);
                list = filelist_filter(list, FALSE);
                real_view_window_new(NULL, list, NULL, NULL);
@@ -1001,12 +979,12 @@ void view_window_colors_update(void)
                {
                ViewWindow *vw = work->data;
                work = work->next;
-
-               image_background_set_color(vw->imd, options->image.use_custom_border_color ? &options->image.border_color : NULL);
+               
+               image_background_set_color_from_options(vw->imd, !!vw->fs);
                }
 }
 
-gint view_window_find_image(ImageWindow *imd, gint *index, gint *total)
+gboolean view_window_find_image(ImageWindow *imd, gint *index, gint *total)
 {
        GList *work;
 
@@ -1070,19 +1048,18 @@ static void view_edit_cb(GtkWidget *widget, gpointer data)
 {
        ViewWindow *vw;
        ImageWindow *imd;
-       gint n;
+       const gchar *key = data;
 
        vw = submenu_item_get_data(widget);
-       n = GPOINTER_TO_INT(data);
        if (!vw) return;
 
-       if (!editor_window_flag_set(n))
+       if (!editor_window_flag_set(key))
                {
                view_fullscreen_toggle(vw, TRUE);
                }
 
        imd = view_window_active_image(vw);
-       file_util_start_editor_from_file(n, image_get_fd(imd), imd->widget);
+       file_util_start_editor_from_file(key, image_get_fd(imd), imd->widget);
 }
 
 static void view_alter_cb(GtkWidget *widget, gpointer data)
@@ -1094,16 +1071,7 @@ static void view_alter_cb(GtkWidget *widget, gpointer data)
        type = GPOINTER_TO_INT(data);
 
        if (!vw) return;
-       image_alter(vw->imd, type);
-}
-
-static void view_info_cb(GtkWidget *widget, gpointer data)
-{
-       ViewWindow *vw = data;
-       ImageWindow *imd;
-
-       imd = view_window_active_image(vw);
-       info_window_new(image_get_fd(imd), NULL, vw->fs ? vw->fs->window : NULL);
+       image_alter_orientation(vw->imd, type);
 }
 
 static void view_wallpaper_cb(GtkWidget *widget, gpointer data)
@@ -1223,13 +1191,13 @@ static void view_close_cb(GtkWidget *widget, gpointer data)
        view_window_close(vw);
 }
 
-static LayoutWindow *view_new_layout_with_path(const gchar *path)
+static LayoutWindow *view_new_layout_with_fd(FileData *fd)
 {
        LayoutWindow *nw;
 
-       nw = layout_new(NULL, FALSE, FALSE);
+       nw = layout_new(NULL, NULL);
        layout_sort_set(nw, options->file_sort.method, options->file_sort.ascending);
-       layout_set_path(nw, path);
+       layout_set_fd(nw, fd);
        return nw;
 }
 
@@ -1238,44 +1206,65 @@ static void view_set_layout_path_cb(GtkWidget *widget, gpointer data)
 {
        ViewWindow *vw = data;
        LayoutWindow *lw;
-       const gchar *path;
        ImageWindow *imd;
 
        imd = view_window_active_image(vw);
 
        if (!imd || !imd->image_fd) return;
-       path = imd->image_fd->path;
-       if (!path) return;
 
        lw = layout_find_by_image_fd(imd);
        if (lw)
-               layout_set_path(lw, path);
+               layout_set_fd(lw, imd->image_fd);
        else
-               view_new_layout_with_path(path);
+               view_new_layout_with_fd(imd->image_fd);
        view_window_close(vw);
 }
 
+static void view_popup_menu_destroy_cb(GtkWidget *widget, gpointer data)
+{
+       GList *editmenu_fd_list = data;
+
+       filelist_free(editmenu_fd_list);
+}
+
+static GList *view_window_get_fd_list(ViewWindow *vw)
+{
+       GList *list = NULL;
+       ImageWindow *imd = view_window_active_image(vw);
+
+       if (imd)
+               {
+               FileData *fd = image_get_fd(imd);
+               if (fd) list = g_list_append(NULL, file_data_ref(fd));
+               }
+       
+       return list;
+}
+
 static GtkWidget *view_popup_menu(ViewWindow *vw)
 {
        GtkWidget *menu;
        GtkWidget *item;
+       GList *editmenu_fd_list;
 
        menu = popup_menu_short_lived();
 
+
        menu_item_add_stock(menu, _("Zoom _in"), GTK_STOCK_ZOOM_IN, G_CALLBACK(view_zoom_in_cb), vw);
        menu_item_add_stock(menu, _("Zoom _out"), GTK_STOCK_ZOOM_OUT, G_CALLBACK(view_zoom_out_cb), vw);
        menu_item_add_stock(menu, _("Zoom _1:1"), GTK_STOCK_ZOOM_100, G_CALLBACK(view_zoom_1_1_cb), vw);
        menu_item_add_stock(menu, _("Fit image to _window"), GTK_STOCK_ZOOM_FIT, G_CALLBACK(view_zoom_fit_cb), vw);
        menu_item_add_divider(menu);
 
-       item = submenu_add_edit(menu, NULL, G_CALLBACK(view_edit_cb), vw);
+       editmenu_fd_list = view_window_get_fd_list(vw);
+       g_signal_connect(G_OBJECT(menu), "destroy",
+                        G_CALLBACK(view_popup_menu_destroy_cb), editmenu_fd_list);
+       item = submenu_add_edit(menu, NULL, G_CALLBACK(view_edit_cb), vw, editmenu_fd_list);
        menu_item_add_divider(item);
        menu_item_add(item, _("Set as _wallpaper"), G_CALLBACK(view_wallpaper_cb), vw);
 
        submenu_add_alter(menu, G_CALLBACK(view_alter_cb), vw);
 
-       menu_item_add_stock(menu, _("_Properties"), GTK_STOCK_PROPERTIES, G_CALLBACK(view_info_cb), vw);
-
        menu_item_add_stock(menu, _("View in _new window"), GTK_STOCK_NEW, G_CALLBACK(view_new_window_cb), vw);
        item = menu_item_add(menu, _("_Go to directory view"), G_CALLBACK(view_set_layout_path_cb), vw);
 
@@ -1284,8 +1273,7 @@ static GtkWidget *view_popup_menu(ViewWindow *vw)
        menu_item_add(menu, _("_Move..."), G_CALLBACK(view_move_cb), vw);
        menu_item_add(menu, _("_Rename..."), G_CALLBACK(view_rename_cb), vw);
        menu_item_add_stock(menu, _("_Delete..."), GTK_STOCK_DELETE, G_CALLBACK(view_delete_cb), vw);
-       if (options->show_copy_path)
-               menu_item_add(menu, _("_Copy path"), G_CALLBACK(view_copy_path_cb), vw);
+       menu_item_add(menu, _("_Copy path"), G_CALLBACK(view_copy_path_cb), vw);
 
        menu_item_add_divider(menu);
 
@@ -1342,7 +1330,7 @@ static void view_dir_list_cancel(GtkWidget *widget, gpointer data)
        /* do nothing */
 }
 
-static void view_dir_list_do(ViewWindow *vw, GList *list, gint skip, gint recurse)
+static void view_dir_list_do(ViewWindow *vw, GList *list, gboolean skip, gboolean recurse)
 {
        GList *work;
 
@@ -1362,11 +1350,11 @@ static void view_dir_list_do(ViewWindow *vw, GList *list, gint skip, gint recurs
 
                                if (recurse)
                                        {
-                                       list = filelist_recursive(fd->path);
+                                       list = filelist_recursive(fd);
                                        }
                                else
                                        { /*FIXME */
-                                       filelist_read(fd->path, &list, NULL);
+                                       filelist_read(fd, &list, NULL);
                                        list = filelist_sort_path(list);
                                        list = filelist_filter(list, FALSE);
                                        }
@@ -1386,7 +1374,7 @@ static void view_dir_list_do(ViewWindow *vw, GList *list, gint skip, gint recurs
 
                vw->list_pointer = vw->list;
                fd = vw->list->data;
-               image_change_fd(vw->imd, fd, image_zoom_get_default(vw->imd, options->image.zoom_mode));
+               image_change_fd(vw->imd, fd, image_zoom_get_default(vw->imd));
 
                work = vw->list->next;
                if (options->image.enable_read_ahead && work)
@@ -1397,7 +1385,7 @@ static void view_dir_list_do(ViewWindow *vw, GList *list, gint skip, gint recurs
                }
        else
                {
-               image_change_fd(vw->imd, NULL, image_zoom_get_default(vw->imd, options->image.zoom_mode));
+               image_change_fd(vw->imd, NULL, image_zoom_get_default(vw->imd));
                }
 }
 
@@ -1478,7 +1466,7 @@ static void view_window_get_dnd_data(GtkWidget *widget, GdkDragContext *context,
                        {
                        GList *work;
 
-                       list = uri_filelist_from_text((gchar *)selection_data->data, TRUE);
+                       list = uri_filelist_from_gtk_selection_data(selection_data);
 
                        work = list;
                        while (work)
@@ -1501,7 +1489,7 @@ static void view_window_get_dnd_data(GtkWidget *widget, GdkDragContext *context,
                        }
                else
                        {
-                       source = collection_from_dnd_data((gchar *)selection_data->data, &list, &info_list);
+                       source = collection_from_dnd_data((gchar *)gtk_selection_data_get_data(selection_data), &list, &info_list);
                        }
 
                if (list)
@@ -1516,7 +1504,7 @@ static void view_window_get_dnd_data(GtkWidget *widget, GdkDragContext *context,
 
                                if (source && info_list)
                                        {
-                                       image_change_from_collection(imd, source, info_list->data, image_zoom_get_default(imd, options->image.zoom_mode));
+                                       image_change_from_collection(imd, source, info_list->data, image_zoom_get_default(imd));
                                        }
                                else
                                        {
@@ -1527,7 +1515,7 @@ static void view_window_get_dnd_data(GtkWidget *widget, GdkDragContext *context,
 
                                                vw->list_pointer = vw->list;
                                                }
-                                       image_change_fd(imd, fd, image_zoom_get_default(imd, options->image.zoom_mode));
+                                       image_change_fd(imd, fd, image_zoom_get_default(imd));
                                        }
                                }
                        }
@@ -1547,34 +1535,15 @@ static void view_window_set_dnd_data(GtkWidget *widget, GdkDragContext *context,
 
        if (fd)
                {
-               gchar *text = NULL;
-               gint len;
-               gint plain_text;
                GList *list;
 
-               switch (info)
-                       {
-                       case TARGET_URI_LIST:
-                               plain_text = FALSE;
-                               break;
-                       case TARGET_TEXT_PLAIN:
-                       default:
-                               plain_text = TRUE;
-                               break;
-                       }
                list = g_list_append(NULL, fd);
-               text = uri_text_from_filelist(list, &len, plain_text);
+               uri_selection_data_set_uris_from_filelist(selection_data, list);
                g_list_free(list);
-               if (text)
-                       {
-                       gtk_selection_data_set(selection_data, selection_data->target,
-                                              8, (guchar *)text, len);
-                       g_free(text);
-                       }
                }
        else
                {
-               gtk_selection_data_set(selection_data, selection_data->target,
+               gtk_selection_data_set(selection_data, gtk_selection_data_get_target(selection_data),
                                       8, NULL, 0);
                }
 }
@@ -1605,7 +1574,7 @@ static void view_window_dnd_init(ViewWindow *vw)
  *-----------------------------------------------------------------------------
  */
 
-static void view_real_removed(ViewWindow *vw, FileData *fd, GList *ignore_list)
+static void view_real_removed(ViewWindow *vw, FileData *fd)
 {
        ImageWindow *imd;
        FileData *image_fd;
@@ -1633,7 +1602,7 @@ static void view_real_removed(ViewWindow *vw, FileData *fd, GList *ignore_list)
                        }
                if (image_get_fd(imd) == image_fd)
                        {
-                       image_change_fd(imd, NULL, image_zoom_get_default(imd, options->image.zoom_mode));
+                       image_change_fd(imd, NULL, image_zoom_get_default(imd));
                        }
                }
 
@@ -1679,72 +1648,34 @@ static void view_real_removed(ViewWindow *vw, FileData *fd, GList *ignore_list)
                                fd = NULL;
                                }
 
-                       image_change_fd(imd, fd, image_zoom_get_default(imd, options->image.zoom_mode));
+                       image_change_fd(imd, fd, image_zoom_get_default(imd));
                        }
                }
 
        image_osd_update(imd);
 }
 
-static void view_real_moved(ViewWindow *vw, FileData *fd)
-{
-/*
-       ImageWindow *imd;
-       const gchar *image_path;
-
-       imd = view_window_active_image(vw);
-*/
-/*
-       image_path = image_get_path(imd);
-
-       if (image_path && strcmp(image_path, fd->change->source) == 0)
-               {
-               image_set_fd(imd, dest);
-               }
-*/
-/*
-       if (vw->list)
-               {
-               GList *work;
-               work = vw->list;
-               while (work)
-                       {
-                       gchar *chk_path;
-
-                       chk_path = work->data;
-
-                       if (strcmp(chk_path, source) == 0)
-                               {
-                               work->data = g_strdup(dest);
-                               g_free(chk_path);
-                               }
-
-                       work = work->next;
-                       }
-               }
-*/
-}
-
-void view_window_maint_removed(FileData *fd, GList *ignore_list)
+static void view_window_notify_cb(FileData *fd, NotifyType type, gpointer data)
 {
-       GList *work = view_window_list;
-       while (work)
-               {
-               ViewWindow *vw = work->data;
-               work = work->next;
+       ViewWindow *vw = data;
 
-               view_real_removed(vw, fd, ignore_list);
-               }
-}
+       if (!(type & NOTIFY_CHANGE) || !fd->change) return;
+       
+       DEBUG_1("Notify view_window: %s %04x", fd->path, type);
 
-void view_window_maint_moved(FileData *fd)
-{
-       GList *work = view_window_list;
-       while (work)
+       switch (fd->change->type)
                {
-               ViewWindow *vw = work->data;
-               work = work->next;
-
-               view_real_moved(vw, fd);
+               case FILEDATA_CHANGE_MOVE:
+               case FILEDATA_CHANGE_RENAME:
+                       break;
+               case FILEDATA_CHANGE_COPY:
+                       break;
+               case FILEDATA_CHANGE_DELETE:
+                       view_real_removed(vw, fd);
+                       break;
+               case FILEDATA_CHANGE_UNSPECIFIED:
+               case FILEDATA_CHANGE_WRITE_METADATA:
+                       break;
                }
 }
+/* vim: set shiftwidth=8 softtabstop=0 cindent cinoptions={1s: */