Skip to content

[9.0] [Search] Root Elasticsearch breadcrumb missing link (#230270)#230782

Merged
seialkali merged 4 commits intoelastic:9.0from
seialkali:backport/9.0/pr-230270
Aug 13, 2025
Merged

[9.0] [Search] Root Elasticsearch breadcrumb missing link (#230270)#230782
seialkali merged 4 commits intoelastic:9.0from
seialkali:backport/9.0/pr-230270

Conversation

@seialkali
Copy link
Contributor

Backport

This will backport the following commits from main to 9.0:

Questions ?

Please refer to the Backport tool documentation

@seialkali seialkali added the backport This PR is a backport of another PR label Aug 6, 2025
@seialkali seialkali enabled auto-merge (squash) August 6, 2025 12:59
@seialkali seialkali marked this pull request as draft August 6, 2025 13:00
auto-merge was automatically disabled August 6, 2025 13:00

Pull request was converted to draft

seialkali and others added 2 commits August 6, 2025 14:51
Root Elasticsearch breadcrumb missing link when set from search
navigation API.

On the search applications page, the breadcrumb for Elasticsearch had a
link back to the home page. This link was missing for the breadcrumbs on
the query rules and other pages that use search navigation to set
breadcrumbs.

**Working breadcrumb on the search applications page**
<img width="600" alt="Screenshot 2025-07-31 at 14 41 51"
src="https://github.com/user-attachments/assets/dbd168f0-08f9-479c-84a6-f139362aec3c"
/>

**Broken breadcrumb on the query rules page**
<img width="600" alt="Screenshot 2025-07-31 at 14 42 01"
src="https://github.com/user-attachments/assets/aec34891-5c25-447d-b977-f2d2d61babc6"
/>

**Fixed breadcrumb on the query rules page**
<img width="600" alt="Screenshot 2025-08-01 at 16 41 36"
src="https://github.com/user-attachments/assets/4a8fb90b-fad4-4c38-94d0-72488e33a797"
/>

Check the PR satisfies following conditions.

Reviewers should verify this PR satisfies this list as well.

- [ ] ~Any text added follows [EUI's writing
guidelines](https://elastic.github.io/eui/#/guidelines/writing), uses
sentence case text and includes [i18n
support](https://github.com/elastic/kibana/blob/main/src/platform/packages/shared/kbn-i18n/README.md)~
- [ ]
~[Documentation](https://www.elastic.co/guide/en/kibana/master/development-documentation.html)
was added for features that require explanation or tutorials~
- [ ] ~[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~
- [ ] ~If a plugin configuration key changed, check if it needs to be
allowlisted in the cloud and added to the [docker
list](https://github.com/elastic/kibana/blob/main/src/dev/build/tasks/os_packages/docker_generator/resources/base/bin/kibana-docker)~
- [ ] ~This was checked for breaking HTTP API changes, and any breaking
changes have been approved by the breaking-change committee. The
`release_note:breaking` label should be applied in these situations.~
- [ ] ~[Flaky Test
Runner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was
used on any tests changed~
- [x] The PR description includes the appropriate Release Notes section,
and the correct `release_note:*` label is applied per the
[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)
- [x] Review the [backport
guidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing)
and apply applicable `backport:*` labels.

Fixes Elasticsearch breadcrumb to include a link to the home page for
search pages.

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
(cherry picked from commit 3de36a3)
@seialkali seialkali force-pushed the backport/9.0/pr-230270 branch from 8fee230 to 7483454 Compare August 6, 2025 13:52
@seialkali seialkali marked this pull request as ready for review August 6, 2025 14:32
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
searchNavigation 6 8 +2

Page load bundle

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

id before after diff
searchNavigation 5.5KB 5.7KB +141.0B

History

@seialkali seialkali merged commit 21b6471 into elastic:9.0 Aug 13, 2025
9 checks passed
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.

4 participants