Skip to content

[Lens] Display the metric title on the dashboard panel#149662

Merged
stratoula merged 1 commit intoelastic:mainfrom
stratoula:display-metric-panel-title
Jan 27, 2023
Merged

[Lens] Display the metric title on the dashboard panel#149662
stratoula merged 1 commit intoelastic:mainfrom
stratoula:display-metric-panel-title

Conversation

@stratoula
Copy link
Copy Markdown
Contributor

@stratoula stratoula commented Jan 27, 2023

Summary

Closes #148563

After this PR being merged, it will be easier to add the description info icon and the custom data range to the new metric. For this reason we decided to also allow the panel title for now.

On the next minors we are going to hide it from the panel and display all the aforementioned info on the viz title.
image

Checklist

@kibana-ci
Copy link
Copy Markdown

💚 Build Succeeded

Metrics [docs]

✅ unchanged

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

@stratoula stratoula added release_note:skip Skip the PR/issue when compiling release notes Feature:Lens backport:skip This PR does not require backporting Team:Visualizations Team label for Lens, elastic-charts, Graph, legacy editors (TSVB, Visualize, Timelion) t// labels Jan 27, 2023
@stratoula stratoula marked this pull request as ready for review January 27, 2023 09:25
@stratoula stratoula requested a review from a team as a code owner January 27, 2023 09:25
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/kibana-visualizations @elastic/kibana-visualizations-external (Team:Visualizations)

Copy link
Copy Markdown
Contributor

@drewdaemon drewdaemon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@stratoula stratoula merged commit a3cbb4e into elastic:main Jan 27, 2023
@ThomThomson
Copy link
Copy Markdown
Contributor

Thanks for doing this! It will make this way more consistent, and more exciting when all of this information is displayed inside the metric. Can't wait to see how that looks.

kqualters-elastic pushed a commit to kqualters-elastic/kibana that referenced this pull request Feb 6, 2023
## Summary

Closes elastic#148563

After this [PR](
elastic#148301) being merged, it will be
easier to add the description info icon and the custom data range to the
new metric. For this reason we decided to also allow the panel title for
now.

On the next minors we are going to hide it from the panel and display
all the aforementioned info on the viz title.
<img width="988" alt="image"
src="https://user-images.githubusercontent.com/17003240/215037834-0f556673-8628-484e-aa32-c34188fc7064.png">

### 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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting Feature:Lens release_note:skip Skip the PR/issue when compiling release notes Team:Visualizations Team label for Lens, elastic-charts, Graph, legacy editors (TSVB, Visualize, Timelion) t// v8.7.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Lens] Hide the edit panel title for new metric visualizations

6 participants