[empathy] accounts-dialog: Add various missing UI spacing
- From: Daniel Boles <dboles src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [empathy] accounts-dialog: Add various missing UI spacing
- Date: Thu, 1 Dec 2016 01:16:49 +0000 (UTC)
commit 5ad66def030024d3bb826916fa9a1a3d10fb02dc
Author: Daniel Boles <dboles src gnome org>
Date: Thu Dec 1 00:47:15 2016 +0000
accounts-dialog: Add various missing UI spacing
* 12 px border around content stops action area's buttons hitting frame
* 12 px border around details does the same for the Edit button there
* 18 px margin for better separation between details and action area
* Remove a no-op VBox
My guess is the whole thing needs a redesign, but for now, this is a
freebie that I think makes it look considerably less clumsy than before.
src/empathy-accounts-dialog.c | 4 +-
src/empathy-accounts-dialog.ui | 50 +++++++++++++++------------------------
2 files changed, 21 insertions(+), 33 deletions(-)
---
diff --git a/src/empathy-accounts-dialog.c b/src/empathy-accounts-dialog.c
index 21b3fd6..feaab2e 100644
--- a/src/empathy-accounts-dialog.c
+++ b/src/empathy-accounts-dialog.c
@@ -2399,10 +2399,9 @@ accounts_dialog_build_ui (EmpathyAccountsDialog *dialog)
/* Tweak the dialog */
gtk_window_set_title (GTK_WINDOW (dialog), _("Messaging and VoIP Accounts"));
gtk_window_set_role (GTK_WINDOW (dialog), "accounts");
-
gtk_window_set_default_size (GTK_WINDOW (dialog), 640, 450);
-
gtk_window_set_type_hint (GTK_WINDOW (dialog), GDK_WINDOW_TYPE_HINT_DIALOG);
+ gtk_container_set_border_width (GTK_CONTAINER (dialog), 12);
/* join the add/remove toolbar to the treeview */
context = gtk_widget_get_style_context (sw);
@@ -2412,6 +2411,7 @@ accounts_dialog_build_ui (EmpathyAccountsDialog *dialog)
gtk_style_context_set_junction_sides (context, GTK_JUNCTION_TOP);
/* add dialog buttons */
+ gtk_widget_set_margin_top (action_area, 18);
gtk_button_box_set_layout (GTK_BUTTON_BOX (action_area), GTK_BUTTONBOX_END);
gtk_dialog_add_buttons (GTK_DIALOG (dialog),
diff --git a/src/empathy-accounts-dialog.ui b/src/empathy-accounts-dialog.ui
index 777187b..674bd80 100644
--- a/src/empathy-accounts-dialog.ui
+++ b/src/empathy-accounts-dialog.ui
@@ -4,7 +4,6 @@
<!-- interface-naming-policy toplevel-contextual -->
<object class="GtkHBox" id="accounts_dialog_hbox">
<property name="visible">True</property>
- <property name="border_width">6</property>
<property name="spacing">18</property>
<child>
<object class="GtkVBox" id="vbox195">
@@ -108,46 +107,35 @@
<property name="can_focus">True</property>
<property name="show_tabs">False</property>
<child>
- <object class="GtkVBox" id="vbox214">
+ <object class="GtkVBox" id="vbox_details">
<property name="visible">True</property>
- <property name="spacing">18</property>
+ <property name="can_focus">False</property>
+ <child>
+ <placeholder/>
+ </child>
<child>
- <object class="GtkVBox" id="vbox_details">
+ <object class="GtkAlignment" id="alignment_infobar">
<property name="visible">True</property>
- <property name="can_focus">False</property>
- <property name="spacing">6</property>
<child>
<placeholder/>
</child>
+ </object>
+ <packing>
+ <property name="expand">False</property>
+ <property name="position">1</property>
+ </packing>
+ </child>
+ <child>
+ <object class="GtkAlignment" id="alignment_settings">
+ <property name="visible">True</property>
+ <property name="can_focus">False</property>
+ <property name="margin">12</property>
<child>
- <object class="GtkAlignment" id="alignment_infobar">
- <property name="visible">True</property>
- <child>
- <placeholder/>
- </child>
- </object>
- <packing>
- <property name="expand">False</property>
- <property name="position">1</property>
- </packing>
- </child>
- <child>
- <object class="GtkAlignment" id="alignment_settings">
- <property name="visible">True</property>
- <property name="can_focus">False</property>
- <property name="left_padding">6</property>
- <property name="right_padding">6</property>
- <child>
- <placeholder/>
- </child>
- </object>
- <packing>
- <property name="position">2</property>
- </packing>
+ <placeholder/>
</child>
</object>
<packing>
- <property name="position">1</property>
+ <property name="position">2</property>
</packing>
</child>
</object>
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]