[gnome-shell/wip/reorg: 10/15] sessionList: always allocate full height

[Ray Strode <halfline src gnome org>]

commit 9a0d0d2fd3c3759911667e9127031cbb7b3663dc
Author: Ray Strode <rstrode redhat com>
Date:   Wed Jun 19 09:15:05 2013 -0400

    sessionList: always allocate full height
    
    This helps to prevent stuff from jumping around on the login dialog
    when the session list opens.

 js/ui/auth/sessionList.js |   28 +++++++++++++++++++++++++---
 1 files changed, 25 insertions(+), 3 deletions(-)
---
diff --git a/js/ui/auth/sessionList.js b/js/ui/auth/sessionList.js
index 8ef4ded..5066018 100644
--- a/js/ui/auth/sessionList.js
+++ b/js/ui/auth/sessionList.js
@@ -117,17 +117,29 @@ const SessionList = new Lang.Class({
         this._itemList = new St.BoxLayout({ style_class: 'login-dialog-session-item-list',
                                             vertical: true });
         this._scrollView.add_actor(this._itemList);
-        this._scrollView.hide();
+        this._hideSessions();
         this.isOpen = false;
         this._populate();
     },
 
+    _hideSessions: function() {
+        this._itemList.can_focus = false;
+        this._itemList.reactive = false;
+        this._scrollView.opacity = 0;
+    },
+
+    _showSessions: function() {
+        this._scrollView.opacity = 255;
+        this._itemList.reactive = true;
+        this._itemList.can_focus = true;
+    },
+
     open: function() {
         if (this.isOpen)
             return;
 
         this._button.add_style_pseudo_class('open');
-        this._scrollView.show();
+        this._showSessions();
         this._triangle.set_text('\u25BE');
 
         this.isOpen = true;
@@ -138,7 +150,7 @@ const SessionList = new Lang.Class({
             return;
 
         this._button.remove_style_pseudo_class('open');
-        this._scrollView.hide();
+        this._hideSessions();
         this._triangle.set_text('\u25B8');
 
         this.isOpen = false;
@@ -202,6 +214,16 @@ const SessionList = new Lang.Class({
                          Lang.bind(this, function() {
                              this.setActiveSession(item.id);
                          }));
+
+            item.actor.can_focus = this._itemList.can_focus;
+            let signalId = this._itemList.connect('notify::can-focus',
+                                                  Lang.bind(this, function() {
+                                                      item.actor.can_focus = this._itemList.can_focus;
+                                                  }));
+            item.actor.connect('destroy',
+                               Lang.bind(this, function() {
+                                   this._itemList.disconnect(signalId);
+                               }));
         }
     }
 });