Fix menu position

anmcgrath · May 27, 2024 · bc4e9a0 · bc4e9a0
1 parent bbcbeec
commit bc4e9a0
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 7 deletions.
diff --git a/src/BlazorDatasheet/Menu/MenuShowOptions.cs b/src/BlazorDatasheet/Menu/MenuShowOptions.cs
@@ -8,14 +8,17 @@ public class MenuShowOptions
     public string Placement { get; set; }
     public double Margin { get; set; }
     public string Trigger { get; }
-    public MouseEventArgs Args { get; }
+    public double ClientX { get; }
+    public double ClientY { get; }
 
-    public MenuShowOptions(string targetId, string placement, double margin, string trigger, MouseEventArgs args)
+    public MenuShowOptions(string targetId, string placement, double margin, string trigger, double clientX,
+        double clientY)
     {
         TargetId = targetId;
         Placement = placement;
         Margin = margin;
         Trigger = trigger;
-        Args = args;
+        ClientX = clientX;
+        ClientY = clientY;
     }
 }
diff --git a/src/BlazorDatasheet/Menu/SheetMenuTarget.razor b/src/BlazorDatasheet/Menu/SheetMenuTarget.razor
@@ -76,7 +76,7 @@
         if (string.IsNullOrEmpty(MenuId))
             return;
 
-        await MenuService.ShowMenu(MenuId, new MenuShowOptions(TargetId, Placement, Margin, trigger, args), MenuData);
+        await MenuService.ShowMenu(MenuId, new MenuShowOptions(TargetId, Placement, Margin, trigger, args.ClientX, args.ClientY), MenuData);
     }
 
 }
diff --git a/src/BlazorDatasheet/wwwroot/js/menu.js b/src/BlazorDatasheet/wwwroot/js/menu.js
@@ -63,8 +63,6 @@ class MenuService {
     showMenuEl(menuEl, options) {
         if (this.isActive(menuEl))
             return
-
-        console.log(options)
 
         // run with set timeout to allow the updated menu to be structured based on context
         setTimeout(() => {
@@ -84,7 +82,7 @@ class MenuService {
 
             menuEl.addEventListener('toggle', onToggle)
             if (options.trigger === 'oncontextmenu') {
-                let rect = new DOMRect(options.args.clientX, options.args.clientY, 1, 1)
+                let rect = new DOMRect(options.clientX, options.clientY, 1, 1)
                 this.positionMenu(menuEl, rect, options.margin, options.placement)
             } else if (options.targetId) {
                 let targetEl = document.getElementById(options.targetId)