[gnome-shell] altTab.js: remove Alt+Tab's special case
- From: Dan Winship <danw src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gnome-shell] altTab.js: remove Alt+Tab's special case
- Date: Wed, 20 Apr 2011 18:46:11 +0000 (UTC)
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]