X-Git-Url: http://git.maemo.org/git/?p=modest;a=blobdiff_plain;f=src%2Fhildon2%2Fmodest-hildon2-window-mgr.c;h=3922dd6055a07647ac347ba122cccf456d7525f6;hp=5b946e1c0c6e601a836fd8f7af068fa72e63a90a;hb=fd1d9e254d08ad38779f8a1bb5c4d6d350040a9b;hpb=b28dbbbd1e18bb6e09c6a9ebbbfcb238e104be67;ds=sidebyside diff --git a/src/hildon2/modest-hildon2-window-mgr.c b/src/hildon2/modest-hildon2-window-mgr.c index 5b946e1..3922dd6 100644 --- a/src/hildon2/modest-hildon2-window-mgr.c +++ b/src/hildon2/modest-hildon2-window-mgr.c @@ -48,6 +48,7 @@ #include "modest-maemo-utils.h" #include "modest-utils.h" #include "modest-tny-msg.h" +#include "modest-tny-account.h" #include /* 'private'/'protected' functions */ @@ -84,11 +85,11 @@ static gboolean modest_hildon2_window_mgr_find_registered_message_uid (ModestWin ModestWindow **win); static GList *modest_hildon2_window_mgr_get_window_list (ModestWindowMgr *self); static gboolean modest_hildon2_window_mgr_close_all_windows (ModestWindowMgr *self); -static gboolean window_can_close (ModestWindow *window); static gboolean window_has_modals (ModestWindow *window); static ModestWindow *modest_hildon2_window_mgr_show_initial_window (ModestWindowMgr *self); static ModestWindow *modest_hildon2_window_mgr_get_current_top (ModestWindowMgr *self); static gboolean modest_hildon2_window_mgr_screen_is_on (ModestWindowMgr *self); +static void modest_hildon2_window_mgr_create_caches (ModestWindowMgr *self); static void osso_display_event_cb (osso_display_state_t state, gpointer data); static void on_account_removed (TnyAccountStore *acc_store, @@ -173,6 +174,7 @@ modest_hildon2_window_mgr_class_init (ModestHildon2WindowMgrClass *klass) mgr_class->show_initial_window = modest_hildon2_window_mgr_show_initial_window; mgr_class->get_current_top = modest_hildon2_window_mgr_get_current_top; mgr_class->screen_is_on = modest_hildon2_window_mgr_screen_is_on; + mgr_class->create_caches = modest_hildon2_window_mgr_create_caches; g_type_class_add_private (gobject_class, sizeof(ModestHildon2WindowMgrPrivate)); @@ -443,6 +445,15 @@ modest_hildon2_window_mgr_register_window (ModestWindowMgr *self, while (modal && GTK_IS_DIALOG (modal)) { GtkWidget *parent; + /* If it's a hildon note then don't try to close it as + this is the default behaviour of WM, delete event + is not issued for this kind of notes as we want the + user to always click on a button */ + if (HILDON_IS_NOTE (modal)) { + gtk_window_present (GTK_WINDOW (modal)); + return FALSE; + } + /* Get the parent */ parent = (GtkWidget *) gtk_window_get_transient_for (GTK_WINDOW (modal)); @@ -481,29 +492,39 @@ modest_hildon2_window_mgr_register_window (ModestWindowMgr *self, if (MODEST_IS_MSG_VIEW_WINDOW (window)) { gchar *uid; TnyHeader *header; + + uid = g_strdup (modest_msg_view_window_get_message_uid (MODEST_MSG_VIEW_WINDOW (window))); + header = modest_msg_view_window_get_header (MODEST_MSG_VIEW_WINDOW (window)); - if (header) { + if (uid == NULL) uid = modest_tny_folder_get_header_unique_id (header); - - /* Embedded messages do not have uid */ - if (uid) { - if (g_list_find_custom (priv->window_list, uid, (GCompareFunc) compare_msguids)) { - g_debug ("%s found another view window showing the same header", __FUNCTION__); - g_free (uid); - g_object_unref (header); - return FALSE; - } + /* Embedded messages do not have uid */ + if (uid) { + if (g_list_find_custom (priv->window_list, uid, (GCompareFunc) compare_msguids)) { + g_debug ("%s found another view window showing the same header", __FUNCTION__); g_free (uid); - } else { - if (g_list_find_custom (priv->window_list, header, (GCompareFunc) compare_headers)) { - g_debug ("%s found another view window showing the same header", __FUNCTION__); - g_object_unref (header); - return FALSE; - } + g_object_unref (header); + return FALSE; + } + g_free (uid); + } else if (header) { + if (g_list_find_custom (priv->window_list, header, (GCompareFunc) compare_headers)) { + g_debug ("%s found another view window showing the same header", __FUNCTION__); + g_object_unref (header); + return FALSE; } - g_object_unref (header); } + if (header) + g_object_unref (header); + } + + /* Do not go backwards */ + if ((MODEST_IS_MSG_VIEW_WINDOW (current_top) || MODEST_IS_MSG_EDIT_WINDOW (current_top)) && + (MODEST_IS_FOLDER_WINDOW (window) || MODEST_IS_ACCOUNTS_WINDOW (window) || + MODEST_IS_MAILBOXES_WINDOW (window))) { + gtk_window_present (GTK_WINDOW (window)); + return FALSE; } if (MODEST_IS_FOLDER_WINDOW (current_top) && MODEST_IS_FOLDER_WINDOW (window)) { @@ -602,7 +623,9 @@ cancel_window_operations (ModestWindow *window) GSList* tmp_list = NULL; type = modest_mail_operation_get_type_operation (MODEST_MAIL_OPERATION (pending_ops->data)); - if (type == MODEST_MAIL_OPERATION_TYPE_RECEIVE || type == MODEST_MAIL_OPERATION_TYPE_OPEN) { + if (type == MODEST_MAIL_OPERATION_TYPE_RECEIVE || + type == MODEST_MAIL_OPERATION_TYPE_OPEN || + type == MODEST_MAIL_OPERATION_TYPE_SEND_AND_RECEIVE) { modest_mail_operation_cancel (pending_ops->data); } g_object_unref (G_OBJECT (pending_ops->data)); @@ -634,24 +657,14 @@ window_has_modals (ModestWindow *window) } static gboolean -window_can_close (ModestWindow *window) -{ - /* An editor can be always closed no matter the dialogs it has - * on top. */ - if (MODEST_IS_MSG_EDIT_WINDOW (window)) - return TRUE; - - return !window_has_modals (window); -} - -static gboolean on_window_destroy (ModestWindow *window, GdkEvent *event, ModestHildon2WindowMgr *self) { gboolean no_propagate = FALSE; - if (!window_can_close (window)) + /* Do not close the window if it has modals on top */ + if (!MODEST_IS_MSG_EDIT_WINDOW (window) && window_has_modals (window)) return TRUE; if (MODEST_IS_MSG_EDIT_WINDOW (window)) { @@ -719,7 +732,7 @@ modest_hildon2_window_mgr_unregister_window (ModestWindowMgr *self, win = g_list_find (priv->window_list, window); if (!win) { - g_warning ("Trying to unregister a window that has not being registered yet"); + g_debug ("Trying to unregister a window that has not being registered yet"); return; } @@ -833,33 +846,59 @@ modest_hildon2_window_mgr_get_main_window (ModestWindowMgr *self, gboolean show) gtk_widget_show_all (GTK_WIDGET (result)); gtk_window_present (GTK_WINDOW (result)); } - + return result; } +static gint +look_for_transient (gconstpointer a, + gconstpointer b) +{ + GtkWindow *win, *child; + + if (a == b) + return 1; + + child = (GtkWindow *) b; + win = (GtkWindow *) a; + + if (gtk_window_get_transient_for (win) == child) + return 0; + else + return 1; +} static GtkWindow * modest_hildon2_window_mgr_get_modal (ModestWindowMgr *self) { ModestHildon2WindowMgrPrivate *priv; GList *toplevel_list; - GtkWindow *toplevel; + GtkWidget *current_top, *toplevel; + HildonWindowStack *stack; g_return_val_if_fail (MODEST_IS_HILDON2_WINDOW_MGR (self), NULL); priv = MODEST_HILDON2_WINDOW_MGR_GET_PRIVATE (self); - toplevel = NULL; + /* Get current top */ + stack = hildon_window_stack_get_default (); + current_top = hildon_window_stack_peek (stack); + toplevel = current_top; toplevel_list = gtk_window_list_toplevels (); - while (toplevel_list) { - if (gtk_window_is_active (toplevel_list->data) && - gtk_window_get_modal (toplevel_list->data)) { - toplevel = (GtkWindow *) toplevel_list->data; + + while (toplevel) { + GList *parent_link; + + parent_link = g_list_find_custom (toplevel_list, toplevel, look_for_transient); + if (parent_link) + toplevel = (GtkWidget *) parent_link->data; + else break; - } - toplevel_list = g_list_next (toplevel_list); } - return toplevel; + if (toplevel && GTK_WIDGET_VISIBLE (toplevel) && gtk_window_get_modal ((GtkWindow *) toplevel)) + return (GtkWindow *) toplevel; + else + return NULL; } @@ -876,11 +915,15 @@ modest_hildon2_window_mgr_set_modal (ModestWindowMgr *self, gtk_window_set_destroy_with_parent (window, TRUE); } -static void -close_all_but_first (HildonWindowStack *stack) +static gboolean +on_idle_close_all_but_first (gpointer data) { gint num_windows, i; gboolean retval; + HildonWindowStack *stack; + + stack = hildon_window_stack_get_default (); + g_return_val_if_fail (stack, FALSE); num_windows = hildon_window_stack_size (stack); @@ -891,16 +934,16 @@ close_all_but_first (HildonWindowStack *stack) current_top = hildon_window_stack_peek (stack); g_signal_emit_by_name (G_OBJECT (current_top), "delete-event", NULL, &retval); } + return FALSE; } static void -on_account_removed (TnyAccountStore *acc_store, +on_account_removed (TnyAccountStore *acc_store, TnyAccount *account, gpointer user_data) { HildonWindowStack *stack; ModestWindow *current_top; - gboolean has_accounts; /* Ignore transport account removals */ if (TNY_IS_TRANSPORT_ACCOUNT (account)) @@ -908,45 +951,24 @@ on_account_removed (TnyAccountStore *acc_store, stack = hildon_window_stack_get_default (); current_top = (ModestWindow *) hildon_window_stack_peek (stack); - has_accounts = modest_account_mgr_has_accounts (modest_runtime_get_account_mgr (), TRUE); /* if we're showing the header view of the currently deleted account, or the outbox and we deleted the last account, then close the window */ - if (current_top && MODEST_IS_HEADER_WINDOW (current_top)) { - ModestHeaderView *header_view; - TnyFolder *folder; - gboolean deleted = FALSE; - - header_view = modest_header_window_get_header_view (MODEST_HEADER_WINDOW (current_top)); - folder = modest_header_view_get_folder (header_view); - if (folder) { - /* Close if it's my account */ - if (!TNY_IS_MERGE_FOLDER (folder)) { - TnyAccount *my_account; - - my_account = tny_folder_get_account (folder); - if (my_account) { - if (my_account == account) { - close_all_but_first (stack); - deleted = TRUE; - } - g_object_unref (my_account); - } - } - - /* Close if viewing outbox and no account left */ - if (tny_folder_get_folder_type (folder) == TNY_FOLDER_TYPE_OUTBOX) { - if (!has_accounts) { - close_all_but_first (stack); - deleted = TRUE; - } - } - g_object_unref (folder); - - if (deleted) - current_top = (ModestWindow *) hildon_window_stack_peek (stack); - } + if (current_top && + (MODEST_IS_HEADER_WINDOW (current_top) || + MODEST_IS_FOLDER_WINDOW (current_top))) { + const gchar *acc_name; + + acc_name = modest_tny_account_get_parent_modest_account_name_for_server_account (account); + + /* We emit it in an idle, because sometimes this + function could called when the account settings + dialog is about to close but still there. That + modal dialog would otherwise, prevent the + windows from being closed */ + if (!strcmp (acc_name, modest_window_get_active_account (current_top))) + g_idle_add (on_idle_close_all_but_first, NULL); } } @@ -1021,6 +1043,17 @@ modest_hildon2_window_mgr_screen_is_on (ModestWindowMgr *self) return (priv->display_state == OSSO_DISPLAY_ON) ? TRUE : FALSE; } +static void +modest_hildon2_window_mgr_create_caches (ModestWindowMgr *self) +{ + g_return_if_fail (MODEST_IS_HILDON2_WINDOW_MGR (self)); + + modest_accounts_window_pre_create (); + + MODEST_WINDOW_MGR_CLASS(parent_class)->create_caches (self); + +} + static void osso_display_event_cb (osso_display_state_t state, gpointer data)