Skip to content

Comments

[9.1] [Controls] Hide Select All checkbox from single select controls (#226311)#226464

Closed
Zacqary wants to merge 1 commit intoelastic:9.1from
Zacqary:backport/9.1/pr-226311
Closed

[9.1] [Controls] Hide Select All checkbox from single select controls (#226311)#226464
Zacqary wants to merge 1 commit intoelastic:9.1from
Zacqary:backport/9.1/pr-226311

Conversation

@Zacqary
Copy link
Contributor

@Zacqary Zacqary commented Jul 3, 2025

Backport

This will backport the following commits from main to 9.1:

Questions ?

Please refer to the Backport tool documentation

…tic#226311)

## Summary

Controls configured to only allow a single selection
<img width="278" alt="Screenshot 2025-07-02 at 3 55 32 PM"
src="https://github.com/user-attachments/assets/193680b6-082e-4b89-b847-5803fe6c4cb6"
/>

Would still display a Select All checkbox and allow the user to select
multiple options this way
<img width="460" alt="Screenshot 2025-07-02 at 3 53 32 PM"
src="https://github.com/user-attachments/assets/7d384015-63db-4a76-9449-35c377280e46"
/>

This PR hides Select All if the single selection option is chosen:
<img width="459" alt="Screenshot 2025-07-02 at 3 53 42 PM"
src="https://github.com/user-attachments/assets/01eae209-d358-4e94-a6e2-6c1d9fb60aa1"
/>

### 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

(cherry picked from commit a3e5226)
@Zacqary Zacqary added the backport This PR is a backport of another PR label Jul 3, 2025
@Zacqary Zacqary requested a review from kibanamachine as a code owner July 3, 2025 16:38
@Zacqary Zacqary added the backport This PR is a backport of another PR label Jul 3, 2025
@Zacqary Zacqary enabled auto-merge (squash) July 3, 2025 16:38
@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
controls 477.9KB 477.9KB +22.0B

@Zacqary
Copy link
Contributor Author

Zacqary commented Jul 8, 2025

Duplicate

@Zacqary Zacqary closed this Jul 8, 2025
auto-merge was automatically disabled July 8, 2025 20:02

Pull request was closed

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.

2 participants