[gnome-boxes/wip/preferences-dialog: 7/7] WIP




commit f4dd4e76eb8a01bedc5d7bc8e3ba665d778bb43d
Author: Felipe Borges <felipeborges gnome org>
Date:   Wed Nov 10 16:57:25 2021 +0100

    WIP

 data/gnome-boxes.gresource.xml                 |  14 +-
 data/ui/preferences/devices-page.ui            | 108 ++++++++
 data/ui/preferences/preferences-window.ui      |  24 ++
 data/ui/preferences/resources-page.ui          | 293 ++++++++++++++++++++
 data/ui/preferences/shared-folder-popover.ui   | 140 ++++++++++
 data/ui/preferences/shared-folder-row.ui       |  25 ++
 data/ui/preferences/shared-folders-widget.ui   |  24 ++
 data/ui/{ => preferences}/snapshot-list-row.ui |   0
 data/ui/preferences/snapshots-page.ui          |  59 ++++
 data/ui/properties-shared-folder-row.ui        |  61 -----
 data/ui/properties-toolbar.ui                  |  62 -----
 data/ui/properties-window.ui                   |  10 -
 data/ui/shared-folders.ui                      |  47 ----
 src/actions-popover.vala                       |  16 +-
 src/app-window.vala                            |  23 --
 src/config-editor.vala                         | 105 -------
 src/libvirt-machine-properties.vala            |  23 +-
 src/libvirt-machine.vala                       |  16 +-
 src/machine.vala                               |   2 +-
 src/meson.build                                |   5 +-
 src/preferences/devices-page.vala              |  54 ++++
 src/preferences/meson.build                    |   8 +
 src/preferences/preferences-window.vala        |  20 ++
 src/preferences/resources-page.vala            | 365 +++++++++++++++++++++++++
 src/preferences/shared-folders-widget.vala     | 109 ++++++++
 src/{ => preferences}/snapshot-list-row.vala   |   2 +-
 src/preferences/snapshots-page.vala            | 111 ++++++++
 src/properties-toolbar.vala                    |  20 --
 src/properties-window.vala                     |  11 +-
 src/shared-folders.vala                        | 102 -------
 src/snapshots-property.vala                    | 162 -----------
 src/spice-display.vala                         |   7 -
 32 files changed, 1376 insertions(+), 652 deletions(-)
---
diff --git a/data/gnome-boxes.gresource.xml b/data/gnome-boxes.gresource.xml
index 5fe5d7ed..f5f4796c 100644
--- a/data/gnome-boxes.gresource.xml
+++ b/data/gnome-boxes.gresource.xml
@@ -21,7 +21,6 @@
     <file preprocess="xml-stripblanks">ui/list-view.ui</file>
     <file preprocess="xml-stripblanks">ui/list-view-row.ui</file>
     <file preprocess="xml-stripblanks">ui/notification.ui</file>
-    <file preprocess="xml-stripblanks">ui/properties-shared-folder-row.ui</file>
     <file preprocess="xml-stripblanks">ui/properties-page-widget.ui</file>
     <file preprocess="xml-stripblanks">ui/properties-toolbar.ui</file>
     <file preprocess="xml-stripblanks">ui/properties-window.ui</file>
@@ -29,9 +28,6 @@
     <file preprocess="xml-stripblanks">ui/searchbar.ui</file>
     <file preprocess="xml-stripblanks">ui/selectionbar.ui</file>
     <file preprocess="xml-stripblanks">ui/selection-toolbar.ui</file>
-    <file preprocess="xml-stripblanks">ui/shared-folders.ui</file>
-    <file preprocess="xml-stripblanks">ui/shared-folder-popover.ui</file>
-    <file preprocess="xml-stripblanks">ui/snapshot-list-row.ui</file>
     <file preprocess="xml-stripblanks">ui/machine-config-editor.ui</file>
     <file preprocess="xml-stripblanks">ui/topbar.ui</file>
     <file preprocess="xml-stripblanks">ui/transfer-info-row.ui</file>
@@ -52,6 +48,16 @@
     <file preprocess="xml-stripblanks">ui/assistant/pages/preparation-page.ui</file>
     <file preprocess="xml-stripblanks">ui/assistant/pages/setup-page.ui</file>
     <file preprocess="xml-stripblanks">ui/assistant/pages/review-page.ui</file>
+    <!-- VM Preferences window -->
+    <file preprocess="xml-stripblanks">ui/preferences/devices-page.ui</file>
+    <file preprocess="xml-stripblanks">ui/preferences/preferences-window.ui</file>
+    <file preprocess="xml-stripblanks">ui/preferences/resources-page.ui</file>
+    <file preprocess="xml-stripblanks">ui/preferences/shared-folders-widget.ui</file>
+    <file preprocess="xml-stripblanks">ui/preferences/shared-folder-popover.ui</file>
+    <file preprocess="xml-stripblanks">ui/preferences/shared-folder-row.ui</file>
+    <file preprocess="xml-stripblanks">ui/preferences/snapshot-list-row.ui</file>
+    <file preprocess="xml-stripblanks">ui/preferences/snapshots-page.ui</file>
+
     <!-- Welcome Tutorial -->
     <file preprocess="xml-stripblanks">ui/welcome-tutorial.ui</file>
     <file preprocess="xml-stripblanks">ui/welcome-tutorial-page.ui</file>
diff --git a/data/ui/preferences/devices-page.ui b/data/ui/preferences/devices-page.ui
new file mode 100644
index 00000000..42187b2e
--- /dev/null
+++ b/data/ui/preferences/devices-page.ui
@@ -0,0 +1,108 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<interface>
+  <requires lib="gtk+" version="3.0"/>
+  <template class="BoxesDevicesPage" parent="HdyPreferencesPage">
+    <property name="visible">True</property>
+    <property name="icon_name">input-mouse-symbolic</property>
+    <property name="title" translatable="yes">Devices &amp; Shares</property>
+
+    <child>
+      <object class="HdyPreferencesGroup" id="devices_group">
+        <property name="visible">True</property>
+        <property name="title" translatable="yes">Devices</property>
+      </object>
+    </child>
+
+    <child>
+      <object class="BoxesSharedFoldersWidget" id="shared_folders_widget"/>
+    </child>
+
+    <child>
+      <object class="HdyPreferencesGroup">
+        <property name="visible">True</property>
+        <property name="title" translatable="yes">CDROM/DVD Drive</property>
+
+        <child>
+          <object class="HdyActionRow" id="cdrom_row">
+            <property name="visible">True</property>
+            <property name="activatable">False</property>
+            <property name="title" translatable="yes">No CDROM/DVD image</property>
+
+            <child>
+              <object class="GtkStack" id="cdrom_stack">
+                <property name="visible">True</property>
+
+                <child>
+                  <object class="GtkButton" id="cdrom_select_button">
+                    <property name="visible">True</property>
+                    <property name="valign">center</property>
+                    <property name="label" translatable="yes">Select</property>
+                  </object>
+                </child>
+
+                <child>
+                  <object class="GtkMenuButton" id="cdrom_options_button">
+                    <property name="visible">True</property>
+                    <property name="valign">center</property>
+                    <property name="halign">end</property>
+                    <property name="popover">cdrom_popover</property>
+                    <child>
+                      <object class="GtkImage">
+                        <property name="visible">True</property>
+                        <property name="icon-name">emblem-system-symbolic</property>
+                      </object>
+                    </child>
+                  </object>
+                </child>
+              </object>
+            </child>
+          </object>
+        </child>
+      </object>
+    </child>
+  </template>
+
+  <object class="GtkPopover" id="cdrom_popover">
+    <child>
+      <object class="GtkGrid">
+        <property name="visible">True</property>
+        <property name="margin">10</property>
+
+        <child>
+          <object class="GtkModelButton">
+            <property name="visible">True</property>
+            <property name="text" translatable="yes">Remove</property>
+          </object>
+          <packing>
+            <property name="left-attach">1</property>
+            <property name="top-attach">0</property>
+          </packing>
+        </child>
+
+        <child>
+          <object class="GtkModelButton">
+            <property name="visible">True</property>
+            <property name="text" translatable="yes">Eject</property>
+          </object>
+          <packing>
+            <property name="left-attach">1</property>
+            <property name="top-attach">1</property>
+          </packing>
+        </child>
+
+        <child>
+          <object class="GtkCheckButton">
+            <property name="visible">True</property>
+            <property name="valign">center</property>
+            <property name="label" translatable="yes">Make Bootable</property>
+          </object>
+          <packing>
+            <property name="left-attach">0</property>
+            <property name="top-attach">2</property>
+            <property name="width">3</property>
+          </packing>
+        </child>
+      </object>
+    </child>
+  </object>
+</interface>
diff --git a/data/ui/preferences/preferences-window.ui b/data/ui/preferences/preferences-window.ui
new file mode 100644
index 00000000..ace4ce98
--- /dev/null
+++ b/data/ui/preferences/preferences-window.ui
@@ -0,0 +1,24 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<interface>
+  <template class="BoxesPreferencesWindow" parent="HdyPreferencesWindow">
+    <property name="role">boxes-vm-preferences</property>
+    <property name="modal">True</property>
+    <property name="window-position">center</property>
+    <property name="destroy-with-parent">True</property>
+    <property name="icon-name">gtk-preferences</property>
+    <property name="can-swipe-back">True</property>
+    <signal name="delete-event" handler="gtk_widget_hide_on_delete"/>
+
+    <child>
+      <object class="BoxesResourcesPage" id="resources_page"/>
+    </child>
+
+    <child>
+      <object class="BoxesDevicesPage" id="devices_page"/>
+    </child>
+
+    <child>
+      <object class="BoxesSnapshotsPage" id="snapshots_page"/>
+    </child>
+  </template>
+</interface>
diff --git a/data/ui/preferences/resources-page.ui b/data/ui/preferences/resources-page.ui
new file mode 100644
index 00000000..84505387
--- /dev/null
+++ b/data/ui/preferences/resources-page.ui
@@ -0,0 +1,293 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<interface>
+  <requires lib="gtk+" version="3.0"/>
+  <template class="BoxesResourcesPage" parent="HdyPreferencesPage">
+    <property name="visible">True</property>
+    <property name="icon_name">applications-system-symbolic</property>
+    <property name="title" translatable="yes">Resources</property>
+
+    <child>
+      <object class="HdyPreferencesGroup">
+        <property name="visible">True</property>
+
+        <child>
+          <object class="GtkBox">
+            <property name="visible">True</property>
+            <property name="spacing">40</property>
+
+            <child>
+              <object class="GtkLabel">
+                <property name="visible">True</property>
+                <property name="valign">center</property>
+                <property name="label" translatable="yes">_Name</property>
+                <property name="use-underline">True</property>
+                <attributes>
+                  <attribute name="weight" value="bold"/>
+                </attributes>
+              </object>
+            </child>
+
+            <child>
+              <object class="GtkEntry" id="box_name_entry">
+                <property name="visible">True</property>
+                <property name="valign">center</property>
+                <property name="hexpand">True</property>
+              </object>
+            </child>
+          </object>
+        </child>
+      </object>
+    </child>
+
+    <child>
+      <object class="HdyPreferencesGroup" id="resources_group">
+        <property name="visible">True</property>
+
+        <child>
+          <object class="HdyActionRow">
+            <property name="visible">True</property>
+            <property name="title" translatable="yes">CPU</property>
+
+            <child>
+              <object class="GtkLevelBar" id="cpu_level_bar">
+                <property name="visible">True</property>
+                <property name="hexpand">True</property>
+                <property name="halign">fill</property>
+                <property name="valign">center</property>
+                <property name="width-request">200</property>
+              </object>
+            </child>
+          </object>
+        </child>
+
+        <child>
+          <object class="HdyActionRow">
+            <property name="visible">True</property>
+            <property name="title" translatable="yes">I/O</property>
+
+            <child>
+              <object class="GtkLevelBar" id="io_level_bar">
+                <property name="visible">True</property>
+                <property name="hexpand">True</property>
+                <property name="halign">fill</property>
+                <property name="valign">center</property>
+                <property name="width-request">200</property>
+              </object>
+            </child>
+          </object>
+        </child>
+
+        <child>
+          <object class="HdyActionRow">
+            <property name="visible">True</property>
+            <property name="title" translatable="yes">Network</property>
+
+            <child>
+              <object class="GtkLevelBar" id="net_level_bar">
+                <property name="visible">True</property>
+                <property name="hexpand">True</property>
+                <property name="halign">fill</property>
+                <property name="valign">center</property>
+                <property name="width-request">200</property>
+              </object>
+            </child>
+          </object>
+        </child>
+      </object>
+    </child>
+
+    <child>
+      <object class="HdyPreferencesGroup">
+        <property name="visible">True</property>
+        <property name="title" translatable="yes">Resources</property>
+        <property name="description" translatable="yes">Changes to the settings bellow take effect after you 
restart your box.</property>
+
+        <child>
+          <object class="HdyActionRow" id="memory_row">
+            <property name="visible">True</property>
+            <property name="activatable">False</property>
+            <property name="title" translatable="yes">Memory</property>
+            <property name="use-underline">True</property>
+
+            <child>
+              <object class="GtkSpinButton" id="memory_spin_button">
+                <property name="visible">True</property>
+                <property name="can_focus">True</property>
+                <property name="valign">center</property>
+                <signal name="input" handler="on_memory_spin_button_input"/>
+                <signal name="output" handler="on_memory_spin_button_output"/>
+              </object>
+            </child>
+          </object>
+        </child>
+
+        <child>
+          <object class="HdyActionRow" id="storage_row">
+            <property name="visible">True</property>
+            <property name="activatable">False</property>
+            <property name="title" translatable="yes">Storage</property>
+            <property name="use-underline">True</property>
+
+            <child>
+              <object class="GtkSpinButton" id="storage_spin_button">
+                <property name="visible">True</property>
+                <property name="can_focus">True</property>
+                <property name="valign">center</property>
+                <signal name="input" handler="on_storage_spin_button_input"/>
+                <signal name="output" handler="on_storage_spin_button_output"/>
+              </object>
+            </child>
+          </object>
+        </child>
+
+        <child>
+          <object class="HdyActionRow">
+            <property name="visible">True</property>
+            <property name="activatable">False</property>
+            <property name="title" translatable="yes">CPUs</property>
+            <property name="use-underline">True</property>
+
+            <child>
+              <object class="GtkSpinButton" id="cpus_spin_button">
+                <property name="visible">True</property>
+                <property name="can_focus">True</property>
+                <property name="input_purpose">number</property>
+                <property name="valign">center</property>
+              </object>
+            </child>
+          </object>
+        </child>
+
+        <child>
+          <object class="HdyActionRow" id="acceleration_3d_row">
+            <property name="visible">True</property>
+            <property name="activatable">False</property>
+            <property name="title" translatable="yes">3D acceleration</property>
+            <property name="use-underline">True</property>
+
+            <child>
+              <object class="GtkSwitch" id="acceleration_3d_toggle">
+                <property name="visible">True</property>
+                <property name="valign">center</property>
+              </object>
+            </child>
+          </object>
+        </child>
+      </object>
+    </child>
+
+    <child>
+      <object class="HdyPreferencesGroup">
+        <property name="visible">True</property>
+
+        <child>
+          <object class="HdyActionRow">
+            <property name="visible">True</property>
+            <property name="activatable">False</property>
+            <property name="title" translatable="yes">Allow running in background</property>
+            <property name="use-underline">True</property>
+
+            <child>
+              <object class="GtkSwitch" id="run_in_bg_toggle">
+                <property name="visible">True</property>
+                <property name="valign">center</property>
+              </object>
+            </child>
+          </object>
+        </child>
+      </object>
+    </child>
+
+    <child>
+      <object class="HdyPreferencesGroup">
+        <property name="visible">True</property>
+        <property name="title" translatable="yes">Configuration</property>
+
+        <child>
+          <object class="HdyActionRow">
+            <property name="visible">True</property>
+            <property name="title" translatable="yes">Troubleshooting Logs</property>
+            <property name="subtitle" translatable="yes">Diagnose problems with your box using the log 
file.</property>
+
+            <child>
+              <object class="GtkButton">
+                <property name="visible">True</property>
+                <property name="valign">center</property>
+                <signal name="clicked" handler="on_troubleshooting_logs_button_clicked"/>
+
+                <child>
+                  <object class="GtkImage">
+                    <property name="visible">True</property>
+                    <property name="icon-name">document-open-symbolic</property>
+                  </object>
+                </child>
+              </object>
+            </child>
+          </object>
+        </child>
+
+        <child>
+          <object class="HdyActionRow">
+            <property name="visible">True</property>
+            <property name="title" translatable="yes">Edit Configuration</property>
+            <property name="subtitle" translatable="yes">Edit the Libvirt domain configuration for this 
box.</property>
+
+            <child>
+              <object class="GtkButton">
+                <property name="visible">True</property>
+                <property name="valign">center</property>
+                <signal name="clicked" handler="on_edit_configuration_button_clicked"/>
+
+                <child>
+                  <object class="GtkImage">
+                    <property name="visible">True</property>
+                    <property name="icon-name">document-edit-symbolic</property>
+                  </object>
+                </child>
+              </object>
+            </child>
+          </object>
+        </child>
+
+      </object>
+    </child>
+
+    <child>
+      <object class="HdyPreferencesGroup">
+        <property name="visible">True</property>
+
+        <child>
+          <object class="GtkBox" id="power_buttons_box">
+            <property name="visible">True</property>
+            <property name="halign">end</property>
+            <style>
+              <class name="linked"/>
+            </style>
+
+            <child>
+              <object class="GtkButton">
+                <property name="visible">True</property>
+                <property name="label" translatable="yes">Restart</property>
+                <signal name="clicked" handler="on_restart_button_clicked"/>
+                <style>
+                  <class name="suggested-action"/>
+                </style>
+              </object>
+            </child>
+
+            <child>
+              <object class="GtkButton">
+                <property name="visible">True</property>
+                <property name="label" translatable="yes">Force Shutdown</property>
+                <signal name="clicked" handler="on_force_shutdown_button_clicked"/>
+                <style>
+                  <class name="destructive-action"/>
+                </style>
+              </object>
+            </child>
+          </object>
+        </child>
+      </object>
+    </child>
+  </template>
+</interface>
diff --git a/data/ui/preferences/shared-folder-popover.ui b/data/ui/preferences/shared-folder-popover.ui
new file mode 100644
index 00000000..b6676512
--- /dev/null
+++ b/data/ui/preferences/shared-folder-popover.ui
@@ -0,0 +1,140 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<interface>
+  <requires lib="gtk+" version="3.19"/>
+  <template class="BoxesSharedFolderPopover" parent="GtkPopover">
+    <property name="can_focus">False</property>
+    <property name="modal">True</property>
+    <property name="position">bottom</property>
+    <child>
+      <object class="GtkBox">
+        <property name="visible">True</property>
+        <property name="can_focus">False</property>
+        <property name="margin_left">36</property>
+        <property name="margin_right">16</property>
+        <property name="margin_top">12</property>
+        <property name="margin_bottom">12</property>
+        <property name="spacing">23</property>
+        <child>
+          <object class="GtkBox">
+            <property name="visible">True</property>
+            <property name="can_focus">False</property>
+            <property name="orientation">vertical</property>
+            <child>
+              <object class="GtkLabel">
+                <property name="visible">True</property>
+                <property name="can_focus">False</property>
+                <property name="halign">end</property>
+                <property name="valign">start</property>
+                <property name="margin_top">6</property>
+                <property name="label" translatable="yes">Local Folder</property>
+              </object>
+              <packing>
+                <property name="expand">False</property>
+                <property name="fill">True</property>
+                <property name="position">0</property>
+              </packing>
+            </child>
+            <child>
+              <object class="GtkLabel">
+                <property name="visible">True</property>
+                <property name="can_focus">False</property>
+                <property name="halign">end</property>
+                <property name="valign">center</property>
+                <property name="margin_top">20</property>
+                <property name="label" translatable="yes">Name</property>
+              </object>
+              <packing>
+                <property name="expand">False</property>
+                <property name="fill">True</property>
+                <property name="position">1</property>
+              </packing>
+            </child>
+          </object>
+          <packing>
+            <property name="expand">False</property>
+            <property name="fill">True</property>
+            <property name="position">0</property>
+          </packing>
+        </child>
+        <child>
+          <object class="GtkBox">
+            <property name="visible">True</property>
+            <property name="can_focus">False</property>
+            <property name="orientation">vertical</property>
+            <property name="spacing">6</property>
+            <child>
+              <object class="GtkFileChooserButton" id="file_chooser_button">
+                <property name="visible">True</property>
+                <property name="can_focus">False</property>
+                <property name="action">select-folder</property>
+                <property name="title" translatable="yes">Select Shared Folder</property>
+              </object>
+              <packing>
+                <property name="expand">False</property>
+                <property name="fill">True</property>
+                <property name="position">0</property>
+              </packing>
+            </child>
+            <child>
+              <object class="GtkEntry" id="name_entry">
+                <property name="visible">True</property>
+                <property name="can_focus">True</property>
+              </object>
+              <packing>
+                <property name="expand">False</property>
+                <property name="fill">True</property>
+                <property name="position">1</property>
+              </packing>
+            </child>
+            <child>
+              <object class="GtkBox">
+                <property name="visible">True</property>
+                <property name="can_focus">False</property>
+                <property name="spacing">6</property>
+                <property name="homogeneous">True</property>
+                <child>
+                  <object class="GtkButton">
+                    <property name="label" translatable="yes">Cancel</property>
+                    <property name="visible">True</property>
+                    <property name="can_focus">True</property>
+                    <property name="receives_default">True</property>
+                    <signal name="clicked" handler="on_cancel"/>
+                  </object>
+                  <packing>
+                    <property name="expand">True</property>
+                    <property name="fill">True</property>
+                    <property name="position">0</property>
+                  </packing>
+                </child>
+                <child>
+                  <object class="GtkButton">
+                    <property name="label" translatable="yes">Save</property>
+                    <property name="visible">True</property>
+                    <property name="can_focus">True</property>
+                    <property name="receives_default">True</property>
+                    <signal name="clicked" handler="on_save"/>
+                  </object>
+                  <packing>
+                    <property name="expand">True</property>
+                    <property name="fill">True</property>
+                    <property name="position">1</property>
+                  </packing>
+                </child>
+              </object>
+              <packing>
+                <property name="expand">False</property>
+                <property name="fill">True</property>
+                <property name="position">2</property>
+              </packing>
+            </child>
+          </object>
+          <packing>
+            <property name="expand">False</property>
+            <property name="fill">True</property>
+            <property name="position">1</property>
+          </packing>
+        </child>
+      </object>
+    </child>
+  </template>
+</interface>
\ No newline at end of file
diff --git a/data/ui/preferences/shared-folder-row.ui b/data/ui/preferences/shared-folder-row.ui
new file mode 100644
index 00000000..b8147f10
--- /dev/null
+++ b/data/ui/preferences/shared-folder-row.ui
@@ -0,0 +1,25 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<interface>
+  <template class="BoxesSharedFolderRow" parent="HdyActionRow">
+    <property name="visible">True</property>
+
+    <child>
+      <object class="GtkButton">
+        <property name="visible">True</property>
+        <property name="valign">center</property>
+        <signal name="clicked" handler="on_delete_button_clicked"/>
+        <style>
+          <class name="flat"/>
+        </style>
+
+        <child>
+          <object class="GtkImage">
+            <property name="visible">True</property>
+            <property name="halign">end</property>
+            <property name="icon-name">window-close-symbolic</property>
+          </object>
+        </child>
+      </object>
+    </child>
+  </template>
+</interface>
diff --git a/data/ui/preferences/shared-folders-widget.ui b/data/ui/preferences/shared-folders-widget.ui
new file mode 100644
index 00000000..04045122
--- /dev/null
+++ b/data/ui/preferences/shared-folders-widget.ui
@@ -0,0 +1,24 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<interface>
+  <requires lib="gtk+" version="3.19"/>
+  <template class="BoxesSharedFoldersWidget" parent="HdyPreferencesGroup">
+    <property name="visible">True</property>
+    <property name="title" translatable="yes">Shared Folders</property>
+
+    <child>
+      <object class="GtkStack">
+        <property name="visible">True</property>
+
+        <child>
+          <object class="GtkListBox" id="listbox">
+            <property name="visible">True</property>
+            <style>
+              <class name="content"/>
+            </style>
+          </object>
+        </child>
+      </object>
+    </child>
+
+  </template>
+</interface>
diff --git a/data/ui/snapshot-list-row.ui b/data/ui/preferences/snapshot-list-row.ui
similarity index 100%
rename from data/ui/snapshot-list-row.ui
rename to data/ui/preferences/snapshot-list-row.ui
diff --git a/data/ui/preferences/snapshots-page.ui b/data/ui/preferences/snapshots-page.ui
new file mode 100644
index 00000000..858da282
--- /dev/null
+++ b/data/ui/preferences/snapshots-page.ui
@@ -0,0 +1,59 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<interface>
+  <template class="BoxesSnapshotsPage" parent="HdyPreferencesPage">
+    <property name="visible">True</property>
+    <property name="icon_name">input-mouse-symbolic</property>
+    <property name="title" translatable="yes">Snapshots</property>
+
+    <child>
+      <object class="HdyPreferencesGroup">
+        <property name="visible">True</property>
+        <property name="title" translatable="yes">Snapshots</property>
+
+        <child>
+          <object class="GtkStack" id="stack">
+            <property name="visible">True</property>
+
+            <child>
+              <object class="GtkListBox" id="snapshot_list">
+                <property name="visible">True</property>
+                <signal name="add" handler="update_snapshot_stack_page"/>
+                <signal name="remove" handler="update_snapshot_stack_page"/>
+              </object>
+            </child>
+
+            <child>
+              <object class="HdyStatusPage" id="empty_page">
+                <property name="visible">True</property>
+                <property name="title" translatable="yes">No Snapshots Created Yet</property>
+                <property name="description" translatable="yes">Create one using the button 
bellow.</property>
+              </object>
+            </child>
+
+            <child>
+              <object class="GtkBox" id="activity_box">
+                <property name="visible">True</property>
+                <property name="spacing">20</property>
+                <property name="orientation">vertical</property>
+
+                <child>
+                  <object class="GtkSpinner">
+                    <property name="visible">True</property>
+                    <property name="active">True</property>
+                  </object>
+                </child>
+                <child>
+                  <object class="GtkLabel" id="activity_label">
+                    <property name="visible">True</property>
+                    <property name="label" translatable="yes">Creating new snapshot…</property>
+                  </object>
+                </child>
+              </object>
+            </child>
+          </object>
+        </child>
+      </object>
+    </child>
+
+  </template>
+</interface>
diff --git a/data/ui/properties-toolbar.ui b/data/ui/properties-toolbar.ui
index 79061d2a..3d096ba3 100644
--- a/data/ui/properties-toolbar.ui
+++ b/data/ui/properties-toolbar.ui
@@ -98,68 +98,6 @@
       </packing>
     </child>
 
-    <!-- Text Editor page -->
-    <child>
-      <object class="GtkHeaderBar" id="config_editor">
-        <property name="visible">True</property>
-        <property name="show-close-button">True</property>
-        <style>
-          <class name="titlebar"/>
-        </style>
-
-        <child>
-          <object class="GtkButton">
-            <property name="visible">True</property>
-            <property name="valign">center</property>
-            <signal name="clicked" handler="on_troubleshooting_back_clicked"/>
-            <style>
-              <class name="image-button"/>
-            </style>
-
-            <child internal-child="accessible">
-              <object class="AtkObject">
-                <property name="accessible-name" translatable="yes">Back</property>
-              </object>
-            </child>
-
-            <child>
-              <object class="GtkImage">
-                <property name="visible">True</property>
-                <property name="icon-name">go-previous-symbolic</property>
-              </object>
-            </child>
-          </object>
-
-          <packing>
-            <property name="pack-type">start</property>
-          </packing>
-        </child>
-
-        <child>
-          <object class="GtkButton" id="apply_config_button">
-            <property name="visible">True</property>
-            <property name="sensitive">False</property>
-            <property name="valign">center</property>
-            <property name="use-underline">True</property>
-            <property name="label" translatable="yes">_Apply</property>
-            <signal name="clicked" handler="on_config_editor_apply_config_clicked"/>
-            <style>
-              <class name="text-button"/>
-              <class name="suggested-action"/>
-            </style>
-          </object>
-
-          <packing>
-            <property name="pack-type">end</property>
-          </packing>
-        </child>
-      </object>
-
-      <packing>
-        <property name="name">config_editor</property>
-      </packing>
-    </child>
-
     <!-- File chooser page -->
     <child>
       <object class="GtkHeaderBar" id="file_chooser">
diff --git a/data/ui/properties-window.ui b/data/ui/properties-window.ui
index 8a8e63d3..f8e2dd1e 100644
--- a/data/ui/properties-window.ui
+++ b/data/ui/properties-window.ui
@@ -70,16 +70,6 @@
               </packing>
             </child>
 
-            <child>
-              <object class="BoxesMachineConfigEditor" id="config_editor">
-                <property name="visible">True</property>
-              </object>
-
-              <packing>
-                <property name="name">config_editor</property>
-              </packing>
-            </child>
-
             <child>
               <object class="GtkBox">
                 <property name="visible">True</property>
diff --git a/src/actions-popover.vala b/src/actions-popover.vala
index c953532f..4ed0e194 100644
--- a/src/actions-popover.vala
+++ b/src/actions-popover.vala
@@ -8,7 +8,7 @@
         {"force_shutdown",  force_shutdown_activated},
         {"delete",          delete_activated},
         {"clone",           clone_activated},
-        {"properties",      properties_activated},
+        {"preferences",     preferences_activated},
         {"restart",         restart_activated},
         {"send_file",       send_file_activated}
 
@@ -98,9 +98,9 @@ public void update_for_item (CollectionItem item) {
 
         // Properties (in separate section)
         section = new GLib.Menu ();
-        section.append (_("Properties"), "box.properties");
+        section.append (_("Preferences"), "box.preferences");
         menu.append_section (null, section);
-        var action = action_group.lookup_action ("properties") as GLib.SimpleAction;
+        var action = action_group.lookup_action ("preferences") as GLib.SimpleAction;
         action.set_enabled (!importing);
 
         bind_model (menu, null);
@@ -178,7 +178,13 @@ private void send_file_activated () {
     }
 
 
-    private void properties_activated () {
-        window.show_properties ();
+    private void preferences_activated () {
+        var machine = window.current_item as Machine;
+
+        var preferences = new Boxes.PreferencesWindow () {
+            machine = machine,
+            transient_for = window,
+        };
+        preferences.present ();
     }
 }
diff --git a/src/app-window.vala b/src/app-window.vala
index 7ee13cbc..2f44e813 100644
--- a/src/app-window.vala
+++ b/src/app-window.vala
@@ -213,7 +213,6 @@ private void ui_state_changed () {
         foreach (var ui in new Boxes.UI[] { topbar,
                                             icon_view,
                                             list_view,
-                                            props_window,
                                             //wizard_window,
                                             empty_boxes }) {
             ui.set_state (ui_state);
@@ -295,28 +294,6 @@ public void show_welcome_tutorial () {
         }
     }
 
-    public void show_properties () {
-        if (current_item != null) {
-            if (ui_state == UIState.COLLECTION && selection_mode)
-                selection_mode = false;
-            set_state (UIState.PROPERTIES);
-
-            return;
-        }
-
-        var selected_items = view.get_selected_items ();
-
-        if (ui_state == UIState.COLLECTION && selection_mode)
-            selection_mode = false;
-
-        // Show for the first selected item
-        foreach (var item in selected_items) {
-            current_item = item;
-            set_state (UIState.PROPERTIES);
-            break;
-        }
-    }
-
     public void show_send_file () {
         var dialog = new Gtk.FileChooserDialog (
                 _("Select files to transfer"), this, Gtk.FileChooserAction.OPEN,
diff --git a/src/libvirt-machine-properties.vala b/src/libvirt-machine-properties.vala
index 44918ec5..3b65c114 100644
--- a/src/libvirt-machine-properties.vala
+++ b/src/libvirt-machine-properties.vala
@@ -148,7 +148,8 @@ public string collect_logs () {
             try {
                 var config = machine.domain.get_config (0);
                 if (!VMConfigurator.is_install_config (config) && !machine.firmware_is_efi)
-                    add_snapshots_property (ref list);
+                    print ("NOTHIN!");
+                    //add_snapshots_property (ref list);
             } catch (GLib.Error e) {
                 warning (e.message);
             }
@@ -318,9 +319,9 @@ private void add_resource_usage_graphs (ref List<Boxes.Property> list) {
         grid.attach (label, 2, 1, 1, 1);
 
         var stats_id = machine.stats_updated.connect (() => {
-            cpu_graph.points = machine.cpu_stats;
-            io_graph.points = machine.io_stats;
-            net_graph.points = machine.net_stats;
+            //cpu_graph.points = machine.cpu_stats;
+            //io_graph.points = machine.io_stats;
+            //net_graph.points = machine.net_stats;
         });
 
         var prop = add_property (ref list, null, grid);
@@ -373,13 +374,6 @@ private void add_system_props_buttons (ref List<Boxes.Property> list) {
             machine.window.props_window.show_troubleshoot_log (log);
         });
 
-        var edit_button = new Gtk.Button.with_mnemonic (_("Edit XML"));
-        edit_button.halign = Gtk.Align.END;
-        grid.attach (edit_button, 2, 0, 1, 1);
-        edit_button.clicked.connect (() => {
-            machine.window.props_window.show_editor_view (machine);
-        });
-
         var prop = add_property (ref list, null, grid);
         ulong flushed_id = 0;
         flushed_id = prop.flushed.connect (() => {
@@ -781,13 +775,6 @@ private async void on_run_in_bg () {
         });
     }
 
-    private Boxes.SnapshotsProperty add_snapshots_property (ref List<Boxes.Property> list) {
-        var property = new SnapshotsProperty (machine);
-        list.append (property);
-
-        return property;
-    }
-
     private void add_3d_acceleration_property (ref List<Boxes.Property> list) {
         var toggle = new Gtk.Switch ();
         toggle.halign = Gtk.Align.START;
diff --git a/src/libvirt-machine.vala b/src/libvirt-machine.vala
index b11c2d45..5779447c 100644
--- a/src/libvirt-machine.vala
+++ b/src/libvirt-machine.vala
@@ -371,9 +371,9 @@ private async void update_net_stat (DomainInfo info, MachineStat *stat) {
 
     public signal void stats_updated ();
 
-    public double[] cpu_stats;
-    public double[] io_stats;
-    public double[] net_stats;
+    public double cpu_stats;
+    public double io_stats;
+    public double net_stats;
     private async void update_stats () {
         try {
             var now = get_monotonic_time ();
@@ -394,14 +394,10 @@ private async void update_stats () {
             warning (err.message);
         }
 
-        cpu_stats = {};
-        io_stats = {};
-        net_stats = {};
-
         foreach (var stat in stats) {
-            cpu_stats += stat.cpu_guest_percent;
-            net_stats += (stat.net_read + stat.net_write);
-            io_stats += (stat.disk_read + stat.disk_write);
+            cpu_stats = stat.cpu_guest_percent;
+            net_stats = (stat.net_read + stat.net_write);
+            io_stats = (stat.disk_read + stat.disk_write);
         }
 
         stats_updated ();
diff --git a/src/machine.vala b/src/machine.vala
index cdc019a9..ce169de6 100644
--- a/src/machine.vala
+++ b/src/machine.vala
@@ -610,7 +610,7 @@ private async void try_connect_display (ConnectFlags flags = ConnectFlags.NONE)
             if (this is LibvirtMachine) {
                 Notification.OKFunc troubleshoot = () => {
                     window.current_item = this;
-                    window.show_properties ();
+                    //window.show_properties ();
 
                     var libvirt_machine = this as LibvirtMachine;
                     var logs = libvirt_machine.properties.collect_logs ();
diff --git a/src/meson.build b/src/meson.build
index a84bde5d..db0275fc 100644
--- a/src/meson.build
+++ b/src/meson.build
@@ -88,7 +88,6 @@ vala_sources = [
   'selectionbar.vala',
   'selection-toolbar.vala',
   'shared-folders.vala',
-  'config-editor.vala',
   'transfer-info-row.vala',
   'troubleshoot-view.vala',
   'topbar.vala',
@@ -109,11 +108,10 @@ vala_sources = [
   'empty-boxes.vala',
   'tracker-iso-query.vala',
   'troubleshoot-log.vala',
-  'snapshot-list-row.vala',
-  'snapshots-property.vala',
 ]
 
 subdir('assistant')
+subdir('preferences')
 
 dependencies = [
   config_h,
@@ -123,7 +121,6 @@ dependencies = [
   dependency ('gtk+-3.0', version: '>= 3.22.20'),
   dependency ('gtk-vnc-2.0', version: '>= 0.4.4'),
   dependency ('gvncpulse-1.0'),
-  dependency ('gtksourceview-4'),
   dependency ('libhandy-1', version: '>= 1.0.0'),
   dependency ('libosinfo-1.0', version: '>= 1.7.0'),
   dependency ('libsecret-1'),
diff --git a/src/preferences/devices-page.vala b/src/preferences/devices-page.vala
new file mode 100644
index 00000000..d94378f9
--- /dev/null
+++ b/src/preferences/devices-page.vala
@@ -0,0 +1,54 @@
+// This file is part of GNOME Boxes. License: LGPLv2+
+using Gtk;
+
+[GtkTemplate (ui = "/org/gnome/Boxes/ui/preferences/devices-page.ui")]
+private class Boxes.DevicesPage : Hdy.PreferencesPage {
+    private LibvirtMachine machine;
+
+    [GtkChild]
+    private unowned Hdy.PreferencesGroup devices_group;
+    [GtkChild]
+    private unowned Boxes.SharedFoldersWidget shared_folders_widget;
+
+    [GtkChild]
+    private unowned Hdy.ActionRow cdrom_row;
+    [GtkChild]
+    private unowned Gtk.Stack cdrom_stack;
+    [GtkChild]
+    private unowned Gtk.Button cdrom_select_button;
+    [GtkChild]
+    private unowned Gtk.MenuButton cdrom_options_button;
+
+    public void setup (LibvirtMachine machine) {
+        this.machine = machine;
+
+        devices_group.visible = !App.is_running_in_flatpak ();
+
+        foreach (var device_config in machine.domain_config.get_devices ()) {
+            if (device_config is GVirConfig.DomainDisk) {
+                var disk_config = device_config as GVirConfig.DomainDisk;
+                var disk_type = disk_config.get_guest_device_type ();
+
+                if (disk_type == GVirConfig.DomainDiskGuestDeviceType.CDROM) {
+                    setup_cdrom_row (disk_config);
+                }
+            }
+        }
+
+        shared_folders_widget.setup (machine.config.uuid);
+    }
+
+    private void setup_cdrom_row (GVirConfig.DomainDisk disk_config) {
+        var source = disk_config.get_source ();
+        if (source == null || source == "") {
+            cdrom_row.set_title (_("No CD/DVD image"));
+
+            cdrom_stack.set_visible_child (cdrom_select_button);
+
+            return;
+        }
+
+        cdrom_stack.set_visible_child (cdrom_options_button);
+        cdrom_row.set_title (get_utf8_basename (source));
+    }
+}
diff --git a/src/preferences/meson.build b/src/preferences/meson.build
new file mode 100644
index 00000000..cd821f5d
--- /dev/null
+++ b/src/preferences/meson.build
@@ -0,0 +1,8 @@
+vala_sources += files( 
+  'devices-page.vala',
+  'preferences-window.vala',
+  'resources-page.vala',
+  'shared-folders-widget.vala',
+  'snapshot-list-row.vala',
+  'snapshots-page.vala',
+)
diff --git a/src/preferences/preferences-window.vala b/src/preferences/preferences-window.vala
new file mode 100644
index 00000000..e3bbd58c
--- /dev/null
+++ b/src/preferences/preferences-window.vala
@@ -0,0 +1,20 @@
+// This file is part of GNOME Boxes. License: LGPLv2+
+using Gtk;
+
+[GtkTemplate (ui = "/org/gnome/Boxes/ui/preferences/preferences-window.ui")]
+private class Boxes.PreferencesWindow : Hdy.PreferencesWindow {
+    public Machine machine {
+        set {
+            resources_page.setup (value as LibvirtMachine);
+            devices_page.setup (value as LibvirtMachine);
+            snapshots_page.setup (value as LibvirtMachine);
+        }
+    }
+
+    [GtkChild]
+    private unowned Boxes.ResourcesPage resources_page;
+    [GtkChild]
+    private unowned Boxes.DevicesPage devices_page;
+    [GtkChild]
+    private unowned Boxes.SnapshotsPage snapshots_page;
+}
diff --git a/src/preferences/resources-page.vala b/src/preferences/resources-page.vala
new file mode 100644
index 00000000..d4b37e0e
--- /dev/null
+++ b/src/preferences/resources-page.vala
@@ -0,0 +1,365 @@
+// This file is part of GNOME Boxes. License: LGPLv2+
+using GLib;
+using Gtk;
+
+[GtkTemplate (ui = "/org/gnome/Boxes/ui/preferences/resources-page.ui")]
+private class Boxes.ResourcesPage : Hdy.PreferencesPage {
+    private LibvirtMachine machine;
+
+    private FileMonitor config_file_monitor;
+
+    private string logs;
+
+    [GtkChild]
+    private unowned Gtk.Entry box_name_entry;
+
+    [GtkChild]
+    private unowned Hdy.PreferencesGroup resources_group;
+    [GtkChild]
+    private unowned Gtk.LevelBar cpu_level_bar;
+    [GtkChild]
+    private unowned Gtk.LevelBar io_level_bar;
+    [GtkChild]
+    private unowned Gtk.LevelBar net_level_bar;
+
+    [GtkChild]
+    private unowned Hdy.ActionRow memory_row;
+    [GtkChild]
+    private unowned Gtk.SpinButton memory_spin_button;
+    [GtkChild]
+    private unowned Hdy.ActionRow storage_row;
+    [GtkChild]
+    private unowned Gtk.SpinButton storage_spin_button;
+    [GtkChild]
+    private unowned Gtk.SpinButton cpus_spin_button;
+    [GtkChild]
+    private unowned Hdy.ActionRow acceleration_3d_row;
+    [GtkChild]
+    private unowned Gtk.Switch acceleration_3d_toggle;
+
+    [GtkChild]
+    private unowned Gtk.Switch run_in_bg_toggle;
+    [GtkChild]
+    private unowned Gtk.Box power_buttons_box;
+
+    public async void setup (LibvirtMachine machine) {
+        this.machine = machine;
+
+        bind_widget_property (box_name_entry, "text", "name");
+
+        machine.notify["state"].connect (on_machine_state_change);
+        on_machine_state_change ();
+
+        mark_recommended_resources ();
+
+        bind_widget_property (run_in_bg_toggle, "active", "run-in-bg");
+
+        var accel3d_is_supported = yield machine.supports_accel3d ();
+        acceleration_3d_row.visible = accel3d_is_supported;
+        if (accel3d_is_supported) {
+            bind_widget_property (acceleration_3d_toggle, "active", "acceleration-3d");
+        } 
+
+        try {
+            var host_topology = machine.connection.get_node_info ();
+
+            setup_memory_row (host_topology.memory);
+            setup_cpu_row (host_topology.cores * host_topology.sockets * host_topology.threads);
+            setup_storage_row ();
+        } catch (GLib.Error error) {
+            warning ("Failed to obtain virtual resources for machine: %s", error.message);
+        }
+    }
+
+    private async void mark_recommended_resources () {
+        var os = yield get_os_for_machine (machine);
+        if (os == null)
+            return;
+
+        var architecture = machine.domain_config.get_os ().get_arch ();
+        var recommended_resources = OSDatabase.get_recommended_resources_for_os (os, architecture);
+        if (recommended_resources != null) {
+            var row_subtitle = _("Recommended %s.");
+
+            memory_row.set_subtitle (row_subtitle.printf (GLib.format_size (recommended_resources.ram,
+                                                                            
GLib.FormatSizeFlags.IEC_UNITS)));
+            storage_row.set_subtitle (row_subtitle.printf (GLib.format_size (recommended_resources.storage,
+                                                                             
GLib.FormatSizeFlags.IEC_UNITS)));
+        }
+    }
+
+    private async Osinfo.Os? get_os_for_machine (LibvirtMachine machine) {
+        var os_id = VMConfigurator.get_os_id (machine.domain_config);
+        if (os_id == null)
+            return null;
+
+        var os_db = MediaManager.get_default ().os_db;
+        try {
+            return yield os_db.get_os_by_id (os_id);
+        } catch (OSDatabaseError error) {
+            warning ("Failed to find OS with ID '%s': %s", os_id, error.message);
+
+            return null;
+        }
+    }
+
+    private void on_machine_state_change () {
+        cpu_level_bar.value = machine.cpu_stats;
+        io_level_bar.value = machine.io_stats;
+        net_level_bar.value = machine.net_stats;
+
+        resources_group.visible = machine.is_running;
+        power_buttons_box.visible = machine.is_running;
+    }
+
+    private void setup_memory_row (ulong host_memory) {
+        uint64 ram = machine.domain_config.memory * Osinfo.KIBIBYTES;
+        uint64 max_ram = host_memory * Osinfo.KIBIBYTES;
+        uint64 min_ram = 128 * Osinfo.MEBIBYTES;
+
+        memory_spin_button.set_range (min_ram, max_ram);
+        memory_spin_button.set_increments (min_ram, max_ram);
+        memory_spin_button.set_value (ram);
+    }
+
+    [GtkCallback]
+    private int on_memory_spin_button_input (Gtk.SpinButton spin_button, out double new_val) {
+        var ram = spin_button.get_adjustment ().get_value ();
+
+        /* FIXME: we should be getting the value with spin_button.get_text () so we can
+         * accept user manual input. This will require to parse the text properly and
+         * convert strings such as 2.0 GiB into 2.0 * Osinfo.MEBIBYTE * 1024.
+         *
+         * As it is now, we don't support manual input, and the value can only be changed
+         * by using the + and - buttons of the GtkSpinButton. */
+        new_val = ram;
+
+        return 1;
+    }
+
+    [GtkCallback]
+    private bool on_memory_spin_button_output (Gtk.SpinButton spin_button) {
+        uint64 ram = (uint64)spin_button.get_adjustment ().get_value ();
+
+        spin_button.text = GLib.format_size (ram, GLib.FormatSizeFlags.IEC_UNITS);
+
+        return true;
+    }
+
+    private void setup_cpu_row (uint host_vcpus) {
+        var cpus = (int)machine.domain_config.get_vcpus ();
+
+        cpus_spin_button.set_range (1, host_vcpus);
+        cpus_spin_button.set_increments (1, host_vcpus);
+        cpus_spin_button.set_value (cpus);
+    }
+
+    private void setup_storage_row () {
+        if (machine.importing || machine.storage_volume == null) {
+            storage_row.hide ();
+
+            return;
+        }
+
+        try {
+            var volume_info = machine.storage_volume.get_info ();
+            var pool = get_storage_pool (machine.connection);
+            var pool_info = pool.get_info ();
+
+            var min_storage = get_minimum_disk_size (volume_info);
+            var max_storage = volume_info.allocation + pool_info.available;
+
+            if (min_storage >= max_storage) {
+                memory_row.subtitle = _("There is not enough space on your machine to increate the maximum 
disk size.");
+            }
+
+            storage_spin_button.set_range (min_storage, max_storage);
+            storage_spin_button.set_increments (256 * 1000 * 1000, volume_info.allocation);
+            storage_spin_button.set_value (volume_info.allocation);
+        } catch (GLib.Error error) {
+            warning ("Failed to setup storage row: %s", error.message);
+        }
+    }
+
+    private uint64 get_minimum_disk_size (GVir.StorageVolInfo volume_info) throws GLib.Error {
+        if (machine.vm_creator == null)
+            return volume_info.capacity;
+
+        Osinfo.Resources minimum_resources = null;
+        if (machine.vm_creator.install_media.os != null) {
+            var os = machine.vm_creator.install_media.os;
+            var architecture = machine.domain_config.get_os ().get_arch ();
+            minimum_resources = OSDatabase.get_minimum_resources_for_os (os, architecture);
+        }
+
+        if (minimum_resources != null && minimum_resources.storage != -1) {
+            return minimum_resources.storage;
+        } else {
+            minimum_resources = OSDatabase.get_minimum_resources ();
+
+            return uint64.min (volume_info.capacity, minimum_resources.storage);
+        }
+    }
+
+    [GtkCallback]
+    private int on_storage_spin_button_input (Gtk.SpinButton spin_button, out double new_val) {
+        var storage = spin_button.get_adjustment ().get_value ();
+
+        /* FIXME: see on_memory_spin_button_input () */
+        new_val = storage;
+
+        return 1;
+    }
+
+    [GtkCallback]
+    private bool on_storage_spin_button_output (Gtk.SpinButton spin_button) {
+        uint64 storage = (uint64)spin_button.get_adjustment ().get_value ();
+
+        spin_button.text = GLib.format_size (storage, GLib.FormatSizeFlags.DEFAULT);
+
+        return true;
+    }
+
+    [GtkCallback]
+    private void on_troubleshooting_logs_button_clicked () {
+        if (logs == null)
+            logs = collect_logs (machine);
+        
+        try {
+            var filename = get_cache ("logs", machine.domain.get_name () + ".logs");
+            File file = GLib.File.new_for_path (filename);
+
+            FileOutputStream os = file.replace (null, false, FileCreateFlags.REPLACE_DESTINATION);
+
+            os.write (logs.data);
+            GLib.AppInfo.launch_default_for_uri (file.get_uri (), null);
+
+            debug ("Showing vm configuration at %s", file.get_uri ());
+            GLib.stdout.printf (logs + "\n");
+        } catch (GLib.Error error) {
+            warning ("Failed to collect machine logs: %s", error.message);
+        }
+    }
+
+    [GtkCallback]
+    private void on_edit_configuration_button_clicked () {
+        var domain_xml = machine.domain_config.to_xml ();
+
+        try {
+            File file = GLib.File.new_for_path (Path.build_filename (Environment.get_home_dir (),
+                                                "." + machine.domain.get_name () + ".draft.txt"));
+            FileOutputStream os = file.replace (null, false, FileCreateFlags.REPLACE_DESTINATION);
+            os.write (domain_xml.data);
+
+            config_file_monitor = file.monitor_file (GLib.FileMonitorFlags.NONE, null);
+            config_file_monitor.changed.connect (on_domain_configuration_edited);
+
+            GLib.AppInfo.launch_default_for_uri (file.get_uri (), null);
+        } catch (GLib.Error error) {
+            warning ("Failed to edit VM configuration: %s", error.message);
+        }
+    }
+
+    private void on_domain_configuration_edited (File file,
+                                                 File? other_file,
+                                                 FileMonitorEvent event_type) {
+        if (event_type != FileMonitorEvent.CHANGED &&
+            event_type != FileMonitorEvent.CHANGES_DONE_HINT) {
+            return;
+        }
+
+        try {
+            uint8[] contents;
+            string etag_out;
+            file.load_contents (null, out contents, out etag_out);
+
+            GVirConfig.Domain new_config = new GVirConfig.Domain.from_xml ((string)contents);
+            var edited_tag = "<edited>%s</edited>".printf (new DateTime.now_local ().to_string ());
+            new_config.set_custom_xml (edited_tag,
+                                       "edited",
+                                       "https://wiki.gnome.org/Apps/Boxes/edited";);
+            machine.domain.set_config (new_config);
+
+            debug ("Overriding configuration for %s", machine.domain.get_name ());
+        } catch (GLib.Error error) {
+            warning ("Failed to load new domain configuration: %s", error.message);
+
+            var message_dialog = new Gtk.MessageDialog (App.app.main_window,
+                                                        Gtk.DialogFlags.DESTROY_WITH_PARENT,
+                                                        Gtk.MessageType.ERROR,
+                                                        Gtk.ButtonsType.CLOSE,
+                                                        _("Failed to save domain configuration: %s"),
+                                                        error.message);
+            message_dialog.run ();
+            message_dialog.destroy ();
+        }
+    }
+
+    [GtkCallback]
+    private void on_force_shutdown_button_clicked () {
+        machine.force_shutdown ();
+    }
+
+    [GtkCallback]
+    private void on_restart_button_clicked () {
+        machine.restart ();
+    }
+
+    private void bind_widget_property (Gtk.Widget widget, string widget_property, string machine_property) {
+        machine.bind_property (machine_property, widget, widget_property, BindingFlags.BIDIRECTIONAL);
+
+        var value = GLib.Value (machine.get_class ().find_property (machine_property).value_type);
+        machine.get_property (machine_property, ref value);
+        widget.set_property (widget_property, value);
+    }
+
+    private string collect_logs (LibvirtMachine machine) {
+        var builder = new StringBuilder ();
+
+        builder.append_printf ("Broker URL: %s\n", machine.source.uri);
+        builder.append_printf ("Domain: %s\n", machine.domain.get_name ());
+        builder.append_printf ("UUID: %s\n", machine.domain.get_uuid ());
+        builder.append_printf ("Persistent: %s\n", machine.domain.get_persistent () ? "yes" : "no");
+        try {
+            var info = machine.domain.get_info ();
+            builder.append_printf ("Cpu time: %"+uint64.FORMAT_MODIFIER+"d\n", info.cpuTime);
+            builder.append_printf ("Memory: %"+uint64.FORMAT_MODIFIER+"d KiB\n", info.memory);
+            builder.append_printf ("Max memory: %"+uint64.FORMAT_MODIFIER+"d KiB\n",
+                                   machine.connection.get_node_info ().memory);
+            builder.append_printf ("CPUs: %d\n", info.nrVirtCpu);
+            builder.append_printf ("State: %s\n", info.state.to_string ());
+        } catch (GLib.Error e) {
+        }
+
+        if (machine.display != null)
+            machine.display.collect_logs (builder);
+
+
+        try {
+            var conf = machine.domain.get_config (GVir.DomainXMLFlags.NONE);
+            builder.append_printf ("\nDomain config:\n");
+            builder.append_printf ("------------------------------------------------------------\n");
+
+            builder.append (conf.to_xml ());
+            builder.append_printf ("\n" +
+                                   "------------------------------------------------------------\n");
+        } catch (GLib.Error error) {
+        }
+
+        try {
+            var logfile = Path.build_filename (Environment.get_user_cache_dir (),
+                                               "libvirt/qemu/log",
+                                               machine.domain.get_name ()  + ".log");
+            string data;
+            FileUtils.get_contents (logfile, out data);
+            builder.append_printf ("\nQEMU log:\n");
+            builder.append_printf ("------------------------------------------------------------\n");
+
+            builder.append (data);
+            builder.append_printf ("------------------------------------------------------------\n");
+        } catch (GLib.Error e) {
+        }
+
+        return builder.str;
+    }
+}
diff --git a/src/preferences/shared-folders-widget.vala b/src/preferences/shared-folders-widget.vala
new file mode 100644
index 00000000..8d5340d2
--- /dev/null
+++ b/src/preferences/shared-folders-widget.vala
@@ -0,0 +1,109 @@
+// This file is part of GNOME Boxes. License: LGPLv2+
+
+[GtkTemplate (ui = "/org/gnome/Boxes/ui/preferences/shared-folder-row.ui")]
+private class Boxes.SharedFolderRow : Hdy.ActionRow {
+    public signal void removed (SharedFolder folder);
+
+    public SharedFolder folder { get; private set; }
+
+    public SharedFolderRow (SharedFolder folder) {
+        this.folder = folder;
+
+        folder.bind_property ("path", this, "title", BindingFlags.SYNC_CREATE);
+        folder.bind_property ("name", this, "subtitle", BindingFlags.SYNC_CREATE);
+    }
+
+    [GtkCallback]
+    private void on_delete_button_clicked () {
+        removed (folder);
+    }
+}
+
+[GtkTemplate (ui = "/org/gnome/Boxes/ui/preferences/shared-folders-widget.ui")]
+private class Boxes.SharedFoldersWidget: Hdy.PreferencesGroup {
+    private string machine_uuid;
+
+    private SharedFoldersManager manager = SharedFoldersManager.get_default ();
+
+    private GLib.ListStore list_model;
+
+    private Boxes.SharedFolderPopover popover;
+
+    [GtkChild]
+    private unowned Gtk.ListBox listbox;
+    //[GtkChild]
+    //private unowned Gtk.MenuButton add_button;
+
+    construct {
+        popover = new SharedFolderPopover ();
+        popover.saved.connect (on_popover_saved);
+        //add_button.popover = popover;
+    }
+
+    public void setup (string machine_uuid) {
+        this.machine_uuid = machine_uuid;
+
+        list_model = manager.get_folders (machine_uuid);
+        listbox.bind_model (list_model, create_shared_folder_row);
+
+        var add_button = new Gtk.MenuButton () {
+            visible = true,
+            image = new Gtk.Image () {
+                icon_name = "list-add-symbolic"
+            },
+            popover = popover
+        }; 
+        add_button.get_style_context ().add_class ("flat");
+
+        listbox.add (add_button);
+    }
+
+    private bool on_popover_saved (string path, string? name) {
+        return manager.add_item (new SharedFolder (machine_uuid, path, name));
+    }
+
+    private Gtk.Widget create_shared_folder_row (Object item) {
+        var folder = item as SharedFolder;
+        var row = new SharedFolderRow (folder);
+
+        row.removed.connect (manager.remove_item);
+
+        return row;
+    }
+}
+
+[GtkTemplate (ui = "/org/gnome/Boxes/ui/preferences/shared-folder-popover.ui")]
+private class Boxes.SharedFolderPopover: Gtk.Popover {
+    public signal bool saved (string path, string name);
+
+    [GtkChild]
+    public unowned Gtk.FileChooserButton file_chooser_button;
+    [GtkChild]
+    public unowned Gtk.Entry name_entry;
+
+    construct {
+        var default_path = Environment.get_user_special_dir (UserDirectory.PUBLIC_SHARE);
+        file_chooser_button.set_current_folder (default_path);
+    }
+
+    [GtkCallback]
+    public void on_cancel (Gtk.Button cancel_button) {
+        popdown ();
+    }
+
+    [GtkCallback]
+    public void on_save (Gtk.Button save_button) {
+        var uri = file_chooser_button.get_uri ();
+        File file = File.new_for_uri (uri);
+        var name = name_entry.get_text ();
+
+        if (uri != null) {
+            if (name == "")
+                name = file.get_basename ();
+
+            saved (file.get_path (), name);
+        }
+
+        popdown ();
+    }
+}
diff --git a/src/snapshot-list-row.vala b/src/preferences/snapshot-list-row.vala
similarity index 98%
rename from src/snapshot-list-row.vala
rename to src/preferences/snapshot-list-row.vala
index dee9b007..2602af2e 100644
--- a/src/snapshot-list-row.vala
+++ b/src/preferences/snapshot-list-row.vala
@@ -1,6 +1,6 @@
 // This file is part of GNOME Boxes. License: LGPLv2+
 
-[GtkTemplate (ui = "/org/gnome/Boxes/ui/snapshot-list-row.ui")]
+[GtkTemplate (ui = "/org/gnome/Boxes/ui/preferences/snapshot-list-row.ui")]
 private class Boxes.SnapshotListRow : Gtk.ListBoxRow {
     public GVir.DomainSnapshot snapshot;
     public string activity_message { get; set; default = ""; }
diff --git a/src/preferences/snapshots-page.vala b/src/preferences/snapshots-page.vala
new file mode 100644
index 00000000..e1ab0345
--- /dev/null
+++ b/src/preferences/snapshots-page.vala
@@ -0,0 +1,111 @@
+// This file is part of GNOME Boxes. License: LGPLv2+
+using Gtk;
+
+[GtkTemplate (ui = "/org/gnome/Boxes/ui/preferences/snapshots-page.ui")]
+private class Boxes.SnapshotsPage : Hdy.PreferencesPage {
+    private LibvirtMachine machine;
+
+    [GtkChild]
+    private unowned Gtk.Stack stack;
+    [GtkChild]
+    private unowned Gtk.ListBox snapshot_list;
+    [GtkChild]
+    private unowned Hdy.StatusPage empty_page;
+    [GtkChild]
+    private unowned Gtk.Box activity_box;
+    [GtkChild]
+    private unowned Gtk.Label activity_label;
+
+    private string? activity {
+        set {
+            if (value == null) {
+                stack.visible_child = snapshot_list;
+            } else {
+                activity_label.label = value;
+                stack.visible_child = activity_box;
+            }
+        }
+    }
+
+    public void setup (LibvirtMachine machine) {
+        this.machine = machine;
+
+        snapshot_list.set_sort_func (config_sort_func);
+
+        //fetch_snapshots.begin (machine);
+
+        var add_button = new Gtk.Button () {
+            visible = true,
+            image = new Gtk.Image () {
+                visible = true,
+                icon_name = "list-add-symbolic"
+            }
+        };
+        snapshot_list.add (add_button);
+        add_button.clicked.connect (create_snapshot);
+    }
+
+    private async void fetch_snapshots (LibvirtMachine machine) {
+        try {
+            var snapshots = yield machine.properties.get_snapshots (null);
+
+            foreach (var snapshot in snapshots) {
+                var row = new SnapshotListRow (snapshot, machine);
+                row.notify["activity-message"].connect (row_activity_changed);
+                snapshot_list.add (row);
+            }
+        } catch (GLib.Error e) {
+            warning ("Could not fetch snapshots: %s", e.message);
+        }
+
+        update_snapshot_stack_page ();
+    }
+
+    private int config_sort_func (Gtk.ListBoxRow row1, Gtk.ListBoxRow row2) {
+        try {
+            var snapshot_row1 = row1 as SnapshotListRow;
+            var snapshot_row2 = row2 as SnapshotListRow;
+
+            var conf1  = snapshot_row1.snapshot.get_config (0);
+            var conf2  = snapshot_row2.snapshot.get_config (0);
+            if (conf1.get_creation_time () < conf2.get_creation_time ())
+                return -1;
+            else
+                return 1;
+        } catch (GLib.Error e) {
+            warning ("Failed to fetch snapshot config: %s", e.message);
+            return 0;
+        }
+    }
+
+    private async void create_snapshot () {
+        if (machine.state == Machine.MachineState.RUNNING)
+            this.activity = _("Creating new snapshot…");
+
+        try {
+            var new_snapshot = yield machine.create_snapshot ();
+            var new_row = new SnapshotListRow (new_snapshot, machine);
+            new_row.notify["activity-message"].connect (row_activity_changed);
+            snapshot_list.add (new_row);
+        } catch (GLib.Error e) {
+            var msg = _("Failed to create snapshot of %s").printf (machine.name);
+            machine.window.notificationbar.display_error (msg);
+            warning (e.message);
+        }
+        this.activity = null;
+    }
+
+    private void row_activity_changed (GLib.Object source, GLib.ParamSpec param_spec) {
+        var row = source as SnapshotListRow;
+        this.activity = row.activity_message;
+    }
+
+    [GtkCallback]
+    private void update_snapshot_stack_page () {
+        var num_snapshots = snapshot_list.get_children ().length ();
+        if (num_snapshots > 1)
+            stack.visible_child = snapshot_list;
+        else
+            stack.visible_child = empty_page;
+    }
+}
diff --git a/src/properties-toolbar.vala b/src/properties-toolbar.vala
index d8bc525e..f45ba9b9 100644
--- a/src/properties-toolbar.vala
+++ b/src/properties-toolbar.vala
@@ -16,11 +16,6 @@
     [GtkChild]
     public unowned Gtk.HeaderBar main;
 
-    [GtkChild]
-    public unowned Gtk.HeaderBar config_editor;
-    [GtkChild]
-    public unowned Gtk.Button apply_config_button;
-
     [GtkChild]
     public unowned Gtk.Button troubleshooting_back_button;
 
@@ -43,7 +38,6 @@ public void setup_ui (AppWindow window, PropertiesWindow props_window) {
         this.props_window = props_window;
 
         window.notify["ui-state"].connect (ui_state_changed);
-        apply_config_button.notify["sensitive"].connect (on_apply_config_editor_sensitivity_changed);
     }
 
     public void click_back_button () {
@@ -63,20 +57,6 @@ private void on_copy_clipboard_clicked () requires (page == PropsWindowPage.TROU
         props_window.copy_troubleshoot_log_to_clipboard ();
     }
 
-    [GtkCallback]
-    private void on_config_editor_apply_config_clicked () {
-        props_window.config_editor.apply ();
-        apply_config_button.sensitive = false;
-    }
-
-    private void on_apply_config_editor_sensitivity_changed () {
-        if (apply_config_button.sensitive) {
-            config_editor.set_title ("*" + config_editor.get_title ());
-        } else {
-            config_editor.set_title (config_editor.get_title ().substring (1));
-        }
-    }
-
     [GtkCallback]
     private void on_title_entry_changed () {
         window.current_item.name = title_entry.text;
diff --git a/src/properties-window.vala b/src/properties-window.vala
index 60b4aca0..006ad9fb 100644
--- a/src/properties-window.vala
+++ b/src/properties-window.vala
@@ -12,7 +12,7 @@
 
 [GtkTemplate (ui = "/org/gnome/Boxes/ui/properties-window.ui")]
 private class Boxes.PropertiesWindow: Gtk.Window, Boxes.UI {
-    public const string[] page_names = { "main", "troubleshoot_log", "file_chooser", "config_editor" };
+    public const string[] page_names = { "main", "troubleshoot_log", "file_chooser" };
 
     public UIState previous_ui_state { get; protected set; }
     public UIState ui_state { get; protected set; }
@@ -34,8 +34,6 @@
     public unowned Properties properties;
     [GtkChild]
     public unowned TroubleshootLog troubleshoot_log;
-    [GtkChild]
-    public unowned MachineConfigEditor config_editor;
 
     public Gtk.FileChooserNative file_chooser;
     [GtkChild]
@@ -68,13 +66,6 @@ public void show_troubleshoot_log (string log) {
         page = PropsWindowPage.TROUBLESHOOTING_LOG;
     }
 
-    public void show_editor_view (LibvirtMachine machine) {
-        page = PropsWindowPage.TEXT_EDITOR;
-        config_editor.setup (machine, topbar.apply_config_button);
-
-        topbar.config_editor.set_title (machine.name);
-    }
-
     public void show_file_chooser (owned FileChosenFunc file_chosen_func) {
         page = PropsWindowPage.FILE_CHOOSER;
         var res = file_chooser.run ();
diff --git a/src/shared-folders.vala b/src/shared-folders.vala
index b3c7c15c..ce14b132 100644
--- a/src/shared-folders.vala
+++ b/src/shared-folders.vala
@@ -179,105 +179,3 @@ private static string get_shared_folder_real_path (SharedFolder folder) {
                                          folder.machine_uuid);
     }
 }
-
-[GtkTemplate (ui = "/org/gnome/Boxes/ui/properties-shared-folder-row.ui")]
-private class Boxes.SharedFolderRow : Gtk.ListBoxRow {
-    public signal void removed (SharedFolder folder);
-    [GtkChild]
-    private unowned Gtk.Label folder_path_label;
-    [GtkChild]
-    private unowned Gtk.Label folder_name_label;
-
-    public SharedFolder folder { get; private set; }
-
-    public SharedFolderRow (SharedFolder folder) {
-        this.folder = folder;
-
-        folder.bind_property ("path", folder_path_label, "label", BindingFlags.SYNC_CREATE);
-        folder.bind_property ("name", folder_name_label, "label", BindingFlags.SYNC_CREATE);
-    }
-
-    [GtkCallback]
-    private void on_delete_button_clicked () {
-        removed (folder);
-    }
-}
-
-[GtkTemplate (ui = "/org/gnome/Boxes/ui/shared-folders.ui")]
-private class Boxes.SharedFoldersWidget: Gtk.Frame {
-    private string machine_uuid;
-
-    private SharedFoldersManager manager = SharedFoldersManager.get_default ();
-
-    private GLib.ListStore list_model;
-
-    private Boxes.SharedFolderPopover popover;
-    [GtkChild]
-    private unowned Gtk.ListBox listbox;
-
-    public SharedFoldersWidget (string machine_uuid) {
-        this.machine_uuid = machine_uuid;
-
-        list_model = manager.get_folders (machine_uuid);
-        listbox.bind_model (list_model, create_shared_folder_row);
-
-        popover = new SharedFolderPopover ();
-        popover.saved.connect (on_popover_saved);
-    }
-
-    private bool on_popover_saved (string path, string? name) {
-        return manager.add_item (new SharedFolder (machine_uuid, path, name));
-    }
-
-    [GtkCallback]
-    private void on_add_button_clicked (Gtk.Button button) {
-        popover.relative_to = button;
-
-        popover.popup ();
-    }
-
-    private Gtk.Widget create_shared_folder_row (Object item) {
-        var folder = item as SharedFolder;
-        var row = new SharedFolderRow (folder);
-
-        row.removed.connect (manager.remove_item);
-
-        return row;
-    }
-}
-
-[GtkTemplate (ui = "/org/gnome/Boxes/ui/shared-folder-popover.ui")]
-private class Boxes.SharedFolderPopover: Gtk.Popover {
-    public signal bool saved (string path, string name);
-
-    [GtkChild]
-    public unowned Gtk.FileChooserButton file_chooser_button;
-    [GtkChild]
-    public unowned Gtk.Entry name_entry;
-
-    construct {
-        var default_path = Environment.get_user_special_dir (UserDirectory.PUBLIC_SHARE);
-        file_chooser_button.set_current_folder (default_path);
-    }
-
-    [GtkCallback]
-    public void on_cancel (Gtk.Button cancel_button) {
-        popdown ();
-    }
-
-    [GtkCallback]
-    public void on_save (Gtk.Button save_button) {
-        var uri = file_chooser_button.get_uri ();
-        File file = File.new_for_uri (uri);
-        var name = name_entry.get_text ();
-
-        if (uri != null) {
-            if (name == "")
-                name = file.get_basename ();
-
-            saved (file.get_path (), name);
-        }
-
-        popdown ();
-    }
-}
diff --git a/src/spice-display.vala b/src/spice-display.vala
index f01d084e..c39aaf23 100644
--- a/src/spice-display.vala
+++ b/src/spice-display.vala
@@ -412,9 +412,6 @@ private void on_new_file_transfer (Spice.MainChannel main_channel, Object transf
             if (webdav_channel == null || !webdav_channel.port_opened)
                 break;
 
-            var frame = create_shared_folders_frame ();
-            add_property (ref list, _("Folder Shares"), new Gtk.Label (""), frame);
-
             break;
         }
 
@@ -523,10 +520,6 @@ public override void send_keys (uint[] keyvals) {
         return frame;
     }
 
-    private Gtk.Frame create_shared_folders_frame () {
-        return new SharedFoldersWidget (machine.config.uuid);
-    }
-
     private bool is_usb_kbd_or_mouse (uint8 class, uint8 subclass, uint8 protocol) {
         var ret = false;
 


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]