Use same autosave heuristic when closing widget

colin-grant-work · colin-grant-work · commit 2b07a76b0376 · 2025-04-22T17:57:39.000-07:00
diff --git a/packages/core/src/browser/saveable-service.ts b/packages/core/src/browser/saveable-service.ts
@@ -222,7 +222,8 @@ export class SaveableService implements FrontendApplicationContribution {
         if (!Saveable.isDirty(widget)) {
             return false;
         }
-        if (this.autoSave !== 'off') {
+        const saveable = Saveable.get(widget);
+        if (this.autoSave !== 'off' && (!saveable || this.shouldAutoSave(widget, saveable))) {
             return true;
         }
         const notLastWithDocument = !Saveable.closingWidgetWouldLoseSaveable(widget, Array.from(this.saveThrottles.keys()));

Original file line number	Diff line number	Diff line change
`@@ -222,7 +222,8 @@ export class SaveableService implements FrontendApplicationContribution {`
`222`	`222`	`if (!Saveable.isDirty(widget)) {`
`223`	`223`	`return false;`
`224`	`224`	`}`
`225`		`- if (this.autoSave !== 'off') {`
	`225`	`+ const saveable = Saveable.get(widget);`
	`226`	`+ if (this.autoSave !== 'off' && (!saveable \|\| this.shouldAutoSave(widget, saveable))) {`
`226`	`227`	`return true;`
`227`	`228`	`}`
`228`	`229`	`const notLastWithDocument = !Saveable.closingWidgetWouldLoseSaveable(widget, Array.from(this.saveThrottles.keys()));`