[gnome-music/wip/jfelder/tag-test: 18/22] remove coverstack button, align location label properly

[Jean Felder <jfelder src gnome org>]

commit 696b942da827a8cea062a41d30d02ba946193050
Author: Sumaid Syed <sumaidsyed gmail com>
Date:   Mon Jul 29 17:50:02 2019 +0530

    remove coverstack button, align location label properly

 data/ui/TagEditorDialog.ui | 110 +++++++++++++++++----------------------------
 1 file changed, 41 insertions(+), 69 deletions(-)
---
diff --git a/data/ui/TagEditorDialog.ui b/data/ui/TagEditorDialog.ui
index a5423f11..2dfdc4cb 100644
--- a/data/ui/TagEditorDialog.ui
+++ b/data/ui/TagEditorDialog.ui
@@ -25,7 +25,7 @@
             <child type="overlay">
               <object class="NotificationsPopup" id="_notifications_popup">
                 <property name="halign">center</property>
-                <property name="transition-type">slide-down</property>
+                <property name="transition_type">slide-down</property>
                 <property name="valign">start</property>
               </object>
             </child>
@@ -35,6 +35,7 @@
                 <property name="can_focus">False</property>
                 <property name="halign">start</property>
                 <property name="margin">12</property>
+                <property name="row_spacing">5</property>
                 <child>
                   <object class="GtkBox">
                     <property name="visible">True</property>
@@ -43,16 +44,12 @@
                     <property name="vexpand">True</property>
                     <property name="valign">fill</property>
                     <child>
-                      <object class="GtkButton" id="_coverart_button">
+                      <object class="GtkBox" id="coverart_box">
                         <property name="visible">True</property>
                         <property name="can_focus">False</property>
-                        <property name="focus_on_click">False</property>
-                        <property name="receives_default">False</property>
                         <property name="halign">start</property>
                         <property name="valign">fill</property>
-                        <property name="relief">none</property>
-                        <property name="margin-right">20</property>
-                        <signal name="clicked" handler="_choose_cover" swapped="no"/>
+                        <property name="margin_right">20</property>
                         <child>
                           <object class="CoverStack" id="_cover_stack">
                             <property name="visible">True</property>
@@ -76,62 +73,38 @@
                   </packing>
                 </child>
                 <child>
-                  <object class="GtkBox">
+                  <object class="GtkLabel" id="location">
                     <property name="visible">True</property>
                     <property name="can_focus">False</property>
-                    <property name="orientation">vertical</property>
-                    <property name="spacing">6</property>
-                    <property name="margin-top">12</property>
-                    <property name="margin-left">10</property>
+                    <property name="justify">right</property>
+                    <property name="margin_right">20</property>
+                    <property name="label" translatable="yes">Location</property>
+                    <property name="xalign">1</property>
+                    <attributes>
+                      <attribute name="weight" value="bold"/>
+                    </attributes>
+                    <style>
+                      <class name="dim-label"/>
+                    </style>
+                  </object>
+                  <packing>
+                    <property name="left_attach">0</property>
+                    <property name="top_attach">4</property>
+                  </packing>
+                </child>
+                <child>
+                  <object class="GtkLabel" id="_url">
+                    <property name="can_focus">False</property>
                     <property name="halign">start</property>
