Skip to content

[Lens powered by ES|QL] Update Switch to Query mode modal warning message#251051

Merged
mariairiartef merged 4 commits intoelastic:mainfrom
mariairiartef:lens/update-modal-warning-message
Jan 30, 2026
Merged

[Lens powered by ES|QL] Update Switch to Query mode modal warning message#251051
mariairiartef merged 4 commits intoelastic:mainfrom
mariairiartef:lens/update-modal-warning-message

Conversation

@mariairiartef
Copy link
Copy Markdown
Contributor

@mariairiartef mariairiartef commented Jan 30, 2026

Summary

Close https://github.com/elastic/kibana-team/issues/2735

The warning message in the "Switch to query mode" modal stated "Once query mode is activated you cannot switch back to visual mode." However, this only applies after saving the changes via the "Apply and close" button. We've updated the message to clarify that users can still revert the conversion before clicking "Apply and close".

Additionally, we have updated the modal title to "Switch to query mode" for consistency.

Screenshot 2026-01-30 at 15 57 20

Checklist

@mariairiartef mariairiartef self-assigned this Jan 30, 2026
@mariairiartef mariairiartef requested a review from a team as a code owner January 30, 2026 12:36
@mariairiartef mariairiartef added Team:Visualizations Team label for Lens, elastic-charts, Graph, legacy editors (TSVB, Visualize, Timelion) t// release_note:skip Skip the PR/issue when compiling release notes Feature:Lens backport:skip This PR does not require backporting labels Jan 30, 2026
@elasticmachine
Copy link
Copy Markdown
Contributor

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

@mariairiartef mariairiartef requested a review from Copilot January 30, 2026 12:39
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the warning message in the "Switch to Query mode" modal to clarify that users can revert the conversion before saving changes. The previous message incorrectly implied an immediate irreversible change, while the updated message specifies that the conversion only becomes permanent after saving the chart.

Changes:

  • Updated the warning message text to indicate that switching to query mode is only irreversible after saving the chart

@mariairiartef mariairiartef enabled auto-merge (squash) January 30, 2026 14:00
@mariairiartef mariairiartef enabled auto-merge (squash) January 30, 2026 14:59
@mariairiartef mariairiartef merged commit 86124db into elastic:main Jan 30, 2026
16 checks passed
hannahbrooks pushed a commit to hannahbrooks/kibana that referenced this pull request Jan 30, 2026
…sage (elastic#251051)

## Summary

Close elastic/kibana-team#2735

The warning message in the "Switch to query mode" modal stated "Once
query mode is activated you cannot switch back to visual mode." However,
this only applies after saving the changes via the "Apply and close"
button. We've updated the message to clarify that users can still revert
the conversion before clicking "Apply and close".

Additionally, we have updated the modal title to "Switch to query mode"
for consistency.

<img width="624" height="427" alt="Screenshot 2026-01-30 at 15 57 20"
src="https://github.com/user-attachments/assets/6da31673-278f-4d8f-ac06-a14a1a42a9da"
/>

## Checklist

- [x] Any text added follows [EUI's writing
guidelines](https://elastic.github.io/eui/#/guidelines/writing), uses
sentence case text and includes [i18n
support](https://github.com/elastic/kibana/blob/main/src/platform/packages/shared/kbn-i18n/README.md)
- [x] The PR description includes the appropriate Release Notes section,
and the correct `release_note:*` label is applied per the
[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)
- [x] Review the [backport
guidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing)
and apply applicable `backport:*` labels.
mbondyra added a commit to mbondyra/kibana that referenced this pull request Feb 2, 2026
…iew_cps

* commit '32efd9b2fb078ade51073fd2d0068bc74c029d6b': (49 commits)
  [Security Solution] Rules exceptions subfeatures (elastic#245722)
  [BK] Upgrade axios (elastic#251150)
  Fix AI Connector form fields resetting to default value when cleared by user (elastic#251095)
  deduplicate otel dependencies (elastic#250841)
  Adds initial agents.md file (elastic#250833)
  [index management] Faster index list loading (elastic#246276)
  skip failing test suite (elastic#251086)
  skip failing test suite (elastic#251048)
  [Security Solutions] Trial Companion - adjust UX design (elastic#250910)
  [Traces][Discover] Prevent flyout remount when switching document types in Trace Waterfall (elastic#250406)
  [DOCS][Cases][9.4 & Serverless]: Doc new `Maximum amount of cases to open` setting for case action (elastic#250993)
  [Discover][Traces] Explore trace.id from logs in Discover (elastic#249632)
  Remove ! from SOs docs link (elastic#251097)
  [ML] Maps: Add telemetry events for file uploads (elastic#247543)
  [Fleet] Fix dupplicate ids when copying an integration policy or an agent policy (elastic#250971)
  [Dashboards as Code] Add snake case object keys util (elastic#250962)
  [Core] Remove URL Overflow & Deprecate `storeInSessionStorage` setting (elastic#242972)
  [One Workflow] fix: Fix Variable Retrieval in Workflow Execution Engine (elastic#250852)
  Rework Elastic Managed LLMs page (elastic#251069)
  [Lens powered by ES|QL] Update Switch to Query mode modal warning message (elastic#251051)
  ...
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// v9.4.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants