Resize window according to toggle state (#96)

* Hide/show sliders depending on global switch - have a nice animation and window resize doing so

* Remove resizing capabilities to allow resize according to toggle state

* avoid unclosable window

* Make the linter happy

Author: teamcons

data/com.github.elfenware.badger.gschema.xml

@@ -77,15 +77,5 @@
     path="/com/github/elfenware/badger/state/"
     id="com.github.elfenware.badger.state"
     gettext-domain="com.github.elfenware.badger">
-    <key name="window-width" type="i">
-      <default>180</default>
-      <summary>The saved width of the window.</summary>
-      <description>The saved width of the window.</description>
-    </key>
-    <key name="window-height" type="i">
-      <default>200</default>
-      <summary>The saved height of the window.</summary>
-      <description>The saved height of the window.</description>
-    </key>
   </schema>
 </schemalist>

src/MainGrid.vala

@@ -182,12 +182,9 @@ public class Badger.MainGrid : Gtk.Box {
         /**************************************************/
 
         // If the "all" flag is false, the switch is off, hide the scales 
-        Gtk.Revealer revealer = new Gtk.Revealer ();
+        this.revealer = new Gtk.Revealer ();
         revealer.set_transition_type (Gtk.RevealerTransitionType.SLIDE_DOWN);
-        settings.bind ("all", revealer, "reveal_child", SettingsBindFlags.DEFAULT);
-
         revealer.set_child (scale_box);
         append (revealer);
-
     }
 }

src/MainWindow.vala

@@ -33,15 +33,15 @@ public class Badger.MainWindow : Gtk.Window {
 
     construct {
         Intl.setlocale ();
-        settings = new GLib.Settings ("com.github.elfenware.badger.state");
+        settings = new GLib.Settings ("com.github.elfenware.badger.timers");
+
+        // We cannot resize window if it is allowed to change
+        set_size_request (12, 12);
+        set_resizable (false);
 
-        set_default_size (
-            settings.get_int ("window-width"),
-            settings.get_int ("window-height")
-        );
 
-        set_title (_("Badger"));
-        Gtk.Label title_widget = new Gtk.Label (_("Badger"));
+        set_title ("Badger");
+        Gtk.Label title_widget = new Gtk.Label (this.get_title ());
         title_widget.add_css_class (Granite.STYLE_CLASS_TITLE_LABEL);
 
         this.headerbar = new Gtk.HeaderBar ();
@@ -59,33 +59,31 @@ public class Badger.MainWindow : Gtk.Window {
 
         set_child (handle);
 
-        var global_switch = new GLib.Settings ("com.github.elfenware.badger.timers");
+        // Avoid showing the window without content despite toggle off.
+        main.revealer.reveal_child = settings.get_boolean ("all");
 
-        // Resize window when content gets hidden
-        global_switch.notify["all"].connect (() => {
-            if (global_switch.get_boolean ("all") == false) {
-                this.height_request = 0;
-                this.default_height = 0;
-                print ("eugh");
-            }
-        });
+        // Resize window accordingly to state of global switch
+        settings.changed["all"].connect ( on_toggle_changed);
 
-        // save state
+        // Avoid a bug whence reopened windows cannot be closed
         close_request.connect (e => {
             return before_destroy ();
         });
     }
 
+    private void on_toggle_changed () {
+        debug ("\nToggle changed!");
+        main.revealer.reveal_child = settings.get_boolean ("all");
 
-    // save state
-    private bool before_destroy () {
-        int width, height;
-
-        get_default_size (out width, out height);
-
-        settings.set_int ("window-width", width);
-        settings.set_int ("window-height", height);
+        if (!settings.get_boolean ("all")) {
+            debug ("\nToggle is off! Resizing window");
+            set_size_request (12, 12);
+            queue_resize ();
+        }
+    }
 
+    // Avoid a bug whence reopened windows cannot be closed
+    private bool before_destroy () {
         hide ();
         return true;
     }