[gnome-shell] altTab.js: remove Alt+Tab's special case



commit 092e1a691d57a3be205100f7d3910534d3c59f84
Author: Rui Matos <tiagomatos gmail com>
Date:   Wed Feb 23 10:52:25 2011 +0000

    altTab.js: remove Alt+Tab's special case
    
    Alt+Tab's special case of "switch to the most-recently-used window even if
    it's in the same app" is actually an hindrance for users to get a firm mental
    model of Alt+Tab. Now that we have Alt+Above_Tab the special case is no longer
    needed.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=647907

 js/ui/altTab.js |   22 ++--------------------
 1 files changed, 2 insertions(+), 20 deletions(-)
---
diff --git a/js/ui/altTab.js b/js/ui/altTab.js
index 54a4567..ed541b3 100644
--- a/js/ui/altTab.js
+++ b/js/ui/altTab.js
@@ -156,29 +156,11 @@ AltTabPopup.prototype = {
                     this._select(0, 0);
             }
         } else if (this._appIcons.length == 1) {
-            if (!backward && this._appIcons[0].cachedWindows.length > 1) {
-                // For compatibility with the multi-app case below
-                this._select(0, 1, true);
-            } else
-                this._select(0);
+            this._select(0);
         } else if (backward) {
             this._select(this._appIcons.length - 1);
         } else {
-            let firstWindows = this._appIcons[0].cachedWindows;
-            if (firstWindows.length > 1) {
-                let curAppNextWindow = firstWindows[1];
-                let nextAppWindow = this._appIcons[1].cachedWindows[0];
-
-                // If the next window of the current app is more-recently-used
-                // than the first window of the next app, then select it.
-                if (curAppNextWindow.get_workspace() == global.screen.get_active_workspace() &&
-                    curAppNextWindow.get_user_time() > nextAppWindow.get_user_time())
-                    this._select(0, 1, true);
-                else
-                    this._select(1);
-            } else {
-                this._select(1);
-            }
+            this._select(1);
         }
 
         // There's a race condition; if the user released Alt before



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