Skip to content

Commit

Permalink
Fixes accordion disclosure keyboard focus border (elastic#190436)
Browse files Browse the repository at this point in the history
## Summary

This fixes keyboard navigation focus border around the accordion element
on the Exploratory View.


https://github.com/user-attachments/assets/e863f52a-9745-4f55-acc5-d2aa12464f3c


### Checklist

Delete any items that are not applicable to this PR.

- [x] Any UI touched in this PR is usable by keyboard only (learn more
about [keyboard accessibility](https://webaim.org/techniques/keyboard/))
- [x] Any UI touched in this PR does not create any new axe failures
(run axe in browser:
[FF](https://addons.mozilla.org/en-US/firefox/addon/axe-devtools/),
[Chrome](https://chrome.google.com/webstore/detail/axe-web-accessibility-tes/lhdoppojpmngadmnindnejefpokejbdd?hl=en-US))
- [x] This renders correctly on smaller devices using a responsive
layout. (You can test this [in your
browser](https://www.browserstack.com/guide/responsive-testing-on-local-server))
- [x] This was checked for [cross-browser
compatibility](https://www.elastic.co/support/matrix#matrix_browsers)
  • Loading branch information
bryce-b authored Aug 13, 2024
1 parent be2b853 commit 6ee8051
Showing 1 changed file with 2 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,8 @@ export function Series({ item, isExpanded, toggleExpanded }: Props) {
aria-label={ACCORDION_LABEL}
onToggle={toggleExpanded}
arrowDisplay={!seriesProps.series.dataType ? 'none' : undefined}
extraAction={
buttonElement="div"
buttonContent={
<EuiFlexGroup alignItems="center" gutterSize="s" wrap>
<EuiFlexItem grow={false}>
<EuiFlexGroup alignItems="center" responsive={false} gutterSize="s">
Expand Down

0 comments on commit 6ee8051

Please sign in to comment.