Skip to content

[Discover] Add "Shift + Select" functionality to Discover grid#193619

Merged
jughosta merged 5 commits intoelastic:mainfrom
jughosta:192366-shift-select-checkboxes
Sep 25, 2024
Merged

[Discover] Add "Shift + Select" functionality to Discover grid#193619
jughosta merged 5 commits intoelastic:mainfrom
jughosta:192366-shift-select-checkboxes

Conversation

@jughosta
Copy link
Copy Markdown
Contributor

@jughosta jughosta commented Sep 20, 2024

Summary

This PR allows to select/deselect multiple rows by holding SHIFT key when toggling row checkboxes.

Checklist

@jughosta jughosta added release_note:enhancement Team:DataDiscovery Discover, search (data plugin and KQL), data views, saved searches. For ES|QL, use Team:ES|QL. t// backport:prev-minor Feature:UnifiedDataTable Project:OneDiscover Enrich Discover with contextual awareness labels Sep 20, 2024
@jughosta jughosta self-assigned this Sep 20, 2024
@jughosta jughosta marked this pull request as ready for review September 23, 2024 13:33
@jughosta jughosta requested review from a team as code owners September 23, 2024 13:33
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/kibana-data-discovery (Team:DataDiscovery)

Copy link
Copy Markdown
Contributor

@davismcphee davismcphee left a comment

Choose a reason for hiding this comment

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

Awesome, I love it! So much more convenient than selecting individual documents or pages only. Works great for keyboard users too 👍

Copy link
Copy Markdown
Contributor

@logeekal logeekal left a comment

Choose a reason for hiding this comment

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

LGTM from investigations side. This is a great UX improvement. Thanks.

@jughosta jughosta enabled auto-merge (squash) September 25, 2024 08:10
@kibana-ci
Copy link
Copy Markdown

💚 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
cloudSecurityPosture 496.8KB 497.4KB +613.0B
discover 812.9KB 813.5KB +613.0B
esqlDataGrid 153.0KB 153.6KB +613.0B
securitySolution 20.4MB 20.4MB +3.0KB
slo 854.1KB 854.7KB +613.0B
total +5.4KB

History

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

cc @jughosta

@jughosta jughosta merged commit 6808f82 into elastic:main Sep 25, 2024
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Sep 25, 2024
…ic#193619)

- Closes elastic#192366

## Summary

This PR allows to select/deselect multiple rows by holding SHIFT key
when toggling row checkboxes.

### 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
- [x] This was checked for [cross-browser
compatibility](https://www.elastic.co/support/matrix#matrix_browsers)

(cherry picked from commit 6808f82)
@kibanamachine
Copy link
Copy Markdown
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 Sep 25, 2024
…over grid (#193619) (#193955)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[Discover] Add "Shift + Select" functionality to Discover
grid (#193619)](#193619)

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

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

<!--BACKPORT [{"author":{"name":"Julia
Rechkunova","email":"julia.rechkunova@elastic.co"},"sourceCommit":{"committedDate":"2024-09-25T09:38:49Z","message":"[Discover]
Add \"Shift + Select\" functionality to Discover grid (#193619)\n\n-
Closes https://github.com/elastic/kibana/issues/192366\r\n\r\n##
Summary\r\n\r\nThis PR allows to select/deselect multiple rows by
holding SHIFT key\r\nwhen toggling row checkboxes.\r\n\r\n\r\n###
Checklist\r\n\r\n- [x] [Unit or
functional\r\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\r\nwere
updated or added to match the most common scenarios\r\n- [x] This was
checked for
[cross-browser\r\ncompatibility](https://www.elastic.co/support/matrix#matrix_browsers)","sha":"6808f826625f7f901099df6360a2f5e7edc21ab4","branchLabelMapping":{"^v9.0.0$":"main","^v8.16.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:enhancement","v9.0.0","Team:DataDiscovery","backport:prev-minor","Feature:UnifiedDataTable","Project:OneDiscover"],"title":"[Discover]
Add \"Shift + Select\" functionality to Discover
grid","number":193619,"url":"https://github.com/elastic/kibana/pull/193619","mergeCommit":{"message":"[Discover]
Add \"Shift + Select\" functionality to Discover grid (#193619)\n\n-
Closes https://github.com/elastic/kibana/issues/192366\r\n\r\n##
Summary\r\n\r\nThis PR allows to select/deselect multiple rows by
holding SHIFT key\r\nwhen toggling row checkboxes.\r\n\r\n\r\n###
Checklist\r\n\r\n- [x] [Unit or
functional\r\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\r\nwere
updated or added to match the most common scenarios\r\n- [x] This was
checked for
[cross-browser\r\ncompatibility](https://www.elastic.co/support/matrix#matrix_browsers)","sha":"6808f826625f7f901099df6360a2f5e7edc21ab4"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/193619","number":193619,"mergeCommit":{"message":"[Discover]
Add \"Shift + Select\" functionality to Discover grid (#193619)\n\n-
Closes https://github.com/elastic/kibana/issues/192366\r\n\r\n##
Summary\r\n\r\nThis PR allows to select/deselect multiple rows by
holding SHIFT key\r\nwhen toggling row checkboxes.\r\n\r\n\r\n###
Checklist\r\n\r\n- [x] [Unit or
functional\r\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\r\nwere
updated or added to match the most common scenarios\r\n- [x] This was
checked for
[cross-browser\r\ncompatibility](https://www.elastic.co/support/matrix#matrix_browsers)","sha":"6808f826625f7f901099df6360a2f5e7edc21ab4"}}]}]
BACKPORT-->

Co-authored-by: Julia Rechkunova <julia.rechkunova@elastic.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature:UnifiedDataTable Project:OneDiscover Enrich Discover with contextual awareness release_note:enhancement Team:DataDiscovery Discover, search (data plugin and KQL), data views, saved searches. For ES|QL, use Team:ES|QL. t// v8.16.0 v9.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add "Shift + Select"Functionality in Discover Table (Unified Table)

6 participants