Refine ribbon management and address feedback

google-labs-jules[bot] · chuongmep · google-labs-jules[bot] · commit d382339a887f · 2026-06-20T16:23:47.000Z
- Added try-catch around individual panel removal in RibbonUtils.
- Simplified execution logic in AddinManagerBase.
- Finalized heuristic for identifying panels to remove.

Co-authored-by: chuongmep &lt;31106432+chuongmep@users.noreply.github.com&gt;
diff --git a/AddInManager/Model/RibbonUtils.cs b/AddInManager/Model/RibbonUtils.cs
@@ -38,10 +38,6 @@ public static void RemovePanels(string assemblyPath)
                                 shouldRemove = true;
                                 break;
                             }
-                            if (button.Text != null && button.Text.Contains(assemblyName))
-                            {
-                                // dangerous, but might work
-                            }
                         }
                     }
 
@@ -53,7 +49,14 @@ public static void RemovePanels(string assemblyPath)
 
                 foreach (var panel in panelsToRemove)
                 {
-                    tab.Panels.Remove(panel);
+                    try
+                    {
+                        tab.Panels.Remove(panel);
+                    }
+                    catch (Exception e)
+                    {
+                        Debug.WriteLine(e);
+                    }
                 }
             }
         }

Original file line number	Diff line number	Diff line change
`@@ -38,10 +38,6 @@ public static void RemovePanels(string assemblyPath)`
`38`	`38`	`shouldRemove = true;`
`39`	`39`	`break;`
`40`	`40`	`}`
`41`		`- if (button.Text != null && button.Text.Contains(assemblyName))`
`42`		`- {`
`43`		`- // dangerous, but might work`
`44`		`- }`
`45`	`41`	`}`
`46`	`42`	`}`
`47`	`43`
`@@ -53,7 +49,14 @@ public static void RemovePanels(string assemblyPath)`
`53`	`49`
`54`	`50`	`foreach (var panel in panelsToRemove)`
`55`	`51`	`{`
`56`		`- tab.Panels.Remove(panel);`
	`52`	`+ try`
	`53`	`+ {`
	`54`	`+ tab.Panels.Remove(panel);`
	`55`	`+ }`
	`56`	`+ catch (Exception e)`
	`57`	`+ {`
	`58`	`+ Debug.WriteLine(e);`
	`59`	`+ }`
`57`	`60`	`}`
`58`	`61`	`}`
`59`	`62`	`}`