From 2dff65da9ccfd4595ec3e8fd3fb7a9ab81b91646 Mon Sep 17 00:00:00 2001 From: Filip131311 Date: Thu, 10 Oct 2024 22:20:32 +0200 Subject: [PATCH] init --- .../src/webview/components/Preview.tsx | 19 +++++++++++++------ 1 file changed, 13 insertions(+), 6 deletions(-) diff --git a/packages/vscode-extension/src/webview/components/Preview.tsx b/packages/vscode-extension/src/webview/components/Preview.tsx index 2896444f2..24125ca1d 100644 --- a/packages/vscode-extension/src/webview/components/Preview.tsx +++ b/packages/vscode-extension/src/webview/components/Preview.tsx @@ -413,11 +413,16 @@ function Preview({ } } - function onContextMenu(e: MouseEvent) { - e.preventDefault(); - } - useEffect(() => { + // this is a fix that disables context menu on windows https://github.com/microsoft/vscode/issues/139824 + // there is an active backlog item that changes the bahaviour of context menu, so it might not be necessery + // in the future https://github.com/microsoft/vscode/issues/225411 + function onContextMenu(e) { + e.stopImmediatePropagation(); + } + + window.addEventListener("contextmenu", onContextMenu, true); + function onBlurChange() { if (!document.hasFocus()) { setIsPanning(false); @@ -426,7 +431,10 @@ function Preview({ } } addEventListener("blur", onBlurChange, true); - return () => removeEventListener("blur", onBlurChange, true); + return () => { + window.removeEventListener("contextmenu", onContextMenu); + removeEventListener("blur", onBlurChange, true); + }; }, []); useEffect(() => { @@ -505,7 +513,6 @@ function Preview({ onMouseUp, onMouseEnter, onMouseLeave, - onContextMenu, }; const resizableProps = useResizableProps({