feat: Prompt when deleting workspace

Floorp-Projects · Nov 30, 2024 · 6dd9334 · 6dd9334
1 parent 4895f2f
commit 6dd9334
Showing 1 changed file with 16 additions and 8 deletions.
diff --git a/browser/base/content/browser-workspaces.mjs b/browser/base/content/browser-workspaces.mjs
@@ -552,15 +552,23 @@ export const gWorkspaces = {
   async deleteWorkspace(workspaceId) {
     let windowId = await this.getCurrentWindowId();
     let currentWorkspaceId = await this.getCurrentWorkspaceId();
-    await WorkspacesService.deleteWorkspace(workspaceId, windowId);
-    this.removeWorkspaceTabs(workspaceId);
-    if (workspaceId == currentWorkspaceId) {
-      await this.changeWorkspace(
-        await WorkspacesWindowIdUtils.getDefaultWorkspaceId(windowId),
-        0
-      );
+    const workspace = await this.getWorkspaceById(workspaceId);
+    const result = Services.prompt.confirm(
+      window,
+      this.l10n.formatValueSync("delete-workspace-prompt-title"),
+      this.l10n.formatValueSync("delete-workspace-prompt-text", { name: workspace.name }),
+    );
+    if (result) {
+      await WorkspacesService.deleteWorkspace(workspaceId, windowId);
+      this.removeWorkspaceTabs(workspaceId);
+      if (workspaceId == currentWorkspaceId) {
+        await this.changeWorkspace(
+          await WorkspacesWindowIdUtils.getDefaultWorkspaceId(windowId),
+          0
+        );
+      }
+      this.rebuildWorkspacesToolbar();
     }
-    this.rebuildWorkspacesToolbar();
   },
 
   async renameWorkspace(workspaceId, newName) {