From c661baadd9683c0033cd2a6ad90157c6d099a6c2 Mon Sep 17 00:00:00 2001 From: kijai <40791699+kijai@users.noreply.github.com> Date: Wed, 5 Nov 2025 18:19:07 +0200 Subject: [PATCH] Fix context too eager preventdefault for Points/Spline editors This was blocking context menu for some other elements. --- web/js/point_editor.js | 3 +++ web/js/spline_editor.js | 3 +++ 2 files changed, 6 insertions(+) diff --git a/web/js/point_editor.js b/web/js/point_editor.js index b06ed75..dc23682 100644 --- a/web/js/point_editor.js +++ b/web/js/point_editor.js @@ -661,7 +661,10 @@ class PointsEditor { createContextMenu = () => { self = this; document.addEventListener('contextmenu', function (e) { + if (e.target.closest(`#points-editor-${self.node.uuid}`) || + e.target.closest('#context-menu')) { e.preventDefault(); + } }); document.addEventListener('click', function (e) { diff --git a/web/js/spline_editor.js b/web/js/spline_editor.js index 3d358a2..0b9163e 100644 --- a/web/js/spline_editor.js +++ b/web/js/spline_editor.js @@ -1024,7 +1024,10 @@ this.lastMousePosition = { x: this.width/2, y: this.height/2 }; this.node.contextMenu = newMenu; document.addEventListener('contextmenu', function (e) { + if (e.target.closest(`#points-editor-${self.node.uuid}`) || + e.target.closest('#context-menu')) { e.preventDefault(); + } }); document.addEventListener('click', function (e) {