Add option to remove anchor arg from context menus

eclipse-theia · Jan 12, 2022 · 88fb763 · 88fb763
1 parent dd900be
commit 88fb763
Show file tree

Hide file tree

Showing 4 changed files with 20 additions and 7 deletions.
diff --git a/packages/core/src/browser/common-frontend-contribution.ts b/packages/core/src/browser/common-frontend-contribution.ts
@@ -472,7 +472,7 @@ export class CommonFrontendContribution implements FrontendApplicationContributi
                         id: mainMenuId,
                         iconClass: 'codicon codicon-menu',
                         title: nls.localizeByDefault('Application Menu'),
-                        menuPath: ['menubar'],
+                        menuPath: MAIN_MENU_BAR,
                         order: 0,
                     });
                 } else {

diff --git a/packages/core/src/browser/context-menu-renderer.ts b/packages/core/src/browser/context-menu-renderer.ts
@@ -94,10 +94,17 @@ export abstract class ContextMenuRenderer {
 }
 
 export interface RenderContextMenuOptions {
-    menuPath: MenuPath
-    anchor: Anchor
-    args?: any[]
-    onHide?: () => void
+    menuPath: MenuPath;
+    anchor: Anchor;
+    args?: any[];
+    /**
+     * Whether the anchor should be passed as an argument to the handlers of commands for this context menu.
+     * If true, the anchor will be appended to the list of arguments or passed as the only argument if no other
+     * arguments are supplied.
+     * Default is `true`.
+     */
+    includeAnchorArg?: boolean;
+    onHide?: () => void;
 }
 export namespace RenderContextMenuOptions {
     export function resolve(menuPathOrOptions: MenuPath | RenderContextMenuOptions, anchor?: Anchor, onHide?: () => void): RenderContextMenuOptions {
@@ -110,7 +117,10 @@ export namespace RenderContextMenuOptions {
             menuPath = menuPathOrOptions.menuPath;
             anchor = menuPathOrOptions.anchor;
             onHide = menuPathOrOptions.onHide;
-            args = menuPathOrOptions.args ? [...menuPathOrOptions.args, anchor] : [anchor];
+            args = menuPathOrOptions.args ? menuPathOrOptions.args.slice() : [];
+            if (menuPathOrOptions.includeAnchorArg !== false) {
+                args.push(anchor);
+            }
         }
         return {
             menuPath,

diff --git a/packages/core/src/browser/shell/sidebar-bottom-menu-widget.tsx b/packages/core/src/browser/shell/sidebar-bottom-menu-widget.tsx
@@ -28,6 +28,7 @@ export class SidebarBottomMenuWidget extends SidebarMenuWidget {
         const button = e.currentTarget.getBoundingClientRect();
         this.contextMenuRenderer.render({
             menuPath,
+            includeAnchorArg: false,
             anchor: {
                 x: button.left + button.width,
                 y: button.top + button.height,

diff --git a/packages/core/src/browser/shell/sidebar-menu-widget.tsx b/packages/core/src/browser/shell/sidebar-menu-widget.tsx
@@ -55,6 +55,7 @@ export class SidebarMenuWidget extends ReactWidget {
       return;
     }
     this.menus.push(menu);
+    this.menus.sort((a, b) => a.order - b.order);
     this.update();
   }
 
@@ -73,6 +74,7 @@ export class SidebarMenuWidget extends ReactWidget {
     const button = e.currentTarget.getBoundingClientRect();
     this.contextMenuRenderer.render({
       menuPath,
+      includeAnchorArg: false,
       anchor: {
         x: button.left + button.width,
         y: button.top,
@@ -82,7 +84,7 @@ export class SidebarMenuWidget extends ReactWidget {
 
   protected render(): React.ReactNode {
     return <React.Fragment>
-      {this.menus.sort((a, b) => a.order - b.order).map(menu => <i
+      {this.menus.map(menu => <i
         key={menu.id}
         className={menu.iconClass}
         title={menu.title}