[gnome-shell-extensions/gnome-3-26] auto-move-windows: Minor style cleanups



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]