mirror of
https://github.com/DayuanJiang/next-ai-draw-io.git
synced 2026-09-02 01:20:23 +08:00
5406778dd6
The draw.io iframe registers a window.onbeforeunload handler that returns a non-empty string whenever its internal editor.modified flag is true. After the user edits text in a shape, that flag is set and never cleared. Per Electron BrowserWindow docs, returning a non-void value from any beforeunload handler in the page tree silently cancels the window close without showing a dialog. This is what caused the X button (and Cmd+Q) to do nothing for users who had typed in a shape. Calling event.preventDefault() in will-prevent-unload tells Electron to ignore the iframe's beforeunload return value and proceed with the close. The host app already persists diagrams via autosave + visibilitychange, so the prompt was unnecessary. Verified by reproducing the bug, applying the fix, and re-testing.
96 lines
2.6 KiB
TypeScript
96 lines
2.6 KiB
TypeScript
import path from "node:path"
|
|
import { app, BrowserWindow, screen } from "electron"
|
|
|
|
let mainWindow: BrowserWindow | null = null
|
|
|
|
/**
|
|
* Get the icon path based on platform
|
|
* Note: electron-builder converts icon.png during packaging,
|
|
* but at runtime we use PNG directly - Electron handles it
|
|
*/
|
|
function getIconPath(): string | undefined {
|
|
// macOS doesn't need explicit icon - it's embedded in the app bundle
|
|
if (process.platform === "darwin" && app.isPackaged) {
|
|
return undefined
|
|
}
|
|
|
|
const iconName = "icon.png"
|
|
|
|
if (app.isPackaged) {
|
|
return path.join(process.resourcesPath, iconName)
|
|
}
|
|
|
|
// Development: use icon.png from resources
|
|
return path.join(__dirname, "../../resources/icon.png")
|
|
}
|
|
|
|
/**
|
|
* Create the main application window
|
|
*/
|
|
export function createWindow(serverUrl: string): BrowserWindow {
|
|
const { width, height } = screen.getPrimaryDisplay().workAreaSize
|
|
|
|
mainWindow = new BrowserWindow({
|
|
width: Math.min(1400, Math.floor(width * 0.9)),
|
|
height: Math.min(900, Math.floor(height * 0.9)),
|
|
minWidth: 800,
|
|
minHeight: 600,
|
|
title: "Next AI Draw.io",
|
|
icon: getIconPath(),
|
|
show: false, // Don't show until ready
|
|
webPreferences: {
|
|
preload: path.join(__dirname, "../preload/index.js"),
|
|
contextIsolation: true,
|
|
nodeIntegration: false,
|
|
sandbox: true,
|
|
webSecurity: true,
|
|
},
|
|
})
|
|
|
|
// Load the Next.js application
|
|
mainWindow.loadURL(serverUrl)
|
|
|
|
// Show window when ready to prevent flashing
|
|
mainWindow.once("ready-to-show", () => {
|
|
mainWindow?.show()
|
|
})
|
|
|
|
// Open DevTools in development
|
|
if (process.env.NODE_ENV === "development") {
|
|
mainWindow.webContents.openDevTools()
|
|
}
|
|
|
|
// Override the draw.io iframe's beforeunload handler so the window can
|
|
// close after the user edits text in a shape (fixes #815). Diagrams are
|
|
// already persisted via autosave, so the prompt is unnecessary.
|
|
mainWindow.webContents.on("will-prevent-unload", (event) => {
|
|
event.preventDefault()
|
|
})
|
|
|
|
mainWindow.on("closed", () => {
|
|
mainWindow = null
|
|
})
|
|
|
|
// Handle page title updates
|
|
mainWindow.webContents.on("page-title-updated", (event, title) => {
|
|
if (
|
|
title &&
|
|
!title.includes("localhost") &&
|
|
!title.includes("127.0.0.1")
|
|
) {
|
|
mainWindow?.setTitle(title)
|
|
} else {
|
|
event.preventDefault()
|
|
}
|
|
})
|
|
|
|
return mainWindow
|
|
}
|
|
|
|
/**
|
|
* Get the main window instance
|
|
*/
|
|
export function getMainWindow(): BrowserWindow | null {
|
|
return mainWindow
|
|
}
|