[sysprof/wip/chergert/sysprof-3] libsysprof-ui: setup environment editor

[Christian Hergert <chergert src gnome org>]

commit 5004c80d651d169f8b899b76b9b160d680505d34
Author: Christian Hergert <chergert redhat com>
Date:   Sat May 18 13:36:30 2019 -0700

    libsysprof-ui: setup environment editor

 src/libsysprof-ui/sysprof-environ-editor.c         |  1 +
 src/libsysprof-ui/sysprof-profiler-assistant.c     | 14 +++--
 src/libsysprof-ui/ui/sysprof-environ-editor-row.ui | 65 ++++++++++++++++------
 src/libsysprof-ui/ui/sysprof-profiler-assistant.ui |  3 +-
 4 files changed, 60 insertions(+), 23 deletions(-)
---
diff --git a/src/libsysprof-ui/sysprof-environ-editor.c b/src/libsysprof-ui/sysprof-environ-editor.c
index a8538de..8fe2882 100644
--- a/src/libsysprof-ui/sysprof-environ-editor.c
+++ b/src/libsysprof-ui/sysprof-environ-editor.c
@@ -68,6 +68,7 @@ sysprof_environ_editor_create_dummy_row (SysprofEnvironEditor *self)
 
   label = g_object_new (GTK_TYPE_LABEL,
                         "label", _("New variable…"),
+                        "margin", 6,
                         "visible", TRUE,
                         "xalign", 0.0f,
                         NULL);
diff --git a/src/libsysprof-ui/sysprof-profiler-assistant.c b/src/libsysprof-ui/sysprof-profiler-assistant.c
index 5558b24..34ee350 100644
--- a/src/libsysprof-ui/sysprof-profiler-assistant.c
+++ b/src/libsysprof-ui/sysprof-profiler-assistant.c
@@ -30,12 +30,13 @@
 
 struct _SysprofProfilerAssistant
 {
-  GtkBin       parent_instance;
+  GtkBin                parent_instance;
 
   /* Template Objects */
-  GtkEntry    *command_line;
-  GtkRevealer *process_revealer;
-  GtkListBox  *process_list_box;
+  GtkEntry             *command_line;
+  GtkRevealer          *process_revealer;
+  GtkListBox           *process_list_box;
+  SysprofEnvironEditor *environ_editor;
 };
 
 G_DEFINE_TYPE (SysprofProfilerAssistant, sysprof_profiler_assistant, GTK_TYPE_BIN)
@@ -128,6 +129,7 @@ sysprof_profiler_assistant_class_init (SysprofProfilerAssistantClass *klass)
 
   gtk_widget_class_set_template_from_resource (widget_class, 
"/org/gnome/sysprof/ui/sysprof-profiler-assistant.ui");
   gtk_widget_class_bind_template_child (widget_class, SysprofProfilerAssistant, command_line);
+  gtk_widget_class_bind_template_child (widget_class, SysprofProfilerAssistant, environ_editor);
   gtk_widget_class_bind_template_child (widget_class, SysprofProfilerAssistant, process_list_box);
   gtk_widget_class_bind_template_child (widget_class, SysprofProfilerAssistant, process_revealer);
 
@@ -137,6 +139,8 @@ sysprof_profiler_assistant_class_init (SysprofProfilerAssistantClass *klass)
 static void
 sysprof_profiler_assistant_init (SysprofProfilerAssistant *self)
 {
+  g_autoptr(SysprofEnviron) environ = sysprof_environ_new ();
+
   gtk_widget_init_template (GTK_WIDGET (self));
 
   g_signal_connect_object (self->command_line,
@@ -156,4 +160,6 @@ sysprof_profiler_assistant_init (SysprofProfilerAssistant *self)
                            G_CALLBACK (sysprof_profiler_assistant_notify_reveal_child_cb),
                            self,
                            G_CONNECT_SWAPPED);
+
+  sysprof_environ_editor_set_environ (self->environ_editor, environ);
 }
diff --git a/src/libsysprof-ui/ui/sysprof-environ-editor-row.ui 
b/src/libsysprof-ui/ui/sysprof-environ-editor-row.ui
index 56b176a..4467a44 100644
--- a/src/libsysprof-ui/ui/sysprof-environ-editor-row.ui
+++ b/src/libsysprof-ui/ui/sysprof-environ-editor-row.ui
@@ -1,47 +1,78 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<!-- Generated with glade 3.22.0 -->
 <interface>
-  <template class="IdeEnvironmentEditorRow" parent="GtkListBoxRow">
+  <requires lib="gtk+" version="3.10"/>
+  <template class="SysprofEnvironEditorRow" parent="GtkListBoxRow">
+    <property name="can_focus">False</property>
     <child>
       <object class="GtkBox">
-        <property name="orientation">horizontal</property>
+        <property name="visible">True</property>
+        <property name="can_focus">False</property>
         <property name="spacing">12</property>
-        <property name="visible">true</property>
         <child>
           <object class="GtkEntry" id="key_entry">
-            <property name="has-frame">false</property>
-            <property name="visible">true</property>
+            <property name="visible">True</property>
+            <property name="can_focus">True</property>
+            <property name="margin_start">3</property>
+            <property name="has_frame">False</property>
           </object>
+          <packing>
+            <property name="expand">False</property>
+            <property name="fill">True</property>
+            <property name="position">0</property>
+          </packing>
         </child>
         <child>
           <object class="GtkLabel" id="eq_label">
+            <property name="visible">True</property>
+            <property name="can_focus">False</property>
             <property name="label">=</property>
-            <property name="visible">true</property>
             <style>
               <class name="dim-label"/>
             </style>
           </object>
+          <packing>
+            <property name="expand">False</property>
+            <property name="fill">True</property>
+            <property name="position">1</property>
+          </packing>
         </child>
         <child>
           <object class="GtkEntry" id="value_entry">
-            <property name="hexpand">true</property>
-            <property name="has-frame">false</property>
-            <property name="visible">true</property>
+            <property name="visible">True</property>
+            <property name="can_focus">True</property>
+            <property name="hexpand">True</property>
+            <property name="has_frame">False</property>
           </object>
+          <packing>
+            <property name="expand">False</property>
+            <property name="fill">True</property>
+            <property name="position">2</property>
+          </packing>
         </child>
         <child>
           <object class="GtkButton" id="delete_button">
-            <property name="visible">true</property>
-            <property name="tooltip-text" translatable="yes">Remove environment variable</property>
-            <style>
-              <class name="image-button"/>
-              <class name="flat"/>
-            </style>
+            <property name="visible">True</property>
+            <property name="can_focus">False</property>
+            <property name="receives_default">False</property>
+            <property name="tooltip_text" translatable="yes">Remove environment variable</property>
             <child>
               <object class="GtkImage">
-                <property name="icon-name">list-remove-symbolic</property>
-                <property name="visible">true</property>
+                <property name="visible">True</property>
+                <property name="can_focus">False</property>
+                <property name="icon_name">list-remove-symbolic</property>
               </object>
             </child>
+            <style>
+              <class name="image-button"/>
+              <class name="flat"/>
+            </style>
           </object>
+          <packing>
+            <property name="expand">False</property>
+            <property name="fill">True</property>
+            <property name="position">3</property>
+          </packing>
         </child>
       </object>
     </child>
diff --git a/src/libsysprof-ui/ui/sysprof-profiler-assistant.ui 
b/src/libsysprof-ui/ui/sysprof-profiler-assistant.ui
index dd2abc5..f51a512 100644
--- a/src/libsysprof-ui/ui/sysprof-profiler-assistant.ui
+++ b/src/libsysprof-ui/ui/sysprof-profiler-assistant.ui
@@ -205,9 +205,8 @@
                           </object>
                         </child>
                         <child>
-                          <object class="GtkScrolledWindow">
+                          <object class="GtkFrame">
                             <property name="shadow-type">in</property>
-                            <property name="propagate-natural-height">true</property>
                             <property name="visible">true</property>
                             <child>
                               <object class="SysprofEnvironEditor" id="environ_editor">