VesperOS/gnome-shell-extensions
6
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

auto-move-windows: fix off-by-one error

Browse Source 
The internal workspace index starts from 0, but the one in the
settings starts from 1.
This commit is contained in:
Giovanni Campagna
2011-02-20 18:41:47 +01:00
parent f62424e9bb
commit e7c2d2c277
1 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
extensions/auto-move-windows/extension.js
View File

@@ -55,8 +55,9 @@ WindowMover.prototype = {
let apps_to_space = spaces[j].split(":");
// Match application id
if (apps_to_space[0] == app_id) {
let workspace_num = parseInt(apps_to_space[1]);
this._ensureAtLeastWorkspaces(workspace_num);
let workspace_num = parseInt(apps_to_space[1]) - 1;
// FIXME: does not work with automatic management of workspaces
// this._ensureAtLeastWorkspaces(workspace_num);
window.change_workspace_by_index(workspace_num, false, global.get_current_time());
}