Skip to content

Comments

Fixing layout of the embeddablePanelHeading containing description an…#219428

Merged
olapawlus merged 10 commits intoelastic:mainfrom
olapawlus:embeddablePanelHeading
May 6, 2025
Merged

Fixing layout of the embeddablePanelHeading containing description an…#219428
olapawlus merged 10 commits intoelastic:mainfrom
olapawlus:embeddablePanelHeading

Conversation

@olapawlus
Copy link
Member

@olapawlus olapawlus commented Apr 28, 2025

It has been assumed that everything of the embeddable panel heading will stay on one line, and this is a proposal based on that assumption.

This video shows how the layout responds:

Screen.Recording.2025-05-02.at.20.37.57.mov

Introduced ellipsis for panel titles and custom time range badges when the text overflows.

Fixed the issue with the description icon, which was wrapping to the next line.

Fixed the layout of the embeddable panel heading containing three elements: title, icon, and badge.

Closes: #211252 and #219868

Improved layout of children within the parent container, as the previous styles didn’t prevent significant disproportion caused by varying title and time-range-badge text lengths (disproportion is shown in the first screenshot below).
Screenshot 2025-05-02 at 19 39 01
Now, each child (which is title and time-range-badge) will grow/shrink proportionally, ensuring consistent layout regardless of their content length. (as shown in the screenshots below)
Screenshot 2025-05-02 at 19 39 30
Screenshot 2025-05-02 at 19 40 06

@olapawlus olapawlus added bug Fixes for quality problems that affect the customer experience Feature:Dashboard Dashboard related features regression Team:Presentation Presentation Team for Dashboard, Input Controls, and Canvas t// loe:small Small Level of Effort impact:high Addressing this issue will have a high level of impact on the quality/strength of our product. labels Apr 28, 2025
@olapawlus olapawlus self-assigned this Apr 28, 2025
@olapawlus olapawlus changed the title fixing layout of the embeddablePanelHeading containing description an… Fixing layout of the embeddablePanelHeading containing description an… Apr 28, 2025
@olapawlus olapawlus requested a review from andreadelrio April 28, 2025 16:02
@olapawlus
Copy link
Member Author

Uploading Screen Recording 2025-04-28 at 17.39.15.mov…

Copy link
Contributor

@nickpeihl nickpeihl left a comment

Choose a reason for hiding this comment

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

lgtm! But let's see what @andreadelrio thinks.

@olapawlus I think you can make this PR "Ready for review". Then we'll need to add the appropriate release-* and backport-version labels to this PR.

@olapawlus olapawlus marked this pull request as ready for review April 29, 2025 08:50
@olapawlus olapawlus requested a review from a team as a code owner April 29, 2025 08:50
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-presentation (Team:Presentation)

@olapawlus olapawlus removed the request for review from andreadelrio April 29, 2025 09:02
@mbondyra mbondyra added v8.19.0 and removed v8.17.6 labels Apr 30, 2025
@olapawlus olapawlus requested a review from andreadelrio April 30, 2025 09:43
@nickpeihl nickpeihl requested review from gvnmagni and removed request for andreadelrio April 30, 2025 18:33
@nickpeihl
Copy link
Contributor

@gvnmagni Would you mind looking at the changes in this PR? Is it okay that we force the time range badge on the panel in the same line as the panel title and truncate with an ellipsis as necessary? I guess the other option would be to wrap the time range badge to the next line if it doesn't fit.

@olapawlus
Copy link
Member Author

olapawlus commented May 2, 2025

FYI - title content used to wrap onto at most two lines. Here is how it looked in 7.17

Screenshot 2025-04-30 at 4 10 40 PM

[Solved]

Could you confirm if only the title with description icon, and time range badge are the elements inside the embeddable header container? If so, we can safely assume the header height will not exceed two lines.

I'm asking because, to match the layout shown in the screenshot, the parent container should allow its children to wrap to a new line if necessary — but only up to two lines. Each direct child should be styled in a way that prevents its own content from wrapping internally. This ensures that if there are any elements beyond the title with icon, and time range badge, the header container won't push the layout into a third line. Currently, the header's height is hardcoded to 24px, but with change, it should grow dynamically — up to two lines. If we can confirm that only these three elements will be inside the container, we can safely assume the header height will not exceed two lines.

The attached video demonstrates the expected behavior:

