gwl: Avoid adding the window again after removal

anaximeno · anaximeno · commit 4fd113123a97 · 2026-02-25T12:17:42.000-01:00
This was triggering when using the show window in all workspaces option and was leading the app group representation to an irregular state where removed windows would still be counted and show irresponsible thumbnails.
diff --git a/files/usr/share/cinnamon/applets/grouped-window-list@cinnamon.org/workspace.js b/files/usr/share/cinnamon/applets/grouped-window-list@cinnamon.org/workspace.js
@@ -240,8 +240,10 @@ var Workspace = class Workspace {
         && this.state.monitorWatchList.indexOf(metaWindow.get_monitor()) > -1;
     }
 
-    windowWorkspaceChanged(display, metaWorkspace, metaWindow) {
-        this.windowAdded(metaWindow, metaWorkspace);
+    windowWorkspaceChanged(display, metaWindow, metaWorkspace) {
+        // If the window is removed the metaWorkspace will be null, in that
+        // case we wouldn't want to add the window again.
+        if (metaWorkspace) this.windowAdded(metaWorkspace, metaWindow);
     }
 
     windowAdded(metaWorkspace, metaWindow, app, isFavoriteApp) {

Original file line number	Diff line number	Diff line change
`@@ -240,8 +240,10 @@ var Workspace = class Workspace {`
`240`	`240`	`&& this.state.monitorWatchList.indexOf(metaWindow.get_monitor()) > -1;`
`241`	`241`	`}`
`242`	`242`
`243`		`- windowWorkspaceChanged(display, metaWorkspace, metaWindow) {`
`244`		`- this.windowAdded(metaWindow, metaWorkspace);`
	`243`	`+ windowWorkspaceChanged(display, metaWindow, metaWorkspace) {`
	`244`	`+ // If the window is removed the metaWorkspace will be null, in that`
	`245`	`+ // case we wouldn't want to add the window again.`
	`246`	`+ if (metaWorkspace) this.windowAdded(metaWorkspace, metaWindow);`
`245`	`247`	`}`
`246`	`248`
`247`	`249`	`windowAdded(metaWorkspace, metaWindow, app, isFavoriteApp) {`