Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[8.x] fix: [Stateful: Home page] Most Ingest your content section buttons have duplicated actions on them (#196079) #196301

Merged
merged 1 commit into from
Oct 15, 2024

Conversation

kibanamachine
Copy link
Contributor

Backport

This will backport the following commits from main to 8.x:

Questions ?

Please refer to the Backport tool documentation

…ave duplicated actions on them (elastic#196079)

Closes: elastic#194932

## Summary

User reaches the same button two times when navigating using only
keyboard and it can get confusing. Also for the user using screen reader
it is also confusing if reached element is button or link. Better for
element to get focus only one time when navigating in sequence from one
element to another and for the user only to hear one announcement of the
element, button or link (but not button link).

## What was changed?:

1. Removed extra `EuiLinkTo` wrapper
2. `EuiButton` was replaced to `EuiButtonTo`

## Screen

<img width="1792" alt="image"
src="https://github.com/user-attachments/assets/597eb7f0-dd7b-4c14-bccd-d91dc3bdcbcf">

(cherry picked from commit 5ed6981)
@kibanamachine kibanamachine merged commit ebfc4d9 into elastic:8.x Oct 15, 2024
25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants