[balsa/gtk4: 56/193] balsa-message: Manage menu settings




commit ce7e0888da2850cff92d91f8b110f631f84274dd
Author: Peter Bloomfield <PeterBloomfield bellsouth net>
Date:   Tue Jun 23 19:25:12 2020 -0400

    balsa-message: Manage menu settings
    
    In the "paper-clip" attachments button, selecting "force inline" should
    clear any selected part button, clearing it should select "complete
    message", and selecting any individual part should clear "force inline".
    
    Ideally, changing the selection in the parts tree-view should also
    update the attachments button menu, but that is challenging!

 src/balsa-message.c | 19 +++++++++++++++++--
 1 file changed, 17 insertions(+), 2 deletions(-)
---
diff --git a/src/balsa-message.c b/src/balsa-message.c
index 97c623c8a..95f028e0f 100644
--- a/src/balsa-message.c
+++ b/src/balsa-message.c
@@ -212,6 +212,7 @@ struct _BalsaMessage {
 
         GtkWidget *attach_button;
         GAction   *toggle_all_inline_action;
+        GAction   *show_part_action;
 };
 
 G_DEFINE_TYPE(BalsaMessage, balsa_message, GTK_TYPE_BOX)
@@ -784,6 +785,11 @@ toggle_all_inline_change_state(GSimpleAction *action,
 
     balsa_message->force_inline = g_variant_get_boolean(parameter);
 
+    /* If we are forcing in-line, clear all the parts;
+     * otherwise, set "complete message". */
+    g_simple_action_set_state(G_SIMPLE_ACTION(balsa_message->show_part_action),
+                              g_variant_new_string(balsa_message->force_inline ? "" : "0"));
+
     gtk_stack_set_visible_child_name(GTK_STACK(balsa_message->stack), "content");
 
     model = gtk_tree_view_get_model(GTK_TREE_VIEW(balsa_message->treeview));
@@ -812,6 +818,10 @@ show_part_change_state(GSimpleAction *action,
     GtkTreeIter iter;
     BalsaPartInfo *info;
 
+    /* Make sure force-inline is cleared */
+    g_simple_action_set_state(G_SIMPLE_ACTION(balsa_message->toggle_all_inline_action),
+                              g_variant_new_boolean(FALSE));
+
     gtk_stack_set_visible_child_name(GTK_STACK(balsa_message->stack), "content");
 
     model = gtk_tree_view_get_model(GTK_TREE_VIEW(balsa_message->treeview));
@@ -860,6 +870,8 @@ balsa_message_init(BalsaMessage * balsa_message)
                                     balsa_message);
     balsa_message->toggle_all_inline_action =
         g_action_map_lookup_action(G_ACTION_MAP(simple), "toggle-all-inline");
+    balsa_message->show_part_action =
+        g_action_map_lookup_action(G_ACTION_MAP(simple), "show-part");
 
     gtk_widget_insert_action_group(GTK_WIDGET(balsa_message),
                                    "message-menu",
@@ -1741,9 +1753,11 @@ display_content(BalsaMessage * balsa_message)
     g_clear_object(&balsa_message->parts_menu);
 
     /* GAction retains its state when switching messages, so we must
-     * return the all-inline toggle to FALSE: */
+     * return the all-inline toggle to FALSE and clear any show-part: */
     g_simple_action_set_state(G_SIMPLE_ACTION(balsa_message->toggle_all_inline_action),
                               g_variant_new_boolean(FALSE));
+    g_simple_action_set_state(G_SIMPLE_ACTION(balsa_message->show_part_action),
+                              g_variant_new_string(""));
 
     if (balsa_message->info_count > 1) {
        gtk_widget_show_all(balsa_message->attach_button);
@@ -2227,8 +2241,9 @@ add_multipart_mixed(BalsaMessage * balsa_message, LibBalsaMessageBody * body,
                g_mime_content_type_is_type(type, "application", "pgp-signature") ||
                (balsa_app.has_smime &&
                 (g_mime_content_type_is_type(type, "application", "pkcs7-signature") ||
-                 g_mime_content_type_is_type(type, "application", "x-pkcs7-signature"))))
+                 g_mime_content_type_is_type(type, "application", "x-pkcs7-signature")))) {
                 add_body(balsa_message, body, container);
+            }
            g_object_unref(type);
         }
     }


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]