[A11y] Fix space button clearing the index name input

jedrazb commented 3 days ago

Summary

Let's just disallow space input as index names can't contain spaces anyway. This fixes an a11y issue raised recently.

Fix (im clicking space a lot in the vid):

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, uses sentence case text and includes i18n support
[x] Documentation was added for features that require explanation or tutorials
[x] Unit or functional tests were updated or added to match the most common scenarios
[x] If a plugin configuration key changed, check if it needs to be allowlisted in the cloud and added to the docker list
[x] 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.
[x] Flaky Test Runner 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

elasticmachine commented 3 days ago

[job] [logs] Jest Tests #4 / it should allow to add a new host
[job] [logs] Jest Tests #4 / OutputHealth should render output health component when degraded

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id	before	after	diff
`enterpriseSearch`	2.6MB	2.6MB	+47.0B

kibanamachine commented 3 days ago

Starting backport for target branches: 8.x

kibanamachine commented 3 days ago

Status	Branch	Result
✅	8.x

Note: Successful backport PRs will be merged automatically after passing CI.