fix: 修复右键菜单多开问题

LiuYunPlayer · LiuYunPlayer · commit f2dd7d96fbac · 2025-03-01T00:50:15.000+08:00
diff --git a/TuneLab/UI/MainWindow/Editor/PianoWindow/PianoScrollView/PianoScrollViewOperation.cs b/TuneLab/UI/MainWindow/Editor/PianoWindow/PianoScrollView/PianoScrollViewOperation.cs
@@ -138,7 +138,7 @@ bool DetectWaveformPrimaryButton()
                                                 });
                                                 menu.Items.Add(menuItem);
                                             }
-                                            menu.Open(this);
+                                            this.OpenContextMenu(menu);
                                         }
                                     }
                                     else
@@ -315,10 +315,7 @@ bool DetectWaveformPrimaryButton()
                                         }
                                     }
 
-                                    if (menu.ItemCount != 0)
-                                    {
-                                        menu.Open(this);
-                                    }
+                                    this.OpenContextMenu(menu);
                                 }
                                 break;
                             default:
@@ -503,10 +500,7 @@ bool DetectWaveformPrimaryButton()
                                         }
                                     }
 
-                                    if (menu.ItemCount != 0)
-                                    {
-                                        menu.Open(this);
-                                    }
+                                    this.OpenContextMenu(menu);
                                 }
                                 break;
                             default:
diff --git a/TuneLab/UI/MainWindow/Editor/TimelineView/TimelineViewOperation.cs b/TuneLab/UI/MainWindow/Editor/TimelineView/TimelineViewOperation.cs
@@ -94,7 +94,7 @@ protected override void OnMouseDown(MouseDownEventArgs e)
                                     });
                                     menu.Items.Add(menuItem);
                                 }
-                                menu.Open(this);
+                                this.OpenContextMenu(menu);
                             }
                             else if (item is TimeSignatureItem timeSignatureItem)
                             {
@@ -115,7 +115,7 @@ protected override void OnMouseDown(MouseDownEventArgs e)
                                     });
                                     menu.Items.Add(menuItem);
                                 }
-                                menu.Open(this);
+                                this.OpenContextMenu(menu);
                             }
                             else
                             {
@@ -141,7 +141,7 @@ protected override void OnMouseDown(MouseDownEventArgs e)
                                     });
                                     menu.Items.Add(menuItem);
                                 }
-                                menu.Open(this);
+                                this.OpenContextMenu(menu);
                             }
                         }
                         break;
diff --git a/TuneLab/UI/MainWindow/Editor/TrackWindow/TrackScrollView/TrackScrollViewOperation.cs b/TuneLab/UI/MainWindow/Editor/TrackWindow/TrackScrollView/TrackScrollViewOperation.cs
@@ -285,10 +285,7 @@ protected override async void OnMouseDown(MouseDownEventArgs e)
                                 }
                             }
 
-                            if (menu.ItemCount != 0)
-                            {
-                                menu.Open(this);
-                            }
+                            this.OpenContextMenu(menu);
                         }
                         break;
                     default:
diff --git a/TuneLab/Utils/Extensions.cs b/TuneLab/Utils/Extensions.cs
@@ -338,6 +338,20 @@ public static void SetupToolTip(this Control control, string text, PlacementMode
         ToolTip.SetTip(control, text);
     }
 
+    static ContextMenu? mCurrentContextMenu = null;
+    public static void OpenContextMenu(this Control control, ContextMenu menu)
+    {
+        if (mCurrentContextMenu != null)
+            mCurrentContextMenu.Close();
+
+        if (menu.ItemCount == 0)
+            return;
+
+        mCurrentContextMenu = menu;
+        menu.Closed += (_, _) => mCurrentContextMenu = null;
+        menu.Open(control);
+    }
+
     public static Color Lerp(this Color c1, Color c2, double ratio)
     {
         return new Color(

Original file line number	Diff line number	Diff line change
`@@ -138,7 +138,7 @@ bool DetectWaveformPrimaryButton()`
`138`	`138`	`});`
`139`	`139`	`menu.Items.Add(menuItem);`
`140`	`140`	`}`
`141`		`- menu.Open(this);`
	`141`	`+ this.OpenContextMenu(menu);`
`142`	`142`	`}`
`143`	`143`	`}`
`144`	`144`	`else`
`@@ -315,10 +315,7 @@ bool DetectWaveformPrimaryButton()`
`315`	`315`	`}`
`316`	`316`	`}`
`317`	`317`
`318`		`- if (menu.ItemCount != 0)`
`319`		`- {`
`320`		`- menu.Open(this);`
`321`		`- }`
	`318`	`+ this.OpenContextMenu(menu);`
`322`	`319`	`}`
`323`	`320`	`break;`
`324`	`321`	`default:`
`@@ -503,10 +500,7 @@ bool DetectWaveformPrimaryButton()`
`503`	`500`	`}`
`504`	`501`	`}`
`505`	`502`
`506`		`- if (menu.ItemCount != 0)`
`507`		`- {`
`508`		`- menu.Open(this);`
`509`		`- }`
	`503`	`+ this.OpenContextMenu(menu);`
`510`	`504`	`}`
`511`	`505`	`break;`
`512`	`506`	`default:`
Original file line number	Diff line number	Diff line change
`@@ -94,7 +94,7 @@ protected override void OnMouseDown(MouseDownEventArgs e)`
`94`	`94`	`});`
`95`	`95`	`menu.Items.Add(menuItem);`
`96`	`96`	`}`
`97`		`- menu.Open(this);`
	`97`	`+ this.OpenContextMenu(menu);`
`98`	`98`	`}`
`99`	`99`	`else if (item is TimeSignatureItem timeSignatureItem)`
`100`	`100`	`{`
`@@ -115,7 +115,7 @@ protected override void OnMouseDown(MouseDownEventArgs e)`
`115`	`115`	`});`
`116`	`116`	`menu.Items.Add(menuItem);`
`117`	`117`	`}`
`118`		`- menu.Open(this);`
	`118`	`+ this.OpenContextMenu(menu);`
`119`	`119`	`}`
`120`	`120`	`else`
`121`	`121`	`{`
`@@ -141,7 +141,7 @@ protected override void OnMouseDown(MouseDownEventArgs e)`
`141`	`141`	`});`
`142`	`142`	`menu.Items.Add(menuItem);`
`143`	`143`	`}`
`144`		`- menu.Open(this);`
	`144`	`+ this.OpenContextMenu(menu);`
`145`	`145`	`}`
`146`	`146`	`}`
`147`	`147`	`break;`
Original file line number	Diff line number	Diff line change
`@@ -285,10 +285,7 @@ protected override async void OnMouseDown(MouseDownEventArgs e)`
`285`	`285`	`}`
`286`	`286`	`}`
`287`	`287`
`288`		`- if (menu.ItemCount != 0)`
`289`		`- {`
`290`		`- menu.Open(this);`
`291`		`- }`
	`288`	`+ this.OpenContextMenu(menu);`
`292`	`289`	`}`
`293`	`290`	`break;`
`294`	`291`	`default:`