Skip to content

[SharedUX] Make solution switch callout dismissible#260469

Merged
angeles-mb merged 2 commits intoelastic:mainfrom
angeles-mb:256675-make-solution-switch-callout-dismissible
Apr 6, 2026
Merged

[SharedUX] Make solution switch callout dismissible#260469
angeles-mb merged 2 commits intoelastic:mainfrom
angeles-mb:256675-make-solution-switch-callout-dismissible

Conversation

@angeles-mb
Copy link
Copy Markdown
Contributor

@angeles-mb angeles-mb commented Mar 31, 2026 •
edited
Loading

Closes #260303

Summary

This PR:

  • Updates callout and modal strings as suggested on previous PR [SharedUX] Get spaces callout on each solution nav.
  • Replaces the page reload after space update with navigating to the solution home page.
  • Makes the callout dismissible and stores its dismissal in local storage. Dismissal, unlike tour flag is not space-specific, once dismissed it will be dismissed for all spaces.

Testing

Search Screenshot 2026-03-31 at 11 28 57
Security (custom styles) Screenshot 2026-03-31 at 11 31 09
Modal Screenshot 2026-03-31 at 11 29 38

Callout dismissal:

Screen.Recording.2026-03-31.at.12.05.46.mov

@elasticmachine
Copy link
Copy Markdown
Contributor

elasticmachine commented Mar 31, 2026 •
edited
Loading

🤖 Jobs for this PR can be triggered through checkboxes. 🚧

ℹ️ To trigger the CI, please tick the checkbox below 👇

  • Click to trigger kibana-pull-request for this PR!
  • Click to trigger kibana-deploy-project-from-pr for this PR!
  • Click to trigger kibana-deploy-cloud-from-pr for this PR!
  • Click to trigger kibana-entity-store-performance-from-pr for this PR!
  • Click to trigger kibana-storybooks-from-pr for this PR!

@angeles-mb angeles-mb self-assigned this Mar 31, 2026
@angeles-mb angeles-mb added release_note:skip Skip the PR/issue when compiling release notes backport:skip This PR does not require backporting labels Mar 31, 2026
@angeles-mb angeles-mb force-pushed the 256675-make-solution-switch-callout-dismissible branch from d3f80d3 to 8ecfb0e Compare March 31, 2026 13:05
@angeles-mb angeles-mb force-pushed the 256675-make-solution-switch-callout-dismissible branch from 8ecfb0e to 8780676 Compare March 31, 2026 13:41
@angeles-mb angeles-mb added the Team:SharedUX Platform AppEx-SharedUX (formerly Global Experience) t// label Mar 31, 2026
@angeles-mb angeles-mb marked this pull request as ready for review March 31, 2026 16:22
@angeles-mb angeles-mb requested review from a team as code owners March 31, 2026 16:22
PhilippeOberti
PhilippeOberti approved these changes Mar 31, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@PhilippeOberti PhilippeOberti left a comment

Choose a reason for hiding this comment

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

Code review only, LGTM for the @elastic/security-threat-hunting-investigations team

@angeles-mb angeles-mb merged commit b988bf3 into elastic:main Apr 6, 2026
18 checks passed
@elasticmachine
Copy link
Copy Markdown
Contributor

elasticmachine commented Apr 6, 2026

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #143 / discover/cascade layout grouping data fetching does not refetch when returning to a previously expanded group
  • [job] [logs] affected Scout: [ platform / alerting_v2 ] plugin / local-serverless-security_complete - Discover Alerts menu with alerting v2 - should show Alerts menu with v2 ES|QL rule row and New badge

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
agentBuilder 800.4KB 800.3KB -136.0B
automaticImport 127.7KB 127.6KB -136.0B
canvas 1.0MB 1.0MB -136.0B
contentConnectors 467.0KB 466.9KB -136.0B
dashboard 862.9KB 862.7KB -136.0B
dataSources 251.1KB 251.0KB -136.0B
enterpriseSearch 957.3KB 957.2KB -136.0B
eventAnnotationListing 208.7KB 208.6KB -136.0B
filesManagement 104.2KB 104.1KB -136.0B
graph 374.1KB 374.0KB -136.0B
home 72.7KB 72.6KB -136.0B
indexManagement 789.7KB 789.6KB -136.0B
infra 1.3MB 1.3MB -136.0B
kibanaOverview 47.5KB 47.4KB -136.0B
management 32.9KB 32.7KB -136.0B
maps 3.2MB 3.2MB -136.0B
ml 5.8MB 5.8MB -136.0B
observabilityShared 37.4KB 37.2KB -136.0B
osquery 1.3MB 1.3MB -136.0B
searchGettingStarted 235.9KB 235.8KB -136.0B
searchHomepage 188.0KB 187.9KB -136.0B
searchInferenceEndpoints 212.6KB 212.5KB -136.0B
searchPlayground 250.0KB 249.8KB -136.0B
searchQueryRules 131.4KB 131.3KB -136.0B
searchSynonyms 65.3KB 65.1KB -136.0B
security 494.7KB 494.5KB -136.0B
securitySolution 11.6MB 11.6MB -173.0B
securitySolutionEss 45.5KB 45.4KB -136.0B
securitySolutionServerless 63.5KB 63.3KB -136.0B
spaces 235.9KB 236.3KB +428.0B
triggersActionsUi 1.8MB 1.8MB -136.0B
visualizationListing 97.0KB 96.9KB -136.0B
visualizations 334.3KB 334.2KB -136.0B
workplaceAIApp 520.2KB 520.1KB -136.0B
total -4.0KB

Page load bundle

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

id before after diff
esUiShared 96.7KB 96.6KB -136.0B
spaces 25.8KB 26.1KB +299.0B
total +163.0B

History

cc @angeles-mb

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@PhilippeOberti PhilippeOberti PhilippeOberti approved these changes

@kowalczyk-krzysztof kowalczyk-krzysztof kowalczyk-krzysztof approved these changes

@rgodfrey-elastic rgodfrey-elastic rgodfrey-elastic approved these changes

Assignees

@angeles-mb angeles-mb

Labels

backport:skip This PR does not require backporting release_note:skip Skip the PR/issue when compiling release notes Team:SharedUX Platform AppEx-SharedUX (formerly Global Experience) t// v9.4.0

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Make callout dismissible

6 participants

@angeles-mb @elasticmachine @PhilippeOberti @kowalczyk-krzysztof @rgodfrey-elastic @kibanamachine