[gnome-boxes] Use vexpand rather than weird empty grid



commit b374eda1dde6860d7e27a14c847e696806e47eb4
Author: Alexander Larsson <alexl redhat com>
Date:   Tue Oct 30 16:27:53 2012 +0100

    Use vexpand rather than weird empty grid
    
    We don't have to use the empty grid to get an expanding
    item in the notification grid, as we can just make the
    notification itself expand (and then ensure it doesn't
    become large by using a Align.START valign.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=679106

 src/display-page.vala |   13 ++-----------
 1 files changed, 2 insertions(+), 11 deletions(-)
---
diff --git a/src/display-page.vala b/src/display-page.vala
index f6b7b3e..9863494 100644
--- a/src/display-page.vala
+++ b/src/display-page.vala
@@ -118,20 +118,11 @@ private class Boxes.DisplayPage: GLib.Object {
         notification_grid = new Grid ();
         notification_grid.valign = Gtk.Align.START;
         notification_grid.halign = Gtk.Align.CENTER;
-        notification_grid.vexpand = false;
+        notification_grid.vexpand = true;
 
-        // We need one expanding row, or otherwise in the case
-        // where overlay_toolbar_box is hidden we will get zero
-        // expanding rows, which means all empty rows get
-        // to divide all remaining space, so the notifications
-        // are centered
-        var empty_grid = new Grid ();
-        empty_grid.vexpand = true;
-
-        grid.attach (event_box, 0, 0, 1, 3);
+        grid.attach (event_box, 0, 0, 1, 2);
         grid.attach (overlay_toolbar_box, 0, 0, 1, 1);
         grid.attach (notification_grid, 0, 1, 1, 1);
-        grid.attach (empty_grid, 0, 2, 1, 1);
 
         box.show_all ();
     }



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