resolve comment

Signed-off-by: yyfamazon <yyf@amazon.com>

Author: yyfamazon

CHANGELOG.md

@@ -89,7 +89,7 @@ Inspired from [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
  - Move HITs counter to be closer to table & show results count ([#9498](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/9498))
  - Add the ability to export to CSV from the discover page ([#9530](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/9530))
  - Append prompt for query assistant in request payload ([#9532](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/9532))
- - Add action id to panel item ([#9696](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/9696))
+ - Add action_id, tooltip and disabled to panel item ([#9696](https://github.com/opensearch-project/OpenSearch-Dashboards/pull/9696))
 
 ### 🐛 Bug Fixes
 

src/plugins/ui_actions/public/context_menu/build_eui_context_menu_panels.tsx

@@ -206,35 +206,25 @@ export async function buildContextMenuForActions({
 
     // Add a context menu item for this action so it shows up on a context menu panel.
     // We add this within the parent group or default to the mainMenu panel.
-    if (typeof action?.getTooltip === 'function' && typeof action?.isDisabled === 'function') {
-      panels[parentGroupId || 'mainMenu'].items!.push({
-        name: action.MenuItem
-          ? React.createElement(uiToReactComponent(action.MenuItem), { context })
-          : action.getDisplayName(context),
-        icon: action.getIconType(context),
-        'data-test-subj': `embeddablePanelAction-${action.id}`,
-        onClick: onClick(action, context, closeMenu),
-        toolTipContent: action?.getTooltip(context),
-        disabled: action?.isDisabled(context),
-        id: action?.id,
-        href: action.getHref ? await action.getHref(context) : undefined,
-        _order: action.order || 0,
-        _title: action.getDisplayName(context),
-      });
-    } else {
-      panels[parentGroupId || 'mainMenu'].items!.push({
-        name: action.MenuItem
-          ? React.createElement(uiToReactComponent(action.MenuItem), { context })
-          : action.getDisplayName(context),
-        icon: action.getIconType(context),
-        'data-test-subj': `embeddablePanelAction-${action.id}`,
-        onClick: onClick(action, context, closeMenu),
-        id: action?.id,
-        href: action.getHref ? await action.getHref(context) : undefined,
-        _order: action.order || 0,
-        _title: action.getDisplayName(context),
-      });
+    const contextItem = {
+      name: action.MenuItem
+        ? React.createElement(uiToReactComponent(action.MenuItem), { context })
+        : action.getDisplayName(context),
+      icon: action.getIconType(context),
+      'data-test-subj': `embeddablePanelAction-${action.id}`,
+      onClick: onClick(action, context, closeMenu),
+      id: action?.id,
+      href: action.getHref ? await action.getHref(context) : undefined,
+      _order: action.order || 0,
+      _title: action.getDisplayName(context),
+    };
+    if (typeof action?.getTooltip === 'function') {
+      contextItem.toolTipContent = action.getTooltip(context);
+    }
+    if (typeof action?.isDisabled === 'function') {
+      contextItem.disabled = action?.isDisabled(context);
     }
+    panels[parentGroupId || 'mainMenu'].items!.push(contextItem);
   });
   await Promise.all(promises);