[gnome-shell/wip/grab-helper-top-panel: 12/20] viewSelector: Set the active page immediately



commit 7d693cbd1715a41603bc3e2eb40e5060a6cc796e
Author: Jasper St. Pierre <jstpierre mecheye net>
Date:   Sun Nov 25 23:45:51 2012 -0500

    viewSelector: Set the active page immediately
    
    This prevents some focus management issues.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=689062

 js/ui/viewSelector.js |    5 +++--
 1 files changed, 3 insertions(+), 2 deletions(-)
---
diff --git a/js/ui/viewSelector.js b/js/ui/viewSelector.js
index 9c73e8e..bf71e46 100644
--- a/js/ui/viewSelector.js
+++ b/js/ui/viewSelector.js
@@ -211,18 +211,19 @@ const ViewSelector = new Lang.Class({
             return;
 
         if(this._activePage) {
+            let oldPage = this._activePage;
             Tweener.addTween(this._activePage,
                              { opacity: 0,
                                time: 0.1,
                                transition: 'easeOutQuad',
                                onComplete: Lang.bind(this,
                                    function() {
-                                       this._activePage.hide();
-                                       this._activePage = page;
+                                       oldPage.hide();
                                    })
                              });
         }
 
+        this._activePage = page;
         page.show();
         Tweener.addTween(page,
                          { opacity: 255,



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