Skip to content

[8.x] [ES|QL] new pattern for SORT autocomplete (#193595)#193801

Merged
drewdaemon merged 1 commit intoelastic:8.xfrom
drewdaemon:backport/8.x/pr-193595
Sep 24, 2024
Merged

[8.x] [ES|QL] new pattern for SORT autocomplete (#193595)#193801
drewdaemon merged 1 commit intoelastic:8.xfrom
drewdaemon:backport/8.x/pr-193595

Conversation

@drewdaemon
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 8.x:

Questions ?

Please refer to the Backport tool documentation

## Summary

Part of elastic#189662. This PR
- updates the autocomplete behavior for `SORT` to be in line with other
field-list-based experiences like `KEEP`
- introduces a shared function, `handleFragment`, which is used to
abstract some of the logic required to support this behavior
- bulks up the `SORT` tests
- restores the function suggestions which I noticed got lost in
elastic#189959

**Before**

https://github.com/user-attachments/assets/cad1d073-c010-426f-9628-c0fc6b65eb3c

**After**

https://github.com/user-attachments/assets/e148ae58-4430-482c-9f8e-c55779c4d822

### Checklist

- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios

---------

Co-authored-by: Stratoula Kalafateli <efstratia.kalafateli@elastic.co>
(cherry picked from commit f450e22)
@drewdaemon drewdaemon added the backport This PR is a backport of another PR label Sep 23, 2024
@drewdaemon drewdaemon enabled auto-merge (squash) September 23, 2024 21:21
@kibana-ci
Copy link
Copy Markdown

💚 Build Succeeded

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
kbnUiSharedDeps-srcJs 3.4MB 3.4MB +874.0B

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@drewdaemon drewdaemon merged commit 64e4646 into elastic:8.x Sep 24, 2024
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants