Merge pull request #1603 from specklesystems/hotfix/gh/duplicate-top-…

…menu fix(gh): Unload top-menu if it exists before loading
specklesystems · Sep 9, 2022 · 1bda247 · 1bda247
1 parent b95647e
commit 1bda247
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/ConnectorGrasshopper/ConnectorGrasshopper/Loader.cs b/ConnectorGrasshopper/ConnectorGrasshopper/Loader.cs
@@ -1,4 +1,4 @@
-using System;
+using System;
 using System.Collections.Generic;
 using System.Diagnostics;
 using System.Linq;
@@ -120,8 +120,12 @@ private void HandleKitSelectedEvent(object sender, EventArgs args)
     private void AddSpeckleMenu(object sender, ElapsedEventArgs e)
     {
       if (Grasshopper.Instances.DocumentEditor == null || MenuHasBeenAdded) return;
+      var mainMenu = Grasshopper.Instances.DocumentEditor.MainMenuStrip;
+      var menuName = "Speckle 2";
+      if (mainMenu.Items.ContainsKey(menuName))
+        mainMenu.Items.RemoveByKey(menuName);
 
-      speckleMenu = new ToolStripMenuItem("Speckle 2");
+      speckleMenu = new ToolStripMenuItem(menuName);
 
       var kitHeader = speckleMenu.DropDown.Items.Add("Select the converter you want to use.");
       kitHeader.Enabled = false;
@@ -173,7 +177,6 @@ private void AddSpeckleMenu(object sender, ElapsedEventArgs e)
 
       try
       {
-        var mainMenu = Grasshopper.Instances.DocumentEditor.MainMenuStrip;
         Grasshopper.Instances.DocumentEditor.Invoke(new Action(() =>
         {
           if (!MenuHasBeenAdded)