From deae64bb90f5e0a58181d866e03b731e158eb15e Mon Sep 17 00:00:00 2001 From: RoxanaAnamariaTurc <106086831+RoxanaAnamariaTurc@users.noreply.github.com> Date: Wed, 26 Jul 2023 16:45:04 +0100 Subject: [PATCH] A11y: Fixed keyboard accessibility in Tooltip Component (#72323) * A11y: Fixed keyboard accessibility in Tooltip Component * Made updates and fixed the keyboard accessibility --- .../src/components/Tooltip/Popover.tsx | 20 +++---------------- 1 file changed, 3 insertions(+), 17 deletions(-) diff --git a/packages/grafana-ui/src/components/Tooltip/Popover.tsx b/packages/grafana-ui/src/components/Tooltip/Popover.tsx index ea11c2f7aef..cb760b3cf5e 100644 --- a/packages/grafana-ui/src/components/Tooltip/Popover.tsx +++ b/packages/grafana-ui/src/components/Tooltip/Popover.tsx @@ -34,18 +34,8 @@ interface Props extends Omit, 'content'> { class Popover extends PureComponent { render() { - const { - content, - show, - placement, - onMouseEnter, - onMouseLeave, - className, - wrapperClassName, - renderArrow, - referenceElement, - onKeyDown, - } = this.props; + const { content, show, placement, className, wrapperClassName, renderArrow, referenceElement, ...rest } = + this.props; return ( @@ -66,12 +56,7 @@ class Popover extends PureComponent { > {({ ref, style, placement, arrowProps, update }) => { return ( - // TODO: fix keyboard a11y - // eslint-disable-next-line jsx-a11y/no-static-element-interactions
{ }} data-placement={placement} className={`${wrapperClassName}`} + {...rest} >
{typeof content === 'string' && content}