From 5a9cdfd1e6d313da2ca7505d1876e91331be2e38 Mon Sep 17 00:00:00 2001
From: Jim Nelson <jim@yorba.org>
Date: Tue, 25 Sep 2012 11:58:46 -0700
Subject: [PATCH] Composer buttons clean-up: Closes #5810, Closes #5818

---
 ui/composer.glade | 60 +++++++++++++++++++++++++++++++----------------
 1 file changed, 40 insertions(+), 20 deletions(-)

diff --git a/ui/composer.glade b/ui/composer.glade
index 255e2594..661a58b0 100644
--- a/ui/composer.glade
+++ b/ui/composer.glade
@@ -576,6 +576,8 @@
         <property name="can_focus">False</property>
         <property name="margin_left">6</property>
         <property name="margin_right">6</property>
+        <property name="margin_top">6</property>
+        <property name="margin_bottom">6</property>
         <property name="orientation">vertical</property>
         <property name="spacing">6</property>
         <child>
@@ -589,12 +591,28 @@
       </packing>
     </child>
     <child>
-      <object class="GtkBox" id="button_area">
+      <object class="GtkSeparator" id="separator">
+        <property name="visible">True</property>
+        <property name="can_focus">False</property>
+        <property name="margin_left">6</property>
+        <property name="margin_right">6</property>
+      </object>
+      <packing>
+        <property name="expand">False</property>
+        <property name="fill">True</property>
+        <property name="padding">8</property>
+        <property name="position">5</property>
+      </packing>
+    </child>
+    <child>
+      <object class="GtkButtonBox" id="button_area">
         <property name="visible">True</property>
         <property name="can_focus">False</property>
         <property name="margin_left">6</property>
         <property name="margin_right">6</property>
         <property name="margin_bottom">6</property>
+        <property name="spacing">8</property>
+        <property name="layout_style">start</property>
         <child>
           <object class="GtkButton" id="add_attachment_button">
             <property name="label" translatable="yes">_Attach a file</property>
@@ -613,24 +631,6 @@
             <property name="position">0</property>
           </packing>
         </child>
-        <child>
-          <object class="GtkButton" id="Send">
-            <property name="label" translatable="yes">_Send</property>
-            <property name="use_action_appearance">False</property>
-            <property name="visible">True</property>
-            <property name="can_focus">True</property>
-            <property name="receives_default">True</property>
-            <property name="use_action_appearance">False</property>
-            <property name="use_underline">True</property>
-          </object>
-          <packing>
-            <property name="expand">False</property>
-            <property name="fill">True</property>
-            <property name="padding">3</property>
-            <property name="pack_type">end</property>
-            <property name="position">1</property>
-          </packing>
-        </child>
         <child>
           <object class="GtkButton" id="Cancel">
             <property name="label">gtk-cancel</property>
@@ -641,19 +641,39 @@
             <property name="use_action_appearance">False</property>
             <property name="use_stock">True</property>
           </object>
+          <packing>
+            <property name="expand">False</property>
+            <property name="fill">True</property>
+            <property name="padding">3</property>
+            <property name="pack_type">end</property>
+            <property name="position">1</property>
+            <property name="secondary">True</property>
+          </packing>
+        </child>
+        <child>
+          <object class="GtkButton" id="Send">
+            <property name="label" translatable="yes">_Send</property>
+            <property name="use_action_appearance">False</property>
+            <property name="visible">True</property>
+            <property name="can_focus">True</property>
+            <property name="receives_default">True</property>
+            <property name="use_action_appearance">False</property>
+            <property name="use_underline">True</property>
+          </object>
           <packing>
             <property name="expand">False</property>
             <property name="fill">True</property>
             <property name="padding">3</property>
             <property name="pack_type">end</property>
             <property name="position">2</property>
+            <property name="secondary">True</property>
           </packing>
         </child>
       </object>
       <packing>
         <property name="expand">False</property>
         <property name="fill">True</property>
-        <property name="position">5</property>
+        <property name="position">6</property>
       </packing>
     </child>
   </object>