talons

commit 7c2dd3d976388c353632ea0c3199ccdd3e214530
parent 2328d40cf23a5a6c50417bc266e42c5b2b3a90bc
Author: Charles Lehner <cel@celehner.com>
Date:   Wed, 15 Jul 2015 22:14:59 -0400

ManageSieve: fix manager window button spacing

- Remove unused vbox
- Move refresh button to bottom, since it is semantically different
  from the other action buttons

Diffstat:
M
src/plugins/managesieve/sieve_manager.c
 | 
19
++++++++-----------

1 file changed, 8 insertions(+), 11 deletions(-)
diff --git a/src/plugins/managesieve/sieve_manager.c b/src/plugins/managesieve/sieve_manager.c
@@ -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);