From 7c2dd3d976388c353632ea0c3199ccdd3e214530 Mon Sep 17 00:00:00 2001 From: Charles Lehner Date: Wed, 15 Jul 2015 22:14:59 -0400 Subject: [PATCH] ManageSieve: fix manager window button spacing - Remove unused vbox - Move refresh button to bottom, since it is semantically different from the other action buttons --- 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 index c061fd083..de76aae31 100644 --- 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); -- 2.25.1