Skip to content

[9.0] [Search][Accessibility] Show error text when fields are invalid (#238284)#238505

Merged
seialkali merged 1 commit into
elastic:9.0from
seialkali:backport/9.0/pr-238284
Oct 13, 2025
Merged

[9.0] [Search][Accessibility] Show error text when fields are invalid (#238284)#238505
seialkali merged 1 commit into
elastic:9.0from
seialkali:backport/9.0/pr-238284

Conversation

@seialkali
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 9.0:

Questions ?

Please refer to the Backport tool documentation

…tic#238284)

## Summary

This PR ensures that when a field in the RAG Playground is in an error
state, the corresponding error text is displayed below the field.
Previously, only the red error styling was applied without any
accompanying message, which did not meet accessibility requirements.

<img width="600" height="686" alt="Screenshot 2025-10-09 at 16 10 06"
src="https://github.com/user-attachments/assets/ef980bcb-cae0-4c5c-a330-c4e20bbbfd61"
/>

### Testing

1. Navigate to Search → Build → RAG Playground.
2. In the Instructions field, delete all text and press Tab.
3. Confirm that an error message appears below the field (e.g.,
“Instructions cannot be empty”).
4. In the Context fields input, enter any text (e.g., test), then press
Tab.
5. Confirm that invalid entries display a red border and an error
message below the field.

### Checklist

Check the PR satisfies following conditions.

Reviewers should verify this PR satisfies this list as well.

- [x] Any text added follows [EUI's writing
guidelines](https://elastic.github.io/eui/#/guidelines/writing), uses
sentence case text and includes [i18n
support](https://github.com/elastic/kibana/blob/main/src/platform/packages/shared/kbn-i18n/README.md)
- [ ]
~[Documentation](https://www.elastic.co/guide/en/kibana/master/development-documentation.html)
was added for features that require explanation or tutorials~
- [ ] ~[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~
- [ ] ~If a plugin configuration key changed, check if it needs to be
allowlisted in the cloud and added to the [docker
list](https://github.com/elastic/kibana/blob/main/src/dev/build/tasks/os_packages/docker_generator/resources/base/bin/kibana-docker)~
- [ ] ~This was checked for breaking HTTP API changes, and any breaking
changes have been approved by the breaking-change committee. The
`release_note:breaking` label should be applied in these situations.~
- [ ] ~[Flaky Test
Runner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was
used on any tests changed~
- [x] 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)
- [x] Review the [backport
guidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing)
and apply applicable `backport:*` labels.

## Release note

Fixes an issue in the RAG Playground where invalid fields displayed red
styling but no error message. Error text is now shown to help users
identify and correct form issues.

(cherry picked from commit 3a56deb)

# Conflicts:
#	x-pack/solutions/search/plugins/search_playground/public/components/edit_context/context_fields_select.tsx
@seialkali seialkali added the backport This PR is a backport of another PR label Oct 10, 2025
@seialkali seialkali enabled auto-merge (squash) October 10, 2025 16:17
@elasticmachine
Copy link
Copy Markdown
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
searchPlayground 185.8KB 186.0KB +189.0B

@seialkali
Copy link
Copy Markdown
Contributor Author

@elasticmachine merge upstream

@elasticmachine
Copy link
Copy Markdown
Contributor

There are no new commits on the base branch.

@elastic elastic deleted a comment from elasticmachine Oct 13, 2025
@seialkali seialkali disabled auto-merge October 13, 2025 10:23
@seialkali seialkali enabled auto-merge (squash) October 13, 2025 10:27
@seialkali seialkali merged commit 8eaf25a into elastic:9.0 Oct 13, 2025
11 of 12 checks passed
@seialkali seialkali deleted the backport/9.0/pr-238284 branch October 13, 2025 12:33
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