[gnome-shell/wip/overviewTransitions: 1/5] viewSelector: Remove duplicate call to showPage



commit 00956b8dd8afaf9a8dc20613252995705ad7206a
Author: Carlos Soriano <carlos soriano89 gmail com>
Date:   Mon Jul 14 18:55:34 2014 +0200

    viewSelector: Remove duplicate call to showPage
    
    We were caling twice showPage() with the correct
    page, here and in show() / animateFromOverview given
    that _resetShowAppsbutton was called from the signal
    'showing' of overview.
    Also, this call here doesn't belong at all to reseting
    the button.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=732901

 js/ui/viewSelector.js |    3 +--
 1 files changed, 1 insertions(+), 2 deletions(-)
---
diff --git a/js/ui/viewSelector.js b/js/ui/viewSelector.js
index 778bb57..8b1691b 100644
--- a/js/ui/viewSelector.js
+++ b/js/ui/viewSelector.js
@@ -319,6 +319,7 @@ const ViewSelector = new Lang.Class({
     },
 
     zoomFromOverview: function() {
+        this._showPage(this._workspacesPage);
         this._workspacesDisplay.zoomFromOverview();
 
         if (!this._workspacesDisplay.activeWorkspaceHasMaximizedWindows())
@@ -409,8 +410,6 @@ const ViewSelector = new Lang.Class({
         this._showAppsBlocked = true;
         this._showAppsButton.checked = false;
         this._showAppsBlocked = false;
-
-        this._showPage(this._workspacesPage, true);
     },
 
     _onStageKeyPress: function(actor, event) {


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