Skip to content

[8.11] [Dashboard] Add Dashboard title to browser tab title (#171255)#171454

Merged
kibanamachine merged 1 commit intoelastic:8.11from
kibanamachine:backport/8.11/pr-171255
Nov 17, 2023
Merged

[8.11] [Dashboard] Add Dashboard title to browser tab title (#171255)#171454
kibanamachine merged 1 commit intoelastic:8.11from
kibanamachine:backport/8.11/pr-171255

Conversation

@kibanamachine
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 8.11:

Questions ?

Please refer to the Backport tool documentation

Closes elastic#162800

## Summary

This PR re-adds dashboard titles to the browser tab title, which was
accidentally removed as part of the [portable
dashboards](elastic#144332) work. For
example, if I'm on the sample Logs dashboard, the title of that
dashboard will now be reflected in the tab title like it was prior to
`v8.7.0`:

| Before | After |
|--------|--------|
|
![image](https://github.com/elastic/kibana/assets/8698078/79044734-f9f5-41e2-b7e6-27087d37832d)
|
![image](https://github.com/elastic/kibana/assets/8698078/e82740a8-b4ef-488e-981a-57b5ef39948a)
|

The tab title should stay up-to-date with Dashboard title changes, as
demonstrated in this video:

https://github.com/elastic/kibana/assets/8698078/651fff50-70f7-46ff-af47-b274fe6b0a19

Note that this will **only apply** to dashboards in the dashboard app -
dashboards outside of the dashboard app should not change the browser
tab title, unless the consumer does this on their own.

### [Flaky Test
Runner](https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/3987)

![image](https://github.com/elastic/kibana/assets/8698078/aec4100b-9e76-4154-b20b-a7054f7f46a1)

### 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
- [x] This was checked for [cross-browser
compatibility](https://www.elastic.co/support/matrix#matrix_browsers)

### For maintainers

- [ ] This was checked for breaking API changes and was [labeled
appropriately](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)

(cherry picked from commit 6150f76)
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Nov 16, 2023
@kibanamachine kibanamachine enabled auto-merge (squash) November 16, 2023 23:30
@kibana-ci
Copy link
Copy Markdown

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
dashboard 387 388 +1

Async chunks

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

id before after diff
dashboard 374.1KB 374.4KB +357.0B

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

cc @Heenawter

@kibanamachine kibanamachine merged commit faef45f into elastic:8.11 Nov 17, 2023
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