-                    <child>
-                      <object class="GtkBox">
-                        <property name="visible">True</property>
-                        <property name="can_focus">False</property>
-                        <child>
-                          <object class="GtkLabel" id="location">
-                            <property name="visible">True</property>
-                            <property name="can_focus">False</property>
-                            <property name="justify">right</property>
-                            <property name="margin-right">40</property>
-                            <property name="label" translatable="yes">Location</property>
-                            <property name="xalign">1</property>
-                            <attributes>
-                              <attribute name="weight" value="bold"/>
-                            </attributes>
-                            <style>
-                              <class name="dim-label"/>
-                            </style>
-                          </object>
-                          <packing>
-                            <property name="expand">False</property>
-                            <property name="fill">True</property>
-                            <property name="position">0</property>
-                          </packing>
-                        </child>
-                        <child>
-                          <object class="GtkLabel" id="_url">
-                            <property name="can_focus">False</property>
-                            <property name="halign">start</property>
-                            <property name="max_width_chars">60</property>
-                            <property name="ellipsize">end</property>
-                            <style>
-                              <class name="dim-label"/>
-                            </style>
-                          </object>
-                          <packing>
-                            <property name="expand">False</property>
-                            <property name="fill">True</property>
-                            <property name="pack_type">end</property>
-                          </packing>
-                        </child>
-                      </object>
-                    </child>
+                    <property name="max_width_chars">40</property>
+                    <property name="ellipsize">end</property>
+                    <style>
+                      <class name="dim-label"/>
+                    </style>
                   </object>
                   <packing>
-                    <property name="left-attach">0</property>
-                    <property name="top-attach">4</property>
-                    <property name="width">2</property>
+                    <property name="left_attach">1</property>
+                    <property name="top_attach">4</property>
                   </packing>
                 </child>
                 <child>
@@ -142,7 +115,6 @@
                     <property name="spacing">12</property>
                     <child>
                       <object class="GtkBox">
-                        <property name="width_request">260</property>
                         <property name="visible">True</property>
                         <property name="can_focus">False</property>
                         <property name="orientation">vertical</property>
@@ -176,7 +148,7 @@
                                 <property name="can_focus">False</property>
                                 <property name="halign">start</property>
                                 <property name="ellipsize">end</property>
-                                <property name="max-width-chars">20</property>
+                                <property name="max_width_chars">20</property>
                                 <style>
                                   <class name="suggestion-label"/>
                                 </style>
@@ -238,7 +210,7 @@
                                 <property name="can_focus">False</property>
                                 <property name="halign">start</property>
                                 <property name="ellipsize">end</property>
-                                <property name="max-width-chars">20</property>
+                                <property name="max_width_chars">20</property>
                                 <style>
                                   <class name="suggestion-label"/>
                                 </style>
@@ -300,7 +272,7 @@
                                 <property name="can_focus">False</property>
                                 <property name="halign">start</property>
                                 <property name="ellipsize">end</property>
-                                <property name="max-width-chars">20</property>
+                                <property name="max_width_chars">20</property>
                                 <style>
                                   <class name="suggestion-label"/>
                                 </style>
@@ -534,8 +506,8 @@
           <object class="GtkBox">
             <property name="visible">True</property>
             <property name="can_focus">False</property>
-            <property name="margin-bottom">20</property>
-            <property name="margin-left">20</property>
+            <property name="margin_bottom">20</property>
+            <property name="margin_left">20</property>
             <child>
               <object class="GtkBox">
                 <property name="visible">True</property>
@@ -575,7 +547,7 @@
                 <property name="halign">end</property>
                 <property name="hexpand">True</property>
                 <property name="spacing">6</property>
-                <property name="margin-right">10</property>
+                <property name="margin_right">10</property>
                 <child>
                   <object class="GtkBox">
                     <property name="can_focus">False</property>
@@ -649,8 +621,8 @@
                           <object class="GtkImage" id="_prev_button_image">
                             <property name="visible">True</property>
                             <property name="can_focus">False</property>
-                            <property name="icon-name">go-previous-symbolic</property>
-                            <property name="icon-size">1</property>
+                            <property name="icon_name">go-previous-symbolic</property>
+                            <property name="icon_size">1</property>
                           </object>
                         </child>
                       </object>
@@ -670,8 +642,8 @@
                           <object class="GtkImage" id="_next_button_image">
                             <property name="visible">True</property>
                             <property name="can_focus">False</property>
-                            <property name="icon-name">go-next-symbolic</property>
-                            <property name="icon-size">1</property>
+                            <property name="icon_name">go-next-symbolic</property>
+                            <property name="icon_size">1</property>
                           </object>
                         </child>
                       </object>