Skip to content

[Security Solution] [Investigation Guides] Use new timeline with arrow icon#155727

Closed
kqualters-elastic wants to merge 1 commit intoelastic:mainfrom
kqualters-elastic:use-new-timeline-arrow-icon
Closed

[Security Solution] [Investigation Guides] Use new timeline with arrow icon#155727
kqualters-elastic wants to merge 1 commit intoelastic:mainfrom
kqualters-elastic:use-new-timeline-arrow-icon

Conversation

@kqualters-elastic
Copy link
Copy Markdown
Contributor

Summary

This pr is meant to be merged after elastic/eui#6731 is merged, released, and said release is used in Kibana. Updates the investigation/insight markdown plugin and button to use a new "timelineArrow" icon, instead of the placeholder currently in use.
image
image

Checklist

@kqualters-elastic kqualters-elastic added release_note:skip Skip the PR/issue when compiling release notes Team:Threat Hunting:Investigations Security Solution Threat Hunting Investigations Team labels Apr 25, 2023
@kqualters-elastic kqualters-elastic requested a review from a team as a code owner April 25, 2023 14:34
@kibana-ci
Copy link
Copy Markdown

💚 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
securitySolution 9.1MB 9.1MB +108.0B
Unknown metric groups

ESLint disabled line counts

id before after diff
enterpriseSearch 17 19 +2
securitySolution 397 400 +3
total +5

Total ESLint disabled count

id before after diff
enterpriseSearch 18 20 +2
securitySolution 477 480 +3
total +5

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@kqualters-elastic kqualters-elastic deleted the use-new-timeline-arrow-icon branch June 7, 2023 20:02
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 Team:Threat Hunting:Investigations Security Solution Threat Hunting Investigations Team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants