Skip to content

[8.x] [Solution Side Nav] Add back external link indicator to sidenav (#215946)#217245

Merged
kowalczyk-krzysztof merged 1 commit intoelastic:8.xfrom
kowalczyk-krzysztof:backport/8.x/pr-215946
Apr 4, 2025
Merged

[8.x] [Solution Side Nav] Add back external link indicator to sidenav (#215946)#217245
kowalczyk-krzysztof merged 1 commit intoelastic:8.xfrom
kowalczyk-krzysztof:backport/8.x/pr-215946

Conversation

@kowalczyk-krzysztof
Copy link
Member

Backport

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

Questions ?

Please refer to the Backport tool documentation

…tic#215946)

## Summary

This PR adds back external link indicator to sidenav items.

![Screenshot 2025-04-01 at 20 35
14](https://github.com/user-attachments/assets/c82def72-780e-4bb6-812d-d51244e90685)

(cherry picked from commit e21739b)

# Conflicts:
#	src/platform/packages/shared/shared-ux/chrome/navigation/src/ui/navigation.stories.tsx
@elasticmachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #16 / transform basic license transform - actions starting non healthy continuous transform with pivot configuration start transform

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
navigation 19.9KB 20.0KB +95.0B
serverless 20.6KB 20.7KB +95.0B
total +190.0B

Page load bundle

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

id before after diff
core 441.2KB 441.2KB -7.0B

@kowalczyk-krzysztof kowalczyk-krzysztof merged commit 1746395 into elastic:8.x Apr 4, 2025
12 checks passed
@kowalczyk-krzysztof kowalczyk-krzysztof deleted the backport/8.x/pr-215946 branch August 1, 2025 12:28
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