Skip to content

[ai][assistant] Refactor o11y to use new Assistant logo and beacon#204289

Merged
clintandrewhall merged 6 commits intoelastic:mainfrom
clintandrewhall:ai/assistant-logo-obs
Dec 17, 2024
Merged

[ai][assistant] Refactor o11y to use new Assistant logo and beacon#204289
clintandrewhall merged 6 commits intoelastic:mainfrom
clintandrewhall:ai/assistant-logo-obs

Conversation

@clintandrewhall
Copy link
Copy Markdown
Contributor

A follow-up to #203879

Summary

This PR integrates the new Assistant Icon, Beacon, and Avatar into solutions and packages owned by Observability. In most cases this was a 1:1 replacement, but in one case, Icon was replaced with Beacon, (to indicate loading).

Note: the scaling of the icon/avatar before was one different from EUI. The new components match EUI directly and represent a 2x scale change (e.g. 's' becomes 'l', 'm' becomes 'xl', etc).

@clintandrewhall clintandrewhall added review release_note:skip Skip the PR/issue when compiling release notes backport:skip This PR does not require backporting v8.18.0 labels Dec 13, 2024
@clintandrewhall clintandrewhall requested a review from a team as a code owner December 13, 2024 22:33
@botelastic botelastic bot added the Team:Obs AI Assistant Observability AI Assistant label Dec 13, 2024
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/obs-ai-assistant (Team:Obs AI Assistant)

@neptunian
Copy link
Copy Markdown
Contributor

Screenshot 2024-12-16 at 3 21 56 PM
  • Is it intentional that we no longer have a transparent background?
  • This looks like a pixel too high which is overlapping the navigation border below it and is a bit messy looking

@clintandrewhall
Copy link
Copy Markdown
Contributor Author

clintandrewhall commented Dec 16, 2024

@neptunian that's a mistake on my part. I'll have a fix shortly Fixed

@clintandrewhall clintandrewhall enabled auto-merge (squash) December 17, 2024 00:03
@clintandrewhall clintandrewhall merged commit 8f70a2d into elastic:main Dec 17, 2024
@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
observabilityAIAssistant 112 119 +7
observabilityAIAssistantApp 414 421 +7
total +14

Async chunks

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

id before after diff
observabilityAIAssistant 19.3KB 20.6KB +1.3KB
observabilityAIAssistantApp 294.1KB 295.1KB +971.0B
total +2.2KB

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
observabilityAIAssistant 48.4KB 48.3KB -94.0B
observabilityAIAssistantApp 12.4KB 12.8KB +357.0B
total +263.0B
Unknown metric groups

async chunk count

id before after diff
observabilityAIAssistant 2 3 +1
observabilityAIAssistantApp 7 8 +1
total +2

References to deprecated APIs

id before after diff
observabilityAIAssistantApp 7 2 -5

History

@clintandrewhall
Copy link
Copy Markdown
Contributor Author

💚 All backports created successfully

Status Branch Result
8.x

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

@clintandrewhall clintandrewhall removed the backport:skip This PR does not require backporting label Dec 18, 2024
clintandrewhall added a commit that referenced this pull request Dec 18, 2024
…con (#204289) (#204746)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[ai][assistant] Refactor o11y to use new Assistant logo and beacon
(#204289)](#204289)

<!--- Backport version: 8.9.8 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Clint Andrew
Hall","email":"clint.hall@elastic.co"},"sourceCommit":{"committedDate":"2024-12-17T01:44:32Z","message":"[ai][assistant]
Refactor o11y to use new Assistant logo and beacon
(#204289)","sha":"8f70a2dec2579579d82027a8464665c645fc6e48","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["review","release_note:skip","backport:skip","v9.0.0","Team:Obs
AI
Assistant","v8.18.0"],"number":204289,"url":"https://github.com/elastic/kibana/pull/204289","mergeCommit":{"message":"[ai][assistant]
Refactor o11y to use new Assistant logo and beacon
(#204289)","sha":"8f70a2dec2579579d82027a8464665c645fc6e48"}},"sourceBranch":"main","suggestedTargetBranches":["8.x"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","labelRegex":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/204289","number":204289,"mergeCommit":{"message":"[ai][assistant]
Refactor o11y to use new Assistant logo and beacon
(#204289)","sha":"8f70a2dec2579579d82027a8464665c645fc6e48"}},{"branch":"8.x","label":"v8.18.0","labelRegex":"^v8.18.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release_note:skip Skip the PR/issue when compiling release notes review Team:Obs AI Assistant Observability AI Assistant v8.18.0 v9.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants