Skip to content

[9.0] [Search][a11y] show tooltip show more or fewer fields (#237913)#238334

Merged
saarikabhasi merged 2 commits into
elastic:9.0from
saarikabhasi:backport/9.0/pr-237913
Oct 14, 2025
Merged

[9.0] [Search][a11y] show tooltip show more or fewer fields (#237913)#238334
saarikabhasi merged 2 commits into
elastic:9.0from
saarikabhasi:backport/9.0/pr-237913

Conversation

@saarikabhasi
Copy link
Copy Markdown
Member

Backport

This will backport the following commits from main to 9.0:

Questions ?

Please refer to the Backport tool documentation

## Summary

Tool tip is not shown when navigated via keyboard. Adding ref helps with
showing the tool tip when show more/hide button is on focus.

### Testing instructions:
1. In Stateful. Create an Index with documents more than 3 fields. Open
documents page and has more than 3 documents
2. Navigate to the first (or any) table while using only keyboard by
pressing Tab key.
3. Navigate to Show more fields button.
4. Press Esc.
5. Press Enter.
6. Observe tooltips for the button.

Also test for:
Same with the opposite: when user navigates to Show more button ->
presses Enter -> presses Esc -> presses Enter. Result: no tooltip
present on Show more button.

### Checklist

Check the PR satisfies following conditions.

Reviewers should verify this PR satisfies this list as well.

- [ ] 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)
- [ ] Review the [backport
guidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing)
and apply applicable `backport:*` labels.

(cherry picked from commit 37446f0)

# Conflicts:
#	x-pack/platform/packages/shared/kbn-search-index-documents/components/result/result.tsx
@saarikabhasi saarikabhasi added the backport This PR is a backport of another PR label Oct 9, 2025
@saarikabhasi saarikabhasi enabled auto-merge (squash) October 9, 2025 19:21
@elasticmachine
Copy link
Copy Markdown
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

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
enterpriseSearch 1.3MB 1.3MB +83.0B
searchIndices 172.1KB 172.2KB +83.0B
searchPlayground 185.8KB 185.9KB +83.0B
serverlessSearch 343.1KB 343.2KB +83.0B
total +332.0B

Copy link
Copy Markdown
Contributor

@Samiul-TheSoccerFan Samiul-TheSoccerFan left a comment

Choose a reason for hiding this comment

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

LGTM

@saarikabhasi saarikabhasi merged commit 01123b5 into elastic:9.0 Oct 14, 2025
8 of 9 checks passed
@saarikabhasi saarikabhasi deleted the backport/9.0/pr-237913 branch October 14, 2025 22:53
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