[Obs AI Assistant] Remove the copy button if there is no content to c…

…opy (elastic#196986)
viduni94 · Nov 12, 2024 · ec86c25 · ec86c25
1 parent 4a2de76
commit ec86c25
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/x-pack/packages/kbn-ai-assistant/src/chat/chat_item.tsx b/x-pack/packages/kbn-ai-assistant/src/chat/chat_item.tsx
@@ -179,6 +179,7 @@ export function ChatItem({
           onCopyToClipboard={handleCopyToClipboard}
           onToggleEdit={handleToggleEdit}
           onToggleExpand={handleToggleExpand}
+          content={content}
         />
       }
       className={

diff --git a/x-pack/packages/kbn-ai-assistant/src/chat/chat_item_actions.tsx b/x-pack/packages/kbn-ai-assistant/src/chat/chat_item_actions.tsx
@@ -8,6 +8,7 @@
 import React, { useEffect, useState } from 'react';
 import { i18n } from '@kbn/i18n';
 import { EuiButtonIcon, EuiPopover, EuiText } from '@elastic/eui';
+import type { Message } from '@kbn/observability-ai-assistant-plugin/common';
 
 export function ChatItemActions({
   canCopy,
@@ -18,6 +19,7 @@ export function ChatItemActions({
   onToggleEdit,
   onToggleExpand,
   onCopyToClipboard,
+  content,
 }: {
   canCopy: boolean;
   canEdit: boolean;
@@ -27,6 +29,7 @@ export function ChatItemActions({
   onToggleEdit: () => void;
   onToggleExpand: () => void;
   onCopyToClipboard: () => void;
+  content: Message['message']['content'];
 }) {
   const [isPopoverOpen, setIsPopoverOpen] = useState<string | undefined>();
 
@@ -70,7 +73,7 @@ export function ChatItemActions({
         />
       ) : null}
 
-      {canCopy ? (
+      {canCopy && content ? (
         <EuiPopover
           button={
             <EuiButtonIcon