ManageSieve: fix manager window button spacing
authorCharles Lehner <cel@celehner.com>
Thu, 16 Jul 2015 02:14:59 +0000 (22:14 -0400)
committerCharles Lehner <cel@celehner.com>
Thu, 16 Jul 2015 02:19:39 +0000 (22:19 -0400)
- Remove unused vbox
- Move refresh button to bottom, since it is semantically different
  from the other action buttons

src/plugins/managesieve/sieve_manager.c

index c061fd0..de76aae 100644 (file)
@@ -622,7 +622,7 @@ static SieveManagerPage *sieve_manager_page_new()
 {
        SieveManagerPage *page;
        GtkWidget *window;
-       GtkWidget *hbox, *vbox, *vbox_allbuttons, *vbox_buttons;
+       GtkWidget *hbox, *vbox, *vbox_buttons;
        GtkWidget *accounts_menu;
        GtkWidget *label;
        GtkWidget *scrolledwin;
@@ -721,40 +721,37 @@ static SieveManagerPage *sieve_manager_page_new()
 
        /* Buttons */
 
-       vbox_allbuttons = gtk_vbox_new (FALSE, 0);
-       gtk_box_pack_start (GTK_BOX (hbox), vbox_allbuttons, FALSE, FALSE, 0);
-
-       vbox_buttons = gtk_vbox_new (FALSE, 0);
+       vbox_buttons = gtk_vbox_new (FALSE, 8);
        gtk_widget_set_sensitive(vbox_buttons, FALSE);
-       gtk_box_pack_start (GTK_BOX (vbox_allbuttons), vbox_buttons, FALSE, FALSE, 0);
+       gtk_box_pack_start (GTK_BOX (hbox), vbox_buttons, FALSE, FALSE, 0);
 
        /* new */
        btn = gtk_button_new_from_stock(GTK_STOCK_NEW);
-       gtk_box_pack_start (GTK_BOX (vbox_buttons), btn, FALSE, FALSE, 4);
+       gtk_box_pack_start (GTK_BOX (vbox_buttons), btn, FALSE, FALSE, 0);
        g_signal_connect (G_OBJECT(btn), "clicked",
                          G_CALLBACK (filter_add), page);
 
        /* edit */
        btn = gtk_button_new_from_stock (GTK_STOCK_EDIT);
-       gtk_box_pack_start (GTK_BOX (vbox_buttons), btn, FALSE, FALSE, 4);
+       gtk_box_pack_start (GTK_BOX (vbox_buttons), btn, FALSE, FALSE, 0);
        g_signal_connect (G_OBJECT(btn), "clicked",
                        G_CALLBACK (filter_edit), page);
 
        /* delete */
        btn = gtk_button_new_from_stock(GTK_STOCK_DELETE);
-       gtk_box_pack_start (GTK_BOX (vbox_buttons), btn, FALSE, FALSE, 4);
+       gtk_box_pack_start (GTK_BOX (vbox_buttons), btn, FALSE, FALSE, 0);
        g_signal_connect (G_OBJECT(btn), "clicked",
                        G_CALLBACK (filter_delete), page);
 
        /* rename */
        btn = gtk_button_new_with_label("Rename");
-       gtk_box_pack_start (GTK_BOX (vbox_buttons), btn, FALSE, FALSE, 4);
+       gtk_box_pack_start (GTK_BOX (vbox_buttons), btn, FALSE, FALSE, 0);
        g_signal_connect (G_OBJECT(btn), "clicked",
                        G_CALLBACK (filter_rename), page);
 
        /* refresh */
        btn = gtk_button_new_from_stock(GTK_STOCK_REFRESH);
-       gtk_box_pack_start (GTK_BOX (vbox_allbuttons), btn, FALSE, FALSE, 4);
+       gtk_box_pack_end (GTK_BOX (vbox_buttons), btn, FALSE, FALSE, 0);
        g_signal_connect (G_OBJECT(btn), "clicked",
                        G_CALLBACK (account_changed), page);