webui: add JavaScript alert for file save failures

josharian · sketch · josharian · commit 7cae9d5d4c62 · 2025-07-25T13:36:14.000-07:00
Now that Philip fixed #213, this should be safe to unleash on everyone. JavaScript alerts are ugly, but: - this shouldn't happen - if it does, the user should really know that their edits are in peril Co-Authored-By: sketch <hello@sketch.dev> Change-ID: s95d29aed61ef60f6k
diff --git a/webui/src/web-components/sketch-diff2-view.ts b/webui/src/web-components/sketch-diff2-view.ts
@@ -114,9 +114,9 @@ export class SketchDiff2View extends SketchTailwindElement {
         console.log(`File saved: ${path}`);
         (monacoView as any).notifySaveComplete(true);
       } catch (error) {
-        console.error(
-          `Error saving file: ${error instanceof Error ? error.message : String(error)}`,
-        );
+        const errorMessage =
+          error instanceof Error ? error.message : String(error);
+        alert(`Failed to save changes to ${path}:\n\n${errorMessage}`);
         (monacoView as any).notifySaveComplete(false);
       }
     } catch (error) {
