Skip to content

[8.x] [Obs AI Assistant] Remove the copy button if there is no content to copy (#199064)#199874

Merged
kibanamachine merged 1 commit intoelastic:8.xfrom
kibanamachine:backport/8.x/pr-199064
Nov 12, 2024
Merged

[8.x] [Obs AI Assistant] Remove the copy button if there is no content to copy (#199064)#199874
kibanamachine merged 1 commit intoelastic:8.xfrom
kibanamachine:backport/8.x/pr-199064

Conversation

@kibanamachine
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 8.x:

Questions ?

Please refer to the Backport tool documentation

…opy (elastic#199064)

Closes elastic#196986

## Summary

### Problem
The copy button for a visualization does nothing, as there's no content
to copy. When clicked on this button, a message is shown saying "Copied
Message", but nothing is copied.

### Solution
Remove the copy button, when there's no content to copy.

### Checklist

- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios

(cherry picked from commit a613277)
@kibanamachine kibanamachine merged commit c938f31 into elastic:8.x Nov 12, 2024
@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
observabilityAIAssistantApp 239.2KB 239.2KB +3.0B
searchAssistant 162.0KB 162.0KB +3.0B
total +6.0B

cc @viduni94

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants