Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[ES|QL] Give the ability to disable the autofocus on the editor #204706

Merged
merged 1 commit into from
Dec 19, 2024

Conversation

stratoula
Copy link
Contributor

@stratoula stratoula commented Dec 18, 2024

Summary

Right now, the editor autofocuses by default. But there are cases that a consumer won't want this (autofocus might trigger the suggestions).

This PR introduces a new property which allows the consumers to disable this feature.

(I need this for the ES|QL variables feature)

@stratoula stratoula added v9.0.0 backport:version Backport to applied version labels v8.18.0 Feature:ES|QL ES|QL related features in Kibana Team:ESQL ES|QL related features in Kibana release_note:skip Skip the PR/issue when compiling release notes labels Dec 18, 2024
@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
esql 202.5KB 202.5KB +26.0B
Unknown metric groups

API count

id before after diff
@kbn/esql-editor 29 30 +1
esql 25 26 +1
total +2

@stratoula stratoula marked this pull request as ready for review December 18, 2024 12:35
@stratoula stratoula requested a review from a team as a code owner December 18, 2024 12:35
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-esql (Team:ESQL)

@stratoula stratoula merged commit 65437c4 into elastic:main Dec 19, 2024
21 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.x

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

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Dec 19, 2024
…tic#204706)

## Summary

Right now, the editor autofocuses by default. But there are cases that a
consumer won't want this (autofocus might trigger the suggestions).

This PR introduces a new property which allows the consumers to disable
this feature.

(I need this for the ES|QL variables feature)

(cherry picked from commit 65437c4)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.x

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

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Dec 19, 2024
…#204706) (#204897)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[ES|QL] Give the ability to disable the autofocus on the editor
(#204706)](#204706)

<!--- Backport version: 9.4.3 -->

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

<!--BACKPORT [{"author":{"name":"Stratoula
Kalafateli","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-12-19T09:33:21Z","message":"[ES|QL]
Give the ability to disable the autofocus on the editor (#204706)\n\n##
Summary\r\n\r\nRight now, the editor autofocuses by default. But there
are cases that a\r\nconsumer won't want this (autofocus might trigger
the suggestions).\r\n\r\nThis PR introduces a new property which allows
the consumers to disable\r\nthis feature.\r\n\r\n(I need this for the
ES|QL variables
feature)","sha":"65437c4499a8a3627a2dc900f20d261666eeb27e","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","Feature:ES|QL","Team:ESQL","backport:version","v8.18.0"],"title":"[ES|QL]
Give the ability to disable the autofocus on the
editor","number":204706,"url":"https://github.com/elastic/kibana/pull/204706","mergeCommit":{"message":"[ES|QL]
Give the ability to disable the autofocus on the editor (#204706)\n\n##
Summary\r\n\r\nRight now, the editor autofocuses by default. But there
are cases that a\r\nconsumer won't want this (autofocus might trigger
the suggestions).\r\n\r\nThis PR introduces a new property which allows
the consumers to disable\r\nthis feature.\r\n\r\n(I need this for the
ES|QL variables
feature)","sha":"65437c4499a8a3627a2dc900f20d261666eeb27e"}},"sourceBranch":"main","suggestedTargetBranches":["8.x"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/204706","number":204706,"mergeCommit":{"message":"[ES|QL]
Give the ability to disable the autofocus on the editor (#204706)\n\n##
Summary\r\n\r\nRight now, the editor autofocuses by default. But there
are cases that a\r\nconsumer won't want this (autofocus might trigger
the suggestions).\r\n\r\nThis PR introduces a new property which allows
the consumers to disable\r\nthis feature.\r\n\r\n(I need this for the
ES|QL variables
feature)","sha":"65437c4499a8a3627a2dc900f20d261666eeb27e"}},{"branch":"8.x","label":"v8.18.0","branchLabelMappingKey":"^v8.18.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Stratoula Kalafateli <[email protected]>
JoseLuisGJ pushed a commit to JoseLuisGJ/kibana that referenced this pull request Dec 19, 2024
…tic#204706)

## Summary

Right now, the editor autofocuses by default. But there are cases that a
consumer won't want this (autofocus might trigger the suggestions).

This PR introduces a new property which allows the consumers to disable
this feature.

(I need this for the ES|QL variables feature)
benakansara pushed a commit to benakansara/kibana that referenced this pull request Jan 2, 2025
…tic#204706)

## Summary

Right now, the editor autofocuses by default. But there are cases that a
consumer won't want this (autofocus might trigger the suggestions).

This PR introduces a new property which allows the consumers to disable
this feature.

(I need this for the ES|QL variables feature)
CAWilson94 pushed a commit to CAWilson94/kibana that referenced this pull request Jan 13, 2025
…tic#204706)

## Summary

Right now, the editor autofocuses by default. But there are cases that a
consumer won't want this (autofocus might trigger the suggestions).

This PR introduces a new property which allows the consumers to disable
this feature.

(I need this for the ES|QL variables feature)
viduni94 pushed a commit to viduni94/kibana that referenced this pull request Jan 23, 2025
…tic#204706)

## Summary

Right now, the editor autofocuses by default. But there are cases that a
consumer won't want this (autofocus might trigger the suggestions).

This PR introduces a new property which allows the consumers to disable
this feature.

(I need this for the ES|QL variables feature)
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 Feature:ES|QL ES|QL related features in Kibana release_note:skip Skip the PR/issue when compiling release notes Team:ESQL ES|QL related features in Kibana v8.18.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants