[gnome-shell-extensions/gnome-3-26] auto-move-windows: Minor style cleanups
- From: Florian Müllner <fmuellner src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gnome-shell-extensions/gnome-3-26] auto-move-windows: Minor style cleanups
- Date: Thu, 18 Jan 2018 12:50:16 +0000 (UTC)
commit df8bf0cd6a41fa7fe78810b5a072c6a564618304
Author: Florian Müllner <fmuellner gnome org>
Date: Fri Dec 22 01:58:12 2017 +0100
auto-move-windows: Minor style cleanups
Use camelCase for variable names, call the first index variable 'i'
instead of 'j', and fix some whitespace errors.
https://gitlab.gnome.org/GNOME/gnome-shell-extensions/issues/33
extensions/auto-move-windows/extension.js | 16 ++++++++--------
1 file changed, 8 insertions(+), 8 deletions(-)
---
diff --git a/extensions/auto-move-windows/extension.js b/extensions/auto-move-windows/extension.js
index 94620c0..8915747 100644
--- a/extensions/auto-move-windows/extension.js
+++ b/extensions/auto-move-windows/extension.js
@@ -35,8 +35,8 @@ const WindowMover = new Lang.Class({
},
_ensureAtLeastWorkspaces: function(num, window) {
- for (let j = global.screen.n_workspaces; j <= num; j++) {
- window.change_workspace_by_index(j-1, false);
+ for (let i = global.screen.n_workspaces; i <= num; i++) {
+ window.change_workspace_by_index(i - 1, false);
global.screen.append_new_workspace(false, 0);
}
},
@@ -60,16 +60,16 @@ const WindowMover = new Lang.Class({
return;
}
let app_id = app.get_id();
- for ( let j = 0 ; j < spaces.length; j++ ) {
- let apps_to_space = spaces[j].split(":");
+ for (let i = 0; i < spaces.length; i++) {
+ let appsToSpace = spaces[i].split(":");
// Match application id
- if (apps_to_space[0] == app_id) {
- let workspace_num = parseInt(apps_to_space[1]) - 1;
+ if (appsToSpace[0] == app_id) {
+ let workspaceNum = parseInt(appsToSpace[1]) - 1;
- if (workspace_num >= global.screen.n_workspaces)
+ if (workspaceNum >= global.screen.n_workspaces)
this._ensureAtLeastWorkspaces(workspace_num, window);
- window.change_workspace_by_index(workspace_num, false);
+ window.change_workspace_by_index(workspaceNum, false);
}
}
}
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]