Skip to content

Commit 00f0293

Browse files
author
Hector Arce De Las Heras
committed
Accessibility Update: Tooltip Trigger Dimensions
This commit updates the tooltip trigger to have a minimum width and height of 1.5rem (24px) to meet accessibility requirements. This ensures the tooltip trigger is adequately sized for all users.
1 parent 5d14329 commit 00f0293

File tree

3 files changed

+10
-2
lines changed

3 files changed

+10
-2
lines changed

src/components/tooltip/components/tooltipTrigger.tsx

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@ import { ButtonType } from '../../button';
44
import { TooltipTriggerWrapperStyled } from '../tooltip.styled';
55

66
interface ITooltipTrigger {
7+
disabled?: boolean;
78
childrenAsButton: boolean;
89
}
910

@@ -12,7 +13,7 @@ export const TooltipTrigger = (
1213
): React.ReactNode => {
1314
if (props.childrenAsButton) {
1415
return (
15-
<TooltipTriggerWrapperStyled type={ButtonType.BUTTON}>
16+
<TooltipTriggerWrapperStyled disabled={props.disabled} type={ButtonType.BUTTON}>
1617
{props.children}
1718
</TooltipTriggerWrapperStyled>
1819
);

src/components/tooltip/tooltip.styled.ts

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -107,4 +107,7 @@ export const TooltipTriggerWrapperStyled = styled.button`
107107
padding: 0;
108108
border: none;
109109
background: none;
110+
111+
min-height: 1.5rem; // accessible min-size
112+
min-width: 1.5rem; // accessible min-size
110113
`;

src/components/tooltip/tooltipStandAlone.tsx

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,11 @@ const TooltipStandAlone = ({
4141

4242
if (props.disabled) {
4343
return (
44-
<TooltipStyled data-testid={`${props.dataTestId}Tooltip`}>{props.children}</TooltipStyled>
44+
<TooltipStyled data-testid={`${props.dataTestId}Tooltip`}>
45+
<TooltipTrigger disabled childrenAsButton={childrenAsButton}>
46+
{props.children}
47+
</TooltipTrigger>
48+
</TooltipStyled>
4549
);
4650
}
4751

0 commit comments

Comments
 (0)