X-Git-Url: http://git.maemo.org/git/?a=blobdiff_plain;f=src%2Fwidgets%2Fmodest-header-view.c;h=a0454e75ee28eb2b9b6952e1d7ac7d44b3e8d997;hb=7691addfd0661f493672b2fab5274ca28612707d;hp=ad8371a937069251279fa7d2da967fb40c498c41;hpb=acf718d4797fe2ecc45f3dc1dd2f60a614784e94;p=modest diff --git a/src/widgets/modest-header-view.c b/src/widgets/modest-header-view.c index ad8371a..a0454e7 100644 --- a/src/widgets/modest-header-view.c +++ b/src/widgets/modest-header-view.c @@ -31,6 +31,7 @@ #include #include #include +#include #include #include @@ -70,7 +71,11 @@ static gboolean filter_row (GtkTreeModel *model, GtkTreeIter *iter, gpointer data); -static void on_selection_changed (GtkTreeSelection *sel, +static void on_account_removed (TnyAccountStore *self, + TnyAccount *account, + gpointer user_data); + +static void on_selection_changed (GtkTreeSelection *sel, gpointer user_data); static void setup_drag_and_drop (GtkTreeView *self); @@ -104,12 +109,15 @@ struct _ModestHeaderViewPrivate { /* Filter tree model */ gchar **hidding_ids; - guint n_selected; + guint n_selected; - gint sort_colid[2][TNY_FOLDER_TYPE_NUM]; - gint sort_type[2][TNY_FOLDER_TYPE_NUM]; + gint sort_colid[2][TNY_FOLDER_TYPE_NUM]; + gint sort_type[2][TNY_FOLDER_TYPE_NUM]; + gulong selection_changed_handler; + gulong acc_removed_handler; + gboolean empty; }; typedef struct _HeadersCountChangedHelper HeadersCountChangedHelper; @@ -321,27 +329,35 @@ modest_header_view_set_columns (ModestHeaderView *self, const GList *columns, Tn modest_hbox_cell_renderer_append (MODEST_HBOX_CELL_RENDERER (renderer_recpt_box), renderer_compact_date_or_status, FALSE); g_object_set_data (G_OBJECT (renderer_recpt_box), "date-renderer", renderer_compact_date_or_status); + g_object_set (G_OBJECT (renderer_subject_box), "yalign", 1.0, NULL); + gtk_cell_renderer_set_fixed_size (renderer_subject_box, -1, 32); + gtk_cell_renderer_set_fixed_size (renderer_recpt_box, -1, 32); + g_object_set (G_OBJECT (renderer_recpt_box), "yalign", 0.0, NULL); g_object_set(G_OBJECT(renderer_header), "ellipsize", PANGO_ELLIPSIZE_END, NULL); g_object_set (G_OBJECT (renderer_subject), - "ellipsize", PANGO_ELLIPSIZE_END, + "ellipsize", PANGO_ELLIPSIZE_END, "yalign", 1.0, NULL); g_object_set (G_OBJECT (renderer_recpt), - "ellipsize", PANGO_ELLIPSIZE_END, + "ellipsize", PANGO_ELLIPSIZE_END, "yalign", 0.0, NULL); g_object_set(G_OBJECT(renderer_compact_date_or_status), - "xalign", 1.0, + "xalign", 1.0, "yalign", 0.0, NULL); - - gtk_cell_renderer_set_fixed_size (renderer_attach, 32, 32); - gtk_cell_renderer_set_fixed_size (renderer_priority, 32, 32); + g_object_set (G_OBJECT (renderer_priority), + "yalign", 1.0, NULL); + g_object_set (G_OBJECT (renderer_attach), + "yalign", 0.0, NULL); + + gtk_cell_renderer_set_fixed_size (renderer_attach, 32, 26); + gtk_cell_renderer_set_fixed_size (renderer_priority, 32, 26); + gtk_cell_renderer_set_fixed_size (renderer_compact_header, -1, 64); remove_all_columns (self); selection = gtk_tree_view_get_selection(GTK_TREE_VIEW(self)); gtk_tree_selection_set_mode(selection, GTK_SELECTION_MULTIPLE); -/* sortable = gtk_tree_view_get_model (GTK_TREE_VIEW (self)); */ tree_filter = gtk_tree_view_get_model (GTK_TREE_VIEW (self)); sortable = gtk_tree_model_filter_get_model (GTK_TREE_MODEL_FILTER(tree_filter)); @@ -494,9 +510,13 @@ modest_header_view_init (ModestHeaderView *obj) priv->monitor = NULL; priv->observers_lock = g_mutex_new (); + priv->empty = TRUE; + priv->clipboard = modest_runtime_get_email_clipboard (); priv->hidding_ids = NULL; priv->n_selected = 0; + priv->selection_changed_handler = 0; + priv->acc_removed_handler = 0; /* Sort parameters */ for (j=0; j < 2; j++) { @@ -514,16 +534,26 @@ modest_header_view_dispose (GObject *obj) { ModestHeaderView *self; ModestHeaderViewPrivate *priv; + GtkTreeSelection *sel; self = MODEST_HEADER_VIEW(obj); priv = MODEST_HEADER_VIEW_GET_PRIVATE(self); + /* Free in the dispose to avoid unref cycles */ if (priv->folder) { tny_folder_remove_observer (priv->folder, TNY_FOLDER_OBSERVER (obj)); g_object_unref (G_OBJECT (priv->folder)); priv->folder = NULL; } + /* We need to do this here in the dispose because the + selection won't exist when finalizing */ + sel = gtk_tree_view_get_selection (GTK_TREE_VIEW(self)); + if (sel && g_signal_handler_is_connected (sel, priv->selection_changed_handler)) { + g_signal_handler_disconnect (sel, priv->selection_changed_handler); + priv->selection_changed_handler = 0; + } + G_OBJECT_CLASS(parent_class)->dispose (obj); } @@ -536,6 +566,12 @@ modest_header_view_finalize (GObject *obj) self = MODEST_HEADER_VIEW(obj); priv = MODEST_HEADER_VIEW_GET_PRIVATE(self); + if (g_signal_handler_is_connected (modest_runtime_get_account_store (), + priv->acc_removed_handler)) { + g_signal_handler_disconnect (modest_runtime_get_account_store (), + priv->acc_removed_handler); + } + g_mutex_lock (priv->observers_lock); if (priv->monitor) { tny_folder_monitor_stop (priv->monitor); @@ -575,10 +611,11 @@ modest_header_view_new (TnyFolder *folder, ModestHeaderViewStyle style) gtk_tree_view_set_rules_hint (GTK_TREE_VIEW(obj), TRUE); /* alternating row colors */ - sel = gtk_tree_view_get_selection(GTK_TREE_VIEW(self)); - - g_signal_connect (sel, "changed", - G_CALLBACK(on_selection_changed), self); + + sel = gtk_tree_view_get_selection(GTK_TREE_VIEW(self)); + priv->selection_changed_handler = + g_signal_connect_after (sel, "changed", + G_CALLBACK(on_selection_changed), self); g_signal_connect (self, "row-activated", G_CALLBACK (on_header_row_activated), NULL); @@ -586,6 +623,11 @@ modest_header_view_new (TnyFolder *folder, ModestHeaderViewStyle style) g_signal_connect (self, "focus-in-event", G_CALLBACK(on_focus_in), NULL); + priv->acc_removed_handler = g_signal_connect (modest_runtime_get_account_store (), + "account_removed", + G_CALLBACK (on_account_removed), + self); + return GTK_WIDGET(self); } @@ -757,12 +799,12 @@ modest_header_view_get_columns (ModestHeaderView *self) } -gboolean -modest_header_view_is_empty (ModestHeaderView *self) -{ - g_return_val_if_fail (self, FALSE); - return FALSE; /* FIXME */ -} +/* gboolean */ +/* modest_header_view_is_empty (ModestHeaderView *self) */ +/* { */ +/* g_return_val_if_fail (self, FALSE); */ +/* return FALSE; /\* FIXME *\/ */ +/* } */ gboolean @@ -990,8 +1032,8 @@ typedef struct { } SetFolderHelper; static void -folder_refreshed_cb (const GObject *obj, - TnyFolder *folder, +folder_refreshed_cb (ModestMailOperation *mail_op, + TnyFolder *folder, gpointer user_data) { ModestHeaderViewPrivate *priv; @@ -1003,7 +1045,7 @@ folder_refreshed_cb (const GObject *obj, /* User callback */ if (info->cb) - info->cb (obj, folder, info->user_data); + info->cb (mail_op, folder, info->user_data); /* Start the folder count changes observer. We do not need it before the refresh. Note that the monitor could still be @@ -1040,6 +1082,7 @@ modest_header_view_set_folder (ModestHeaderView *self, if (folder) { ModestMailOperation *mail_op = NULL; + GtkTreeSelection *selection; /* Get main window to use it as source of mail operation */ mgr = modest_runtime_get_window_mgr (); @@ -1059,13 +1102,17 @@ modest_header_view_set_folder (ModestHeaderView *self, info->cb = callback; info->user_data = user_data; + /* bug 57631: Clear the selection if exists */ + selection = gtk_tree_view_get_selection(GTK_TREE_VIEW(self)); + gtk_tree_selection_unselect_all(selection); + /* Create the mail operation (source will be the parent widget) */ mail_op = modest_mail_operation_new (MODEST_MAIL_OPERATION_TYPE_RECEIVE, source); modest_mail_operation_queue_add (modest_runtime_get_mail_operation_queue (), mail_op); /* Refresh the folder asynchronously */ - modest_mail_operation_refresh_folder (mail_op, + modest_mail_operation_refresh_folder (mail_op, folder, folder_refreshed_cb, info); @@ -1132,7 +1179,7 @@ static void on_selection_changed (GtkTreeSelection *sel, gpointer user_data) { GtkTreeModel *model; - TnyHeader *header; + TnyHeader *header = NULL; GtkTreePath *path = NULL; GtkTreeIter iter; ModestHeaderView *self; @@ -1271,6 +1318,7 @@ drag_data_get_cb (GtkWidget *widget, GdkDragContext *context, GtkTreeModel *model = NULL; GtkTreeIter iter; GtkTreePath *source_row = NULL; + GtkTreeSelection *sel = NULL; source_row = get_selected_row (GTK_TREE_VIEW (widget), &model); if ((source_row == NULL) || (!gtk_tree_model_get_iter(model, &iter, source_row))) return; @@ -1280,7 +1328,7 @@ drag_data_get_cb (GtkWidget *widget, GdkDragContext *context, gtk_tree_set_row_drag_data (selection_data, model, source_row); break; case MODEST_MSG: { - TnyHeader *hdr; + TnyHeader *hdr = NULL; gtk_tree_model_get (model, &iter, TNY_GTK_HEADER_LIST_MODEL_INSTANCE_COLUMN, &hdr, -1); @@ -1293,6 +1341,11 @@ drag_data_get_cb (GtkWidget *widget, GdkDragContext *context, g_message ("%s: default switch case.", __FUNCTION__); } + /* Set focus on next header */ + sel = gtk_tree_view_get_selection(GTK_TREE_VIEW (widget)); + gtk_tree_path_next (source_row); + gtk_tree_selection_select_path (sel, source_row); + gtk_tree_path_free (source_row); } @@ -1459,7 +1512,7 @@ static void folder_monitor_update (TnyFolderObserver *self, TnyFolderChange *change) { - ModestHeaderViewPrivate *priv; + ModestHeaderViewPrivate *priv = NULL; TnyFolderChangeChanged changed; HeadersCountChangedHelper *helper = NULL; @@ -1486,6 +1539,16 @@ folder_monitor_update (TnyFolderObserver *self, } } +gboolean +modest_header_view_is_empty (ModestHeaderView *self) +{ + ModestHeaderViewPrivate *priv = NULL; + + priv = MODEST_HEADER_VIEW_GET_PRIVATE (MODEST_HEADER_VIEW (self)); + + return priv->empty; +} + void modest_header_view_clear (ModestHeaderView *self) { @@ -1599,8 +1662,10 @@ filter_row (GtkTreeModel *model, visible = !found; } - /* Free */ frees: + priv->empty = priv->empty && !visible; + + /* Free */ if (header) g_object_unref (header); g_free(id); @@ -1611,7 +1676,7 @@ filter_row (GtkTreeModel *model, static void _clear_hidding_filter (ModestHeaderView *header_view) { - ModestHeaderViewPrivate *priv; + ModestHeaderViewPrivate *priv = NULL; guint i; g_return_if_fail (MODEST_IS_HEADER_VIEW (header_view)); @@ -1627,12 +1692,45 @@ _clear_hidding_filter (ModestHeaderView *header_view) void modest_header_view_refilter (ModestHeaderView *header_view) { - GtkTreeModel *model; + GtkTreeModel *model = NULL; + ModestHeaderViewPrivate *priv = NULL; g_return_if_fail (MODEST_IS_HEADER_VIEW (header_view)); + priv = MODEST_HEADER_VIEW_GET_PRIVATE(header_view); + + priv->empty = TRUE; /* Hide cut headers */ model = gtk_tree_view_get_model (GTK_TREE_VIEW (header_view)); if (GTK_IS_TREE_MODEL_FILTER (model)) gtk_tree_model_filter_refilter (GTK_TREE_MODEL_FILTER (model)); } + +/* + * Called when an account is removed. If I'm showing a folder of the + * account that has been removed then clear the view + */ +static void +on_account_removed (TnyAccountStore *self, + TnyAccount *account, + gpointer user_data) +{ + ModestHeaderViewPrivate *priv = NULL; + + /* Ignore changes in transport accounts */ + if (TNY_IS_TRANSPORT_ACCOUNT (account)) + return; + + g_print ("--------------------- HEADER ---------------\n"); + + priv = MODEST_HEADER_VIEW_GET_PRIVATE (user_data); + + if (priv->folder) { + TnyAccount *my_account; + + my_account = tny_folder_get_account (priv->folder); + if (my_account == account) + modest_header_view_clear (MODEST_HEADER_VIEW (user_data)); + g_object_unref (account); + } +}