fix(workspace): properly quit GTK main loop when closing last webview

Replace window.Close() with webkit.QuitMainLoop() in cleanupAndExit
to ensure graceful application shutdown when the last pane is closed
via pane mode (Ctrl+P x). This ensures the GTK main loop terminates
properly instead of leaving the application running without windows.

commit: 81fd4c55a2b9f0b15f57eea2d99bed082ab1be18
author: bamen · 12mo ago · 2025-10-14 18:20 UTC
parent: e4cf6a4
signature: unsigned

1 file · +3 -5

M internal/app/browser/workspace_pane_ops.go +3 -5


@@ -908,11 +908,9 @@ func (wm WorkspaceManager) cleanupAndExit(node paneNode) (*paneNode, error) {
908	908		log.Printf("[workspace] closing final pane; exiting browser")
909	909		wm.cleanupPane(node, wm.nextCleanupGeneration())
910	910		wm.detachHover(node)
911		-	// TODO: Implement proper application quit mechanism
912		-	// For now, the window close will trigger GTK application quit
913		-	if wm.window != nil && wm.window.AsWindow() != nil {
914		-	wm.window.AsWindow().Close()
915		-	}
	911	+
	912	+	// Gracefully quit the GTK main loop
	913	+	webkit.QuitMainLoop()
916	914		return nil, nil
917	915		}
918	916


@@ -908,11 +908,9 @@ func (wm WorkspaceManager) cleanupAndExit(node paneNode) (*paneNode, error) {
908		log.Printf("[workspace] closing final pane; exiting browser")	908		log.Printf("[workspace] closing final pane; exiting browser")
909		wm.cleanupPane(node, wm.nextCleanupGeneration())	909		wm.cleanupPane(node, wm.nextCleanupGeneration())
910		wm.detachHover(node)	910		wm.detachHover(node)
911	-	// TODO: Implement proper application quit mechanism	911	+
912	-	// For now, the window close will trigger GTK application quit	912	+	// Gracefully quit the GTK main loop
913	-	if wm.window != nil && wm.window.AsWindow() != nil {	913	+	webkit.QuitMainLoop()
914	-	wm.window.AsWindow().Close()
915	-	}
916		return nil, nil	914		return nil, nil
917		}	915		}
918			916