* Added Progress Object
[modest] / src / widgets / modest-folder-view.c
index ca97905..4e90143 100644 (file)
@@ -258,7 +258,19 @@ get_cached_icon (const gchar *name)
                                                         MODEST_CACHE_MGR_CACHE_TYPE_PIXBUF);
        
        if (!icon_cache || !g_hash_table_lookup_extended (icon_cache, name, &orig_key, &pixbuf)) {
-               pixbuf = (gpointer)gdk_pixbuf_new_from_file (name, &err);
+
+#if MODEST_PLATFORM_ID==1  /* MODES_PLATFORM_ID: 1 ==> gnome, 2==> maemo */ 
+               pixbuf = (gpointer) gdk_pixbuf_new_from_file (name, &err);
+#else
+               GtkIconTheme *current_theme;
+               current_theme = gtk_icon_theme_get_default ();
+               pixbuf = gtk_icon_theme_load_icon (current_theme,
+                                                  name,
+                                                  26,
+                                                  GTK_ICON_LOOKUP_NO_SVG,
+                                                  NULL);
+#endif
+
                if (!pixbuf) {
                        g_printerr ("modest: error in icon factory while loading '%s': %s\n",
                                    name, err->message);
@@ -352,7 +364,6 @@ modest_folder_view_init (ModestFolderView *obj)
        priv->query          = NULL;
 
        column = gtk_tree_view_column_new ();   
-       gtk_tree_view_append_column (GTK_TREE_VIEW(obj),column);
        
        renderer = gtk_cell_renderer_pixbuf_new();
        gtk_tree_view_column_pack_start (column, renderer, FALSE);
@@ -373,6 +384,8 @@ modest_folder_view_init (ModestFolderView *obj)
        gtk_tree_view_set_headers_clickable (GTK_TREE_VIEW(obj), FALSE);
        gtk_tree_view_set_enable_search     (GTK_TREE_VIEW(obj), FALSE);
 
+       gtk_tree_view_append_column (GTK_TREE_VIEW(obj),column);
+
        setup_drag_and_drop (GTK_TREE_VIEW(obj));
 }
 
@@ -546,7 +559,7 @@ update_model (ModestFolderView *self, ModestTnyAccountStore *account_store)
 
        priv =  MODEST_FOLDER_VIEW_GET_PRIVATE(self);
        
-       /* Notify that there is no folder selected */
+       /* Notify that there is no folder selected*/
        g_signal_emit (G_OBJECT(self), 
                       signals[FOLDER_SELECTION_CHANGED_SIGNAL], 0,
                       NULL, TRUE);
@@ -749,8 +762,6 @@ on_drag_data_get (GtkWidget *widget,
 
 typedef struct _DndHelper {
        gboolean delete_source;
-       GtkWidget *source_widget;
-       GtkWidget *dest_widget;
        GtkTreePath *source_row;
        GdkDragContext *context;
        guint time;
@@ -758,64 +769,38 @@ typedef struct _DndHelper {
 
 
 /*
- * This function saves the source row in the source widget, will be
- * used by the drag-data-delete handler to remove the source row
- */
-static void
-save_and_clean (DndHelper *helper, 
-               gboolean success)
-{
-       /* Save row data */
-       if (success && helper->delete_source)
-               g_object_set_data (G_OBJECT (helper->source_widget),
-                                  ROW_REF_DATA_NAME,
-                                  gtk_tree_path_copy (helper->source_row));
-
-       /* Clean dest row */
-       gtk_tree_view_set_drag_dest_row (GTK_TREE_VIEW (helper->dest_widget),
-                                        NULL,
-                                        GTK_TREE_VIEW_DROP_BEFORE);
-
-}
-
-/*
  * This function is the callback of the
- * modest_mail_operation_xfer_msg() call. We check here if the message
- * was correctly asynchronously transfered
+ * modest_mail_operation_xfer_msg() and
+ * modest_mail_operation_xfer_folder() calls. We check here if the
+ * message/folder was correctly asynchronously transferred. The reason
+ * to use the same callback is that the code is the same, it only has
+ * to check that the operation went fine and then finalize the drag
+ * and drop action
  */
 static void
 on_progress_changed (ModestMailOperation *mail_op, gpointer user_data)
 {
-       ModestMailOperationQueue *queue;
-       gboolean success = FALSE;
+       gboolean success;
        DndHelper *helper;
 
        helper = (DndHelper *) user_data;
 
+       if (!modest_mail_operation_is_finished (mail_op))
+               return;
+
        if (modest_mail_operation_get_status (mail_op) == 
            MODEST_MAIL_OPERATION_STATUS_SUCCESS) {
                success = TRUE;
        } else {
-               const GError *error;
-               error = modest_mail_operation_get_error (mail_op);
-               g_warning ("Error transferring messages: %s\n", error->message);
+               success = FALSE;
        }
 
-       /* Remove the mail operation */ 
-       queue = modest_runtime_get_mail_operation_queue ();
-       modest_mail_operation_queue_remove (queue, mail_op);
-       g_object_unref (G_OBJECT (mail_op));
-
-       /* Save and clean. HACK: Force success to FALSE in order not
-          to save data in the source widget */
-       gtk_tree_path_free (helper->source_row);
-       save_and_clean (helper, FALSE);
-       
        /* Notify the drag source. Never call delete, the monitor will
           do the job if needed */
        gtk_drag_finish (helper->context, success, FALSE, helper->time);
 
        /* Free the helper */
+       gtk_tree_path_free (helper->source_row);        
        g_slice_free (DndHelper, helper);
 }
 
@@ -832,9 +817,7 @@ drag_and_drop_from_header_view (GtkTreeModel *source_model,
 {
        TnyHeader *header;
        TnyFolder *folder;
-       ModestMailOperationQueue *queue;
        ModestMailOperation *mail_op;
-       gboolean started;
        GtkTreeIter source_iter, dest_iter;
 
        /* Get header */
@@ -850,23 +833,13 @@ drag_and_drop_from_header_view (GtkTreeModel *source_model,
                            &folder, -1);
 
        /* Transfer message */
-       queue = modest_runtime_get_mail_operation_queue ();
        mail_op = modest_mail_operation_new ();
-       started = modest_mail_operation_xfer_msg (mail_op, header,
-                                                 folder, helper->delete_source);
-       if (started) {
-               g_signal_connect (G_OBJECT (mail_op), "progress_changed",
-                                 G_CALLBACK (on_progress_changed), helper);
-               modest_mail_operation_queue_add (queue, mail_op);
-       } else {
-               const GError *error;
-               error = modest_mail_operation_get_error (mail_op);
-               if (error)
-                       g_warning ("Error trying to transfer messages: %s\n",
-                                  error->message);
+       modest_mail_operation_queue_add (modest_runtime_get_mail_operation_queue (), 
+                                        mail_op);
+       g_signal_connect (G_OBJECT (mail_op), "progress-changed",
+                         G_CALLBACK (on_progress_changed), helper);
 
-               g_slice_free (DndHelper, helper);
-       }
+       modest_mail_operation_xfer_msg (mail_op, header, folder, helper->delete_source);
 
        /* Frees */
        g_object_unref (G_OBJECT (mail_op));
@@ -887,21 +860,21 @@ drag_and_drop_from_folder_view (GtkTreeModel     *source_model,
                                DndHelper        *helper)
 {
        ModestMailOperation *mail_op;
-       const GError *error;
-       GtkTreeRowReference *source_row_reference;
        GtkTreeIter parent_iter, iter;
-       TnyFolder *folder, *new_folder;
        TnyFolderStore *parent_folder;
-       gboolean success = FALSE;
+       TnyFolder *folder;
 
        /* Check if the drag is possible */
-       if (!gtk_tree_path_compare (helper->source_row, dest_row))
-               goto out;
-
-       if (!gtk_tree_drag_dest_row_drop_possible (GTK_TREE_DRAG_DEST (dest_model),
+       if (!gtk_tree_path_compare (helper->source_row, dest_row) ||
+           !gtk_tree_drag_dest_row_drop_possible (GTK_TREE_DRAG_DEST (dest_model),
                                                   dest_row,
-                                                  selection_data))
-               goto out;
+                                                  selection_data)) {
+
+               gtk_drag_finish (helper->context, FALSE, FALSE, helper->time);
+               gtk_tree_path_free (helper->source_row);        
+               g_slice_free (DndHelper, helper);
+               return;
+       }
 
        /* Get data */
        gtk_tree_model_get_iter (source_model, &parent_iter, dest_row);
@@ -915,53 +888,20 @@ drag_and_drop_from_folder_view (GtkTreeModel     *source_model,
 
        /* Do the mail operation */
        mail_op = modest_mail_operation_new ();
-       new_folder = modest_mail_operation_xfer_folder (mail_op, folder, parent_folder, 
-                                                       helper->delete_source);
+       modest_mail_operation_queue_add (modest_runtime_get_mail_operation_queue (), 
+                                        mail_op);
+       g_signal_connect (G_OBJECT (mail_op), "progress-changed",
+                         G_CALLBACK (on_progress_changed), helper);
+
+       modest_mail_operation_xfer_folder (mail_op, 
+                                          folder, 
+                                          parent_folder,
+                                          helper->delete_source);
 
+       /* Frees */
        g_object_unref (G_OBJECT (parent_folder));
        g_object_unref (G_OBJECT (folder));
-
-       error = modest_mail_operation_get_error (mail_op);
-       if (error) {
-               g_warning ("Error transferring folder: %s\n", error->message);
-               g_object_unref (G_OBJECT (mail_op));
-               goto out;
-       }
        g_object_unref (G_OBJECT (mail_op));
-
-       /* Get a row reference to the source path because the path
-          could change after the insertion. The gtk_drag_finish() is
-          not able to delete the source because that, so we have to
-          do it manually */
-       source_row_reference = gtk_tree_row_reference_new (source_model, helper->source_row);
-       gtk_tree_path_free (helper->source_row);
-
-       /* Insert the dragged row as a child of the dest row */
-       gtk_tree_path_down (dest_row);
-       if (gtk_tree_drag_dest_drag_data_received (GTK_TREE_DRAG_DEST (dest_model),
-                                                  dest_row,
-                                                  selection_data)) {
-
-               GtkTreeIter iter;
-
-               /* Set the newly created folder as the instance in the row */
-               gtk_tree_model_get_iter (dest_model, &iter, dest_row);
-               gtk_tree_store_set (GTK_TREE_STORE (dest_model), &iter,
-                                   TNY_GTK_FOLDER_STORE_TREE_MODEL_INSTANCE_COLUMN, 
-                                   new_folder, -1);
-               g_object_unref (G_OBJECT (new_folder));         
-
-               helper->source_row = gtk_tree_row_reference_get_path (source_row_reference);
-
-               success = TRUE;
-       }
-       gtk_tree_row_reference_free (source_row_reference);
-
-       /* Save and clean */
-       save_and_clean (helper, success);       
-
- out:
-       gtk_drag_finish (helper->context, success, (success && helper->delete_source), helper->time);
 }
 
 /*
@@ -996,7 +936,7 @@ on_drag_data_received (GtkWidget *widget,
 
        /* Check if the get_data failed */
        if (selection_data == NULL || selection_data->length < 0)
-               gtk_drag_finish (context, success, (success && delete_source), time);
+               gtk_drag_finish (context, success, FALSE, time);
 
        /* Get the models */
        source_widget = gtk_drag_get_source_widget (context);
@@ -1020,14 +960,12 @@ on_drag_data_received (GtkWidget *widget,
 
        /* Only allow drops IN other rows */
        if (!dest_row || pos == GTK_TREE_VIEW_DROP_BEFORE || pos == GTK_TREE_VIEW_DROP_AFTER)
-               gtk_drag_finish (context, success, (success && delete_source), time);
+               gtk_drag_finish (context, success, FALSE, time);
 
        /* Create the helper */
        helper = g_slice_new0 (DndHelper);
        helper->delete_source = delete_source;
-       helper->source_widget = source_widget;
-       helper->dest_widget = widget;
-       helper->source_row = source_row;
+       helper->source_row = gtk_tree_path_copy (source_row);
        helper->context = context;
        helper->time = time;
 
@@ -1053,6 +991,9 @@ on_drag_data_received (GtkWidget *widget,
                                                selection_data, 
                                                helper);
        }
+
+       /* Frees */
+       gtk_tree_path_free (source_row);
        gtk_tree_path_free (child_dest_row);
 }
 
@@ -1086,31 +1027,6 @@ drag_drop_cb (GtkWidget      *widget,
 }
 
 /*
- * This function deletes the data that has been dragged from its
- * source widget. Since is a function received by the source of the
- * drag, this function only deletes rows of the folder view
- * widget. The header view widget will need to define its own one.
- */
-static void 
-drag_data_delete_cb (GtkWidget      *widget,
-                    GdkDragContext *context,
-                    gpointer        user_data)
-{
-       GtkTreePath *source_row;
-       GtkTreeModel *model_sort, *model;
-
-       model_sort = gtk_tree_view_get_model (GTK_TREE_VIEW (widget));
-       model = gtk_tree_model_sort_get_model (GTK_TREE_MODEL_SORT (model_sort));
-       source_row = g_object_steal_data (G_OBJECT (widget), ROW_REF_DATA_NAME);
-
-       /* Delete the source row */
-       gtk_tree_drag_source_drag_data_delete (GTK_TREE_DRAG_SOURCE (model),
-                                              source_row);
-
-       gtk_tree_path_free (source_row);
-}
-
-/*
  * This function expands a node of a tree view if it's not expanded
  * yet. Not sure why it needs the threads stuff, but gtk+`example code
  * does that, so that's why they're here.
@@ -1167,6 +1083,7 @@ on_drag_motion (GtkWidget      *widget,
        GtkTreePath *dest_row;
        ModestFolderViewPrivate *priv;
        GdkDragAction suggested_action;
+       gboolean valid_location = FALSE;
 
        priv = MODEST_FOLDER_VIEW_GET_PRIVATE (widget);
 
@@ -1180,8 +1097,17 @@ on_drag_motion (GtkWidget      *widget,
                                           &dest_row,
                                           &pos);
 
-       if (!dest_row)
-               return FALSE;
+       /* Do not allow drops between folders */
+       if (!dest_row ||
+           pos == GTK_TREE_VIEW_DROP_BEFORE || 
+           pos == GTK_TREE_VIEW_DROP_AFTER) {
+               gtk_tree_view_set_drag_dest_row(GTK_TREE_VIEW (widget), NULL, 0);
+               gdk_drag_status(context, 0, time);
+               valid_location = FALSE;
+               goto out;
+       } else {
+               valid_location = TRUE;
+       }
 
        /* Expand the selected row after 1/2 second */
        if (!gtk_tree_view_row_expanded (GTK_TREE_VIEW (widget), dest_row)) {
@@ -1202,15 +1128,17 @@ on_drag_motion (GtkWidget      *widget,
        else
             gdk_drag_status(context, GDK_ACTION_DEFAULT, time);
 
-       return TRUE;
+ out:
+       g_signal_stop_emission_by_name (widget, "drag-motion");
+       return valid_location;
 }
 
 
 /* Folder view drag types */
 const GtkTargetEntry folder_view_drag_types[] =
 {
-       { "GTK_TREE_MODEL_ROW", GTK_TARGET_SAME_WIDGET, FOLDER_ROW },
-       { "GTK_TREE_MODEL_ROW", GTK_TARGET_SAME_APP, HEADER_ROW }
+       { "GTK_TREE_MODEL_ROW", GTK_TARGET_SAME_WIDGET, MODEST_FOLDER_ROW },
+       { "GTK_TREE_MODEL_ROW", GTK_TARGET_SAME_APP,    MODEST_HEADER_ROW }
 };
 
 /*
@@ -1229,10 +1157,10 @@ setup_drag_and_drop (GtkTreeView *self)
                           G_N_ELEMENTS (folder_view_drag_types),
                           GDK_ACTION_MOVE | GDK_ACTION_COPY);
 
-       gtk_signal_connect(GTK_OBJECT (self),
-                          "drag_data_received",
-                          GTK_SIGNAL_FUNC(on_drag_data_received),
-                          NULL);
+       g_signal_connect (G_OBJECT (self),
+                         "drag_data_received",
+                         G_CALLBACK (on_drag_data_received),
+                         NULL);
 
 
        /* Set up the treeview as a dnd source */
@@ -1242,24 +1170,18 @@ setup_drag_and_drop (GtkTreeView *self)
                             G_N_ELEMENTS (folder_view_drag_types),
                             GDK_ACTION_MOVE | GDK_ACTION_COPY);
 
-       gtk_signal_connect(GTK_OBJECT (self),
-                          "drag_data_delete",
-                          GTK_SIGNAL_FUNC(drag_data_delete_cb),
-                          NULL);
-
-       gtk_signal_connect(GTK_OBJECT (self),
-                          "drag_motion",
-                          GTK_SIGNAL_FUNC(on_drag_motion),
-                          NULL);
-
-
-       gtk_signal_connect(GTK_OBJECT (self),
-                          "drag_data_get",
-                          GTK_SIGNAL_FUNC(on_drag_data_get),
-                          NULL);
-
-       gtk_signal_connect(GTK_OBJECT (self),
-                          "drag_drop",
-                          GTK_SIGNAL_FUNC(drag_drop_cb),
-                          NULL);
+       g_signal_connect (G_OBJECT (self),
+                         "drag_motion",
+                         G_CALLBACK (on_drag_motion),
+                         NULL);
+       
+       g_signal_connect (G_OBJECT (self),
+                         "drag_data_get",
+                         G_CALLBACK (on_drag_data_get),
+                         NULL);
+
+       g_signal_connect (G_OBJECT (self),
+                         "drag_drop",
+                         G_CALLBACK (drag_drop_cb),
+                         NULL);
 }