Screen.Recording.2025-05-02.at.20.36.09.mov

@andreadelrio andreadelrio self-requested a review May 5, 2025 20:44
Copy link
Contributor

@andreadelrio andreadelrio left a comment

Choose a reason for hiding this comment

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

Design changes LGTM. Nice job!

@olapawlus
Copy link
Member Author

@elasticmachine merge upstream

@olapawlus olapawlus enabled auto-merge (squash) May 6, 2025 08:07
@olapawlus olapawlus removed the request for review from gvnmagni May 6, 2025 08:34
@elasticmachine
Copy link
Contributor

💚 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
presentationPanel 37.7KB 37.8KB +87.0B

History

cc @olapawlus

@olapawlus olapawlus merged commit ca06247 into elastic:main May 6, 2025
10 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.18, 8.19, 9.0

https://github.com/elastic/kibana/actions/runs/14855472821

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request May 6, 2025
elastic#219428)

It has been assumed that everything of the embeddable panel heading will
stay on one line, and this is a proposal based on that assumption.

This video shows how the layout responds:

https://github.com/user-attachments/assets/db7f3e38-1fbd-4d21-a0a2-96ed1768b744

Introduced ellipsis for panel titles and custom time range badges when
the text overflows.

Fixed the issue with the description icon, which was wrapping to the
next line.

Fixed the layout of the embeddable panel heading containing three
elements: title, icon, and badge.

Closes: elastic#211252 and elastic#219868

Improved layout of children within the parent container, as the previous
styles didn’t prevent significant disproportion caused by varying title
and time-range-badge text lengths (disproportion is shown in the first
screenshot below).
![Screenshot 2025-05-02 at 19 39
01](https://github.com/user-attachments/assets/23c4a4ae-2dc0-4c04-9ecf-ed4abfba8c76)
Now, each child (which is title and time-range-badge) will grow/shrink
proportionally, ensuring consistent layout regardless of their content
length. (as shown in the screenshots below)
![Screenshot 2025-05-02 at 19 39
30](https://github.com/user-attachments/assets/a48d1c9e-2bb0-4c55-96be-3b379091e904)
![Screenshot 2025-05-02 at 19 40
06](https://github.com/user-attachments/assets/72171174-6565-412a-abb9-1fc53b7b094c)

---------

Co-authored-by: Hannah Mudge <Heenawter@users.noreply.github.com>
Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
(cherry picked from commit ca06247)
@kibanamachine
Copy link
Contributor

💔 Some backports could not be created

Status Branch Result
8.18 Backport failed because of merge conflicts
8.19
9.0 Backport failed because of merge conflicts

Note: Successful backport PRs will be merged automatically after passing CI.

Manual backport

To create the backport manually run:

node scripts/backport --pr 219428

Questions ?

Please refer to the Backport tool documentation

@Heenawter Heenawter linked an issue May 6, 2025 that may be closed by this pull request
@kibanamachine kibanamachine added the backport missing Added to PRs automatically when the are determined to be missing a backport. label May 8, 2025
@kibanamachine
Copy link
Contributor

Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync.
cc: @olapawlus

2 similar comments
@kibanamachine
Copy link
Contributor

Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync.
cc: @olapawlus

@kibanamachine
Copy link
Contributor

Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync.
cc: @olapawlus

kibanamachine added a commit that referenced this pull request May 12, 2025
…tion an… (#219428) (#220206)

# Backport

This will backport the following commits from `main` to `8.19`:
- [Fixing layout of the embeddablePanelHeading containing description
an… (#219428)](#219428)

<!--- Backport version: 9.6.6 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sorenlouv/backport)

<!--BACKPORT [{"author":{"name":"Ola
Pawlus","email":"98127445+olapawlus@users.noreply.github.com"},"sourceCommit":{"committedDate":"2025-05-06T08:48:33Z","message":"Fixing
layout of the embeddablePanelHeading containing description an…
(#219428)\n\nIt has been assumed that everything of the embeddable panel
heading will\nstay on one line, and this is a proposal based on that
assumption.\n\nThis video shows how the layout
responds:\n\n\nhttps://github.com/user-attachments/assets/db7f3e38-1fbd-4d21-a0a2-96ed1768b744\n\n\n\nIntroduced
ellipsis for panel titles and custom time range badges when\nthe text
overflows.\n\nFixed the issue with the description icon, which was
wrapping to the\nnext line.\n\nFixed the layout of the embeddable panel
heading containing three\nelements: title, icon, and badge.\n\nCloses:
#211252 and #219868\n\n\nImproved layout of children within the parent
container, as the previous\nstyles didn’t prevent significant
disproportion caused by varying title\nand time-range-badge text lengths
(disproportion is shown in the first\nscreenshot below).\n![Screenshot
2025-05-02 at 19
39\n01](https://github.com/user-attachments/assets/23c4a4ae-2dc0-4c04-9ecf-ed4abfba8c76)\nNow,
each child (which is title and time-range-badge) will
grow/shrink\nproportionally, ensuring consistent layout regardless of
their content\nlength. (as shown in the screenshots below)\n![Screenshot
2025-05-02 at 19
39\n30](https://github.com/user-attachments/assets/a48d1c9e-2bb0-4c55-96be-3b379091e904)\n![Screenshot
2025-05-02 at 19
40\n06](https://github.com/user-attachments/assets/72171174-6565-412a-abb9-1fc53b7b094c)\n\n---------\n\nCo-authored-by:
Hannah Mudge <Heenawter@users.noreply.github.com>\nCo-authored-by:
Elastic Machine
<elasticmachine@users.noreply.github.com>","sha":"ca062472b949de4efb480611c0eebff411f67b00","branchLabelMapping":{"^v9.1.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["bug","Feature:Dashboard","regression","release_note:fix","Team:Presentation","loe:small","impact:high","backport:version","v9.1.0","v8.19.0","v8.18.1","v9.0.1"],"title":"Fixing
layout of the embeddablePanelHeading containing description
an…","number":219428,"url":"https://github.com/elastic/kibana/pull/219428","mergeCommit":{"message":"Fixing
layout of the embeddablePanelHeading containing description an…
(#219428)\n\nIt has been assumed that everything of the embeddable panel
heading will\nstay on one line, and this is a proposal based on that
assumption.\n\nThis video shows how the layout
responds:\n\n\nhttps://github.com/user-attachments/assets/db7f3e38-1fbd-4d21-a0a2-96ed1768b744\n\n\n\nIntroduced
ellipsis for panel titles and custom time range badges when\nthe text
overflows.\n\nFixed the issue with the description icon, which was
wrapping to the\nnext line.\n\nFixed the layout of the embeddable panel
heading containing three\nelements: title, icon, and badge.\n\nCloses:
#211252 and #219868\n\n\nImproved layout of children within the parent
container, as the previous\nstyles didn’t prevent significant
disproportion caused by varying title\nand time-range-badge text lengths
(disproportion is shown in the first\nscreenshot below).\n![Screenshot
2025-05-02 at 19
39\n01](https://github.com/user-attachments/assets/23c4a4ae-2dc0-4c04-9ecf-ed4abfba8c76)\nNow,
each child (which is title and time-range-badge) will
grow/shrink\nproportionally, ensuring consistent layout regardless of
their content\nlength. (as shown in the screenshots below)\n![Screenshot
2025-05-02 at 19
39\n30](https://github.com/user-attachments/assets/a48d1c9e-2bb0-4c55-96be-3b379091e904)\n![Screenshot
2025-05-02 at 19
40\n06](https://github.com/user-attachments/assets/72171174-6565-412a-abb9-1fc53b7b094c)\n\n---------\n\nCo-authored-by:
Hannah Mudge <Heenawter@users.noreply.github.com>\nCo-authored-by:
Elastic Machine
<elasticmachine@users.noreply.github.com>","sha":"ca062472b949de4efb480611c0eebff411f67b00"}},"sourceBranch":"main","suggestedTargetBranches":["8.19","8.18","9.0"],"targetPullRequestStates":[{"branch":"main","label":"v9.1.0","branchLabelMappingKey":"^v9.1.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/219428","number":219428,"mergeCommit":{"message":"Fixing
layout of the embeddablePanelHeading containing description an…
(#219428)\n\nIt has been assumed that everything of the embeddable panel
heading will\nstay on one line, and this is a proposal based on that
assumption.\n\nThis video shows how the layout
responds:\n\n\nhttps://github.com/user-attachments/assets/db7f3e38-1fbd-4d21-a0a2-96ed1768b744\n\n\n\nIntroduced
ellipsis for panel titles and custom time range badges when\nthe text
overflows.\n\nFixed the issue with the description icon, which was
wrapping to the\nnext line.\n\nFixed the layout of the embeddable panel
heading containing three\nelements: title, icon, and badge.\n\nCloses:
#211252 and #219868\n\n\nImproved layout of children within the parent
container, as the previous\nstyles didn’t prevent significant
disproportion caused by varying title\nand time-range-badge text lengths
(disproportion is shown in the first\nscreenshot below).\n![Screenshot
2025-05-02 at 19
39\n01](https://github.com/user-attachments/assets/23c4a4ae-2dc0-4c04-9ecf-ed4abfba8c76)\nNow,
each child (which is title and time-range-badge) will
grow/shrink\nproportionally, ensuring consistent layout regardless of
their content\nlength. (as shown in the screenshots below)\n![Screenshot
2025-05-02 at 19
39\n30](https://github.com/user-attachments/assets/a48d1c9e-2bb0-4c55-96be-3b379091e904)\n![Screenshot
2025-05-02 at 19
40\n06](https://github.com/user-attachments/assets/72171174-6565-412a-abb9-1fc53b7b094c)\n\n---------\n\nCo-authored-by:
Hannah Mudge <Heenawter@users.noreply.github.com>\nCo-authored-by:
Elastic Machine
<elasticmachine@users.noreply.github.com>","sha":"ca062472b949de4efb480611c0eebff411f67b00"}},{"branch":"8.19","label":"v8.19.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.18","label":"v8.18.1","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"9.0","label":"v9.0.1","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Ola Pawlus <98127445+olapawlus@users.noreply.github.com>
Co-authored-by: Hannah Mudge <Heenawter@users.noreply.github.com>
Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
@kibanamachine kibanamachine removed the backport missing Added to PRs automatically when the are determined to be missing a backport. label May 12, 2025
@olapawlus
Copy link
Member Author

💚 All backports created successfully

Status Branch Result
9.0
8.18

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

olapawlus added a commit that referenced this pull request May 12, 2025
…ion an… (#219428) (#220752)

# Backport

This will backport the following commits from `main` to `9.0`:
- [Fixing layout of the embeddablePanelHeading containing description
an… (#219428)](#219428)

<!--- Backport version: 9.6.6 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sorenlouv/backport)

<!--BACKPORT [{"author":{"name":"Ola
Pawlus","email":"98127445+olapawlus@users.noreply.github.com"},"sourceCommit":{"committedDate":"2025-05-06T08:48:33Z","message":"Fixing
layout of the embeddablePanelHeading containing description an…
(#219428)\n\nIt has been assumed that everything of the embeddable panel
heading will\nstay on one line, and this is a proposal based on that
assumption.\n\nThis video shows how the layout
responds:\n\n\nhttps://github.com/user-attachments/assets/db7f3e38-1fbd-4d21-a0a2-96ed1768b744\n\n\n\nIntroduced
ellipsis for panel titles and custom time range badges when\nthe text
overflows.\n\nFixed the issue with the description icon, which was
wrapping to the\nnext line.\n\nFixed the layout of the embeddable panel
heading containing three\nelements: title, icon, and badge.\n\nCloses:
#211252 and #219868\n\n\nImproved layout of children within the parent
container, as the previous\nstyles didn’t prevent significant
disproportion caused by varying title\nand time-range-badge text lengths
(disproportion is shown in the first\nscreenshot below).\n![Screenshot
2025-05-02 at 19
39\n01](https://github.com/user-attachments/assets/23c4a4ae-2dc0-4c04-9ecf-ed4abfba8c76)\nNow,
each child (which is title and time-range-badge) will
grow/shrink\nproportionally, ensuring consistent layout regardless of
their content\nlength. (as shown in the screenshots below)\n![Screenshot
2025-05-02 at 19
39\n30](https://github.com/user-attachments/assets/a48d1c9e-2bb0-4c55-96be-3b379091e904)\n![Screenshot
2025-05-02 at 19
40\n06](https://github.com/user-attachments/assets/72171174-6565-412a-abb9-1fc53b7b094c)\n\n---------\n\nCo-authored-by:
Hannah Mudge <Heenawter@users.noreply.github.com>\nCo-authored-by:
Elastic Machine
<elasticmachine@users.noreply.github.com>","sha":"ca062472b949de4efb480611c0eebff411f67b00","branchLabelMapping":{"^v9.1.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["bug","Feature:Dashboard","regression","release_note:fix","Team:Presentation","loe:small","impact:high","backport:version","v9.1.0","v8.19.0","v8.18.1","v9.0.1"],"title":"Fixing
layout of the embeddablePanelHeading containing description
an…","number":219428,"url":"https://github.com/elastic/kibana/pull/219428","mergeCommit":{"message":"Fixing
layout of the embeddablePanelHeading containing description an…
(#219428)\n\nIt has been assumed that everything of the embeddable panel
heading will\nstay on one line, and this is a proposal based on that
assumption.\n\nThis video shows how the layout
responds:\n\n\nhttps://github.com/user-attachments/assets/db7f3e38-1fbd-4d21-a0a2-96ed1768b744\n\n\n\nIntroduced
ellipsis for panel titles and custom time range badges when\nthe text
overflows.\n\nFixed the issue with the description icon, which was
wrapping to the\nnext line.\n\nFixed the layout of the embeddable panel
heading containing three\nelements: title, icon, and badge.\n\nCloses:
#211252 and #219868\n\n\nImproved layout of children within the parent
container, as the previous\nstyles didn’t prevent significant
disproportion caused by varying title\nand time-range-badge text lengths
(disproportion is shown in the first\nscreenshot below).\n![Screenshot
2025-05-02 at 19
39\n01](https://github.com/user-attachments/assets/23c4a4ae-2dc0-4c04-9ecf-ed4abfba8c76)\nNow,
each child (which is title and time-range-badge) will
grow/shrink\nproportionally, ensuring consistent layout regardless of
their content\nlength. (as shown in the screenshots below)\n![Screenshot
2025-05-02 at 19
39\n30](https://github.com/user-attachments/assets/a48d1c9e-2bb0-4c55-96be-3b379091e904)\n![Screenshot
2025-05-02 at 19
40\n06](https://github.com/user-attachments/assets/72171174-6565-412a-abb9-1fc53b7b094c)\n\n---------\n\nCo-authored-by:
Hannah Mudge <Heenawter@users.noreply.github.com>\nCo-authored-by:
Elastic Machine
<elasticmachine@users.noreply.github.com>","sha":"ca062472b949de4efb480611c0eebff411f67b00"}},"sourceBranch":"main","suggestedTargetBranches":["8.18","9.0"],"targetPullRequestStates":[{"branch":"main","label":"v9.1.0","branchLabelMappingKey":"^v9.1.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/219428","number":219428,"mergeCommit":{"message":"Fixing
layout of the embeddablePanelHeading containing description an…
(#219428)\n\nIt has been assumed that everything of the embeddable panel
heading will\nstay on one line, and this is a proposal based on that
assumption.\n\nThis video shows how the layout
responds:\n\n\nhttps://github.com/user-attachments/assets/db7f3e38-1fbd-4d21-a0a2-96ed1768b744\n\n\n\nIntroduced
ellipsis for panel titles and custom time range badges when\nthe text
overflows.\n\nFixed the issue with the description icon, which was
wrapping to the\nnext line.\n\nFixed the layout of the embeddable panel
heading containing three\nelements: title, icon, and badge.\n\nCloses:
#211252 and #219868\n\n\nImproved layout of children within the parent
container, as the previous\nstyles didn’t prevent significant
disproportion caused by varying title\nand time-range-badge text lengths
(disproportion is shown in the first\nscreenshot below).\n![Screenshot
2025-05-02 at 19
39\n01](https://github.com/user-attachments/assets/23c4a4ae-2dc0-4c04-9ecf-ed4abfba8c76)\nNow,
each child (which is title and time-range-badge) will
grow/shrink\nproportionally, ensuring consistent layout regardless of
their content\nlength. (as shown in the screenshots below)\n![Screenshot
2025-05-02 at 19
39\n30](https://github.com/user-attachments/assets/a48d1c9e-2bb0-4c55-96be-3b379091e904)\n![Screenshot
2025-05-02 at 19
40\n06](https://github.com/user-attachments/assets/72171174-6565-412a-abb9-1fc53b7b094c)\n\n---------\n\nCo-authored-by:
Hannah Mudge <Heenawter@users.noreply.github.com>\nCo-authored-by:
Elastic Machine
<elasticmachine@users.noreply.github.com>","sha":"ca062472b949de4efb480611c0eebff411f67b00"}},{"branch":"8.19","label":"v8.19.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"url":"https://github.com/elastic/kibana/pull/220206","number":220206,"state":"MERGED","mergeCommit":{"sha":"b3450affa4dd7a2c827a29b4769ad79e966686a7","message":"[8.19]
Fixing layout of the embeddablePanelHeading containing description an…
(#219428) (#220206)\n\n# Backport\n\nThis will backport the following
commits from `main` to `8.19`:\n- [Fixing layout of the
embeddablePanelHeading containing description\nan…
(#219428)](https://github.com/elastic/kibana/pull/219428)\n\n\n\n###
Questions ?\nPlease refer to the [Backport
tool\ndocumentation](https://github.com/sorenlouv/backport)\n\n\n\nCo-authored-by:
Ola Pawlus
<98127445+olapawlus@users.noreply.github.com>\nCo-authored-by: Hannah
Mudge <Heenawter@users.noreply.github.com>\nCo-authored-by: Elastic
Machine
<elasticmachine@users.noreply.github.com>"}},{"branch":"8.18","label":"v8.18.1","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"9.0","label":"v9.0.1","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->
olapawlus added a commit that referenced this pull request May 12, 2025
…tion an… (#219428) (#220754)

# Backport

This will backport the following commits from `main` to `8.18`:
- [Fixing layout of the embeddablePanelHeading containing description
an… (#219428)](#219428)

<!--- Backport version: 9.6.6 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sorenlouv/backport)

<!--BACKPORT [{"author":{"name":"Ola
Pawlus","email":"98127445+olapawlus@users.noreply.github.com"},"sourceCommit":{"committedDate":"2025-05-06T08:48:33Z","message":"Fixing
layout of the embeddablePanelHeading containing description an…
(#219428)\n\nIt has been assumed that everything of the embeddable panel
heading will\nstay on one line, and this is a proposal based on that
assumption.\n\nThis video shows how the layout
responds:\n\n\nhttps://github.com/user-attachments/assets/db7f3e38-1fbd-4d21-a0a2-96ed1768b744\n\n\n\nIntroduced
ellipsis for panel titles and custom time range badges when\nthe text
overflows.\n\nFixed the issue with the description icon, which was
wrapping to the\nnext line.\n\nFixed the layout of the embeddable panel
heading containing three\nelements: title, icon, and badge.\n\nCloses:
#211252 and #219868\n\n\nImproved layout of children within the parent
container, as the previous\nstyles didn’t prevent significant
disproportion caused by varying title\nand time-range-badge text lengths
(disproportion is shown in the first\nscreenshot below).\n![Screenshot
2025-05-02 at 19
39\n01](https://github.com/user-attachments/assets/23c4a4ae-2dc0-4c04-9ecf-ed4abfba8c76)\nNow,
each child (which is title and time-range-badge) will
grow/shrink\nproportionally, ensuring consistent layout regardless of
their content\nlength. (as shown in the screenshots below)\n![Screenshot
2025-05-02 at 19
39\n30](https://github.com/user-attachments/assets/a48d1c9e-2bb0-4c55-96be-3b379091e904)\n![Screenshot
2025-05-02 at 19
40\n06](https://github.com/user-attachments/assets/72171174-6565-412a-abb9-1fc53b7b094c)\n\n---------\n\nCo-authored-by:
Hannah Mudge <Heenawter@users.noreply.github.com>\nCo-authored-by:
Elastic Machine
<elasticmachine@users.noreply.github.com>","sha":"ca062472b949de4efb480611c0eebff411f67b00","branchLabelMapping":{"^v9.1.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["bug","Feature:Dashboard","regression","release_note:fix","Team:Presentation","loe:small","impact:high","backport:version","v9.1.0","v8.19.0","v8.18.1","v9.0.1"],"title":"Fixing
layout of the embeddablePanelHeading containing description
an…","number":219428,"url":"https://github.com/elastic/kibana/pull/219428","mergeCommit":{"message":"Fixing
layout of the embeddablePanelHeading containing description an…
(#219428)\n\nIt has been assumed that everything of the embeddable panel
heading will\nstay on one line, and this is a proposal based on that
assumption.\n\nThis video shows how the layout
responds:\n\n\nhttps://github.com/user-attachments/assets/db7f3e38-1fbd-4d21-a0a2-96ed1768b744\n\n\n\nIntroduced
ellipsis for panel titles and custom time range badges when\nthe text
overflows.\n\nFixed the issue with the description icon, which was
wrapping to the\nnext line.\n\nFixed the layout of the embeddable panel
heading containing three\nelements: title, icon, and badge.\n\nCloses:
#211252 and #219868\n\n\nImproved layout of children within the parent
container, as the previous\nstyles didn’t prevent significant
disproportion caused by varying title\nand time-range-badge text lengths
(disproportion is shown in the first\nscreenshot below).\n![Screenshot
2025-05-02 at 19
39\n01](https://github.com/user-attachments/assets/23c4a4ae-2dc0-4c04-9ecf-ed4abfba8c76)\nNow,
each child (which is title and time-range-badge) will
grow/shrink\nproportionally, ensuring consistent layout regardless of
their content\nlength. (as shown in the screenshots below)\n![Screenshot
2025-05-02 at 19
39\n30](https://github.com/user-attachments/assets/a48d1c9e-2bb0-4c55-96be-3b379091e904)\n![Screenshot
2025-05-02 at 19
40\n06](https://github.com/user-attachments/assets/72171174-6565-412a-abb9-1fc53b7b094c)\n\n---------\n\nCo-authored-by:
Hannah Mudge <Heenawter@users.noreply.github.com>\nCo-authored-by:
Elastic Machine
<elasticmachine@users.noreply.github.com>","sha":"ca062472b949de4efb480611c0eebff411f67b00"}},"sourceBranch":"main","suggestedTargetBranches":["8.18","9.0"],"targetPullRequestStates":[{"branch":"main","label":"v9.1.0","branchLabelMappingKey":"^v9.1.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/219428","number":219428,"mergeCommit":{"message":"Fixing
layout of the embeddablePanelHeading containing description an…
(#219428)\n\nIt has been assumed that everything of the embeddable panel
heading will\nstay on one line, and this is a proposal based on that
assumption.\n\nThis video shows how the layout
responds:\n\n\nhttps://github.com/user-attachments/assets/db7f3e38-1fbd-4d21-a0a2-96ed1768b744\n\n\n\nIntroduced
ellipsis for panel titles and custom time range badges when\nthe text
overflows.\n\nFixed the issue with the description icon, which was
wrapping to the\nnext line.\n\nFixed the layout of the embeddable panel
heading containing three\nelements: title, icon, and badge.\n\nCloses:
#211252 and #219868\n\n\nImproved layout of children within the parent
container, as the previous\nstyles didn’t prevent significant
disproportion caused by varying title\nand time-range-badge text lengths
(disproportion is shown in the first\nscreenshot below).\n![Screenshot
2025-05-02 at 19
39\n01](https://github.com/user-attachments/assets/23c4a4ae-2dc0-4c04-9ecf-ed4abfba8c76)\nNow,
each child (which is title and time-range-badge) will
grow/shrink\nproportionally, ensuring consistent layout regardless of
their content\nlength. (as shown in the screenshots below)\n![Screenshot
2025-05-02 at 19
39\n30](https://github.com/user-attachments/assets/a48d1c9e-2bb0-4c55-96be-3b379091e904)\n![Screenshot
2025-05-02 at 19
40\n06](https://github.com/user-attachments/assets/72171174-6565-412a-abb9-1fc53b7b094c)\n\n---------\n\nCo-authored-by:
Hannah Mudge <Heenawter@users.noreply.github.com>\nCo-authored-by:
Elastic Machine
<elasticmachine@users.noreply.github.com>","sha":"ca062472b949de4efb480611c0eebff411f67b00"}},{"branch":"8.19","label":"v8.19.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"url":"https://github.com/elastic/kibana/pull/220206","number":220206,"state":"MERGED","mergeCommit":{"sha":"b3450affa4dd7a2c827a29b4769ad79e966686a7","message":"[8.19]
Fixing layout of the embeddablePanelHeading containing description an…
(#219428) (#220206)\n\n# Backport\n\nThis will backport the following
commits from `main` to `8.19`:\n- [Fixing layout of the
embeddablePanelHeading containing description\nan…
(#219428)](https://github.com/elastic/kibana/pull/219428)\n\n\n\n###
Questions ?\nPlease refer to the [Backport
tool\ndocumentation](https://github.com/sorenlouv/backport)\n\n\n\nCo-authored-by:
Ola Pawlus
<98127445+olapawlus@users.noreply.github.com>\nCo-authored-by: Hannah
Mudge <Heenawter@users.noreply.github.com>\nCo-authored-by: Elastic
Machine
<elasticmachine@users.noreply.github.com>"}},{"branch":"8.18","label":"v8.18.1","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"9.0","label":"v9.0.1","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->
akowalska622 pushed a commit to akowalska622/kibana that referenced this pull request May 29, 2025
elastic#219428)

It has been assumed that everything of the embeddable panel heading will
stay on one line, and this is a proposal based on that assumption.

This video shows how the layout responds:


https://github.com/user-attachments/assets/db7f3e38-1fbd-4d21-a0a2-96ed1768b744



Introduced ellipsis for panel titles and custom time range badges when
the text overflows.

Fixed the issue with the description icon, which was wrapping to the
next line.

Fixed the layout of the embeddable panel heading containing three
elements: title, icon, and badge.

Closes: elastic#211252 and elastic#219868


Improved layout of children within the parent container, as the previous
styles didn’t prevent significant disproportion caused by varying title
and time-range-badge text lengths (disproportion is shown in the first
screenshot below).
![Screenshot 2025-05-02 at 19 39
01](https://github.com/user-attachments/assets/23c4a4ae-2dc0-4c04-9ecf-ed4abfba8c76)
Now, each child (which is title and time-range-badge) will grow/shrink
proportionally, ensuring consistent layout regardless of their content
length. (as shown in the screenshots below)
![Screenshot 2025-05-02 at 19 39
30](https://github.com/user-attachments/assets/a48d1c9e-2bb0-4c55-96be-3b379091e904)
![Screenshot 2025-05-02 at 19 40
06](https://github.com/user-attachments/assets/72171174-6565-412a-abb9-1fc53b7b094c)

---------

Co-authored-by: Hannah Mudge <Heenawter@users.noreply.github.com>
Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
qn895 pushed a commit to qn895/kibana that referenced this pull request Jun 3, 2025
elastic#219428)

It has been assumed that everything of the embeddable panel heading will
stay on one line, and this is a proposal based on that assumption.

This video shows how the layout responds:


https://github.com/user-attachments/assets/db7f3e38-1fbd-4d21-a0a2-96ed1768b744



Introduced ellipsis for panel titles and custom time range badges when
the text overflows.

Fixed the issue with the description icon, which was wrapping to the
next line.

Fixed the layout of the embeddable panel heading containing three
elements: title, icon, and badge.

Closes: elastic#211252 and elastic#219868


Improved layout of children within the parent container, as the previous
styles didn’t prevent significant disproportion caused by varying title
and time-range-badge text lengths (disproportion is shown in the first
screenshot below).
![Screenshot 2025-05-02 at 19 39
01](https://github.com/user-attachments/assets/23c4a4ae-2dc0-4c04-9ecf-ed4abfba8c76)
Now, each child (which is title and time-range-badge) will grow/shrink
proportionally, ensuring consistent layout regardless of their content
length. (as shown in the screenshots below)
![Screenshot 2025-05-02 at 19 39
30](https://github.com/user-attachments/assets/a48d1c9e-2bb0-4c55-96be-3b379091e904)
![Screenshot 2025-05-02 at 19 40
06](https://github.com/user-attachments/assets/72171174-6565-412a-abb9-1fc53b7b094c)

---------

Co-authored-by: Hannah Mudge <Heenawter@users.noreply.github.com>
Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:version Backport to applied version labels bug Fixes for quality problems that affect the customer experience Feature:Dashboard Dashboard related features impact:high Addressing this issue will have a high level of impact on the quality/strength of our product. loe:small Small Level of Effort regression release_note:fix Team:Presentation Presentation Team for Dashboard, Input Controls, and Canvas t// v8.18.1 v8.18.2 v8.19.0 v9.0.1 v9.0.2 v9.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Dashboard] Description icon missing [Dashboards] Custom time range badge hides in panel heading

8 participants