webui: add JavaScript alert for file save failures
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
index 03aaa3f..8dbf5db 100644
--- a/webui/src/web-components/sketch-diff2-view.ts
+++ b/webui/src/web-components/sketch-diff2-view.ts
@@ -114,9 +114,9 @@
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) {