Skip to content

[8.19] Fix: success message and scroll after adding panel from library (#220122)#220703

Merged
kibanamachine merged 1 commit intoelastic:8.19from
kibanamachine:backport/8.19/pr-220122
May 9, 2025
Merged

[8.19] Fix: success message and scroll after adding panel from library (#220122)#220703
kibanamachine merged 1 commit intoelastic:8.19from
kibanamachine:backport/8.19/pr-220122

Conversation

@kibanamachine
Copy link
Contributor

Backport

This will backport the following commits from main to 8.19:

Questions ?

Please refer to the Backport tool documentation

…tic#220122)

Initially there was no success message for any type in the library and
there was no scrolling to added panel.

This change sets displaySuccessMessage to true for each plugin
registered in the Add from Library flyout.

It ensures that:
- users see a success toast after adding a panel,
- the newly added panel is automatically scrolled into view.

Closes: elastic#188775

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
(cherry picked from commit e7e8d2f)
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label May 9, 2025
@kibanamachine kibanamachine enabled auto-merge (squash) May 9, 2025 17:31
@botelastic botelastic bot added the Feature:Embedding Embedding content via iFrame label May 9, 2025
@kibanamachine kibanamachine merged commit 7501611 into elastic:8.19 May 9, 2025
13 checks passed
@elasticmachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #17 / Security Solution - Telemetry Security Telemetry - Indices metadata task telemetry @ess @serverless indices metadata should publish index stats events

Metrics [docs]

Page load bundle

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

id before after diff
discover 21.9KB 21.9KB +3.0B
lens 58.8KB 58.8KB +3.0B
links 7.7KB 7.8KB +3.0B
maps 40.1KB 40.1KB +3.0B
visualizations 35.1KB 35.1KB +3.0B
total +15.0B

cc @olapawlus

olapawlus added a commit that referenced this pull request May 14, 2025
…#220122) (#220776)

# Backport

This will backport the following commits from `main` to `9.0`:
- [Fix: success message and scroll after adding panel from library
(#220122)](#220122)

<!--- 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-09T17:24:49Z","message":"Fix:
success message and scroll after adding panel from library
(#220122)\n\nInitially there was no success message for any type in the
library and\nthere was no scrolling to added panel.\n\nThis change sets
displaySuccessMessage to true for each plugin\nregistered in the Add
from Library flyout.\n\nIt ensures that:\n- users see a success toast
after adding a panel,\n- the newly added panel is automatically scrolled
into view.\n\nCloses: #188775\n\n---------\n\nCo-authored-by:
kibanamachine
<42973632+kibanamachine@users.noreply.github.com>","sha":"e7e8d2f13b44ae78b33ada4a3904b17066c6fd37","branchLabelMapping":{"^v9.1.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["bug","Feature:Dashboard","Feature:Embedding","regression","release_note:fix","Team:Presentation","loe:small","impact:high","backport:version","v9.1.0","v8.19.0","v8.18.2","v9.0.2"],"title":"Fix:
success message and scroll after adding panel from
library","number":220122,"url":"https://github.com/elastic/kibana/pull/220122","mergeCommit":{"message":"Fix:
success message and scroll after adding panel from library
(#220122)\n\nInitially there was no success message for any type in the
library and\nthere was no scrolling to added panel.\n\nThis change sets
displaySuccessMessage to true for each plugin\nregistered in the Add
from Library flyout.\n\nIt ensures that:\n- users see a success toast
after adding a panel,\n- the newly added panel is automatically scrolled
into view.\n\nCloses: #188775\n\n---------\n\nCo-authored-by:
kibanamachine
<42973632+kibanamachine@users.noreply.github.com>","sha":"e7e8d2f13b44ae78b33ada4a3904b17066c6fd37"}},"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/220122","number":220122,"mergeCommit":{"message":"Fix:
success message and scroll after adding panel from library
(#220122)\n\nInitially there was no success message for any type in the
library and\nthere was no scrolling to added panel.\n\nThis change sets
displaySuccessMessage to true for each plugin\nregistered in the Add
from Library flyout.\n\nIt ensures that:\n- users see a success toast
after adding a panel,\n- the newly added panel is automatically scrolled
into view.\n\nCloses: #188775\n\n---------\n\nCo-authored-by:
kibanamachine
<42973632+kibanamachine@users.noreply.github.com>","sha":"e7e8d2f13b44ae78b33ada4a3904b17066c6fd37"}},{"branch":"8.19","label":"v8.19.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"url":"https://github.com/elastic/kibana/pull/220703","number":220703,"state":"MERGED","mergeCommit":{"sha":"7501611e3277d378ea00e1731b39be0ebeb7769b","message":"[8.19]
Fix: success message and scroll after adding panel from library
(#220122) (#220703)\n\n# Backport\n\nThis will backport the following
commits from `main` to `8.19`:\n- [Fix: success message and scroll after
adding panel from
library\n(#220122)](https://github.com/elastic/kibana/pull/220122)\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>"}},{"branch":"8.18","label":"v8.18.2","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"9.0","label":"v9.0.2","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
olapawlus added a commit that referenced this pull request May 14, 2025
#220122) (#220777)

# Backport

This will backport the following commits from `main` to `8.18`:
- [Fix: success message and scroll after adding panel from library
(#220122)](#220122)

<!--- 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-09T17:24:49Z","message":"Fix:
success message and scroll after adding panel from library
(#220122)\n\nInitially there was no success message for any type in the
library and\nthere was no scrolling to added panel.\n\nThis change sets
displaySuccessMessage to true for each plugin\nregistered in the Add
from Library flyout.\n\nIt ensures that:\n- users see a success toast
after adding a panel,\n- the newly added panel is automatically scrolled
into view.\n\nCloses: #188775\n\n---------\n\nCo-authored-by:
kibanamachine
<42973632+kibanamachine@users.noreply.github.com>","sha":"e7e8d2f13b44ae78b33ada4a3904b17066c6fd37","branchLabelMapping":{"^v9.1.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["bug","Feature:Dashboard","Feature:Embedding","regression","release_note:fix","Team:Presentation","loe:small","impact:high","backport:version","v9.1.0","v8.19.0","v8.18.2","v9.0.2"],"title":"Fix:
success message and scroll after adding panel from
library","number":220122,"url":"https://github.com/elastic/kibana/pull/220122","mergeCommit":{"message":"Fix:
success message and scroll after adding panel from library
(#220122)\n\nInitially there was no success message for any type in the
library and\nthere was no scrolling to added panel.\n\nThis change sets
displaySuccessMessage to true for each plugin\nregistered in the Add
from Library flyout.\n\nIt ensures that:\n- users see a success toast
after adding a panel,\n- the newly added panel is automatically scrolled
into view.\n\nCloses: #188775\n\n---------\n\nCo-authored-by:
kibanamachine
<42973632+kibanamachine@users.noreply.github.com>","sha":"e7e8d2f13b44ae78b33ada4a3904b17066c6fd37"}},"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/220122","number":220122,"mergeCommit":{"message":"Fix:
success message and scroll after adding panel from library
(#220122)\n\nInitially there was no success message for any type in the
library and\nthere was no scrolling to added panel.\n\nThis change sets
displaySuccessMessage to true for each plugin\nregistered in the Add
from Library flyout.\n\nIt ensures that:\n- users see a success toast
after adding a panel,\n- the newly added panel is automatically scrolled
into view.\n\nCloses: #188775\n\n---------\n\nCo-authored-by:
kibanamachine
<42973632+kibanamachine@users.noreply.github.com>","sha":"e7e8d2f13b44ae78b33ada4a3904b17066c6fd37"}},{"branch":"8.19","label":"v8.19.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"url":"https://github.com/elastic/kibana/pull/220703","number":220703,"state":"MERGED","mergeCommit":{"sha":"7501611e3277d378ea00e1731b39be0ebeb7769b","message":"[8.19]
Fix: success message and scroll after adding panel from library
(#220122) (#220703)\n\n# Backport\n\nThis will backport the following
commits from `main` to `8.19`:\n- [Fix: success message and scroll after
adding panel from
library\n(#220122)](https://github.com/elastic/kibana/pull/220122)\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>"}},{"branch":"8.18","label":"v8.18.2","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"9.0","label":"v9.0.2","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
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 Feature:Embedding Embedding content via iFrame

3 participants