From 0e207aff74d0d14f8db1151584a8c3940ee2a422 Mon Sep 17 00:00:00 2001 From: Jeremy Neal Date: Wed, 8 Feb 2023 13:56:19 -0500 Subject: [PATCH 001/135] Render ActionList.Group header inside list, use li for semantic HTML reasons. --- src/ActionList/Group.tsx | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/ActionList/Group.tsx b/src/ActionList/Group.tsx index f6332760c1f..eda267b1be3 100644 --- a/src/ActionList/Group.tsx +++ b/src/ActionList/Group.tsx @@ -58,7 +58,6 @@ export const Group: React.FC> = ({ }} {...props} > - {title &&
} > = ({ aria-labelledby={title ? labelId : undefined} role={role || (listRole && 'group')} > + {title &&
} {props.children} @@ -91,6 +91,7 @@ const Header: React.FC> = ({variant, title, fontSize: 0, fontWeight: 'bold', color: 'fg.muted', + listStyle: 'none', ...(variant === 'filled' && { backgroundColor: 'canvas.subtle', marginX: 0, @@ -102,7 +103,7 @@ const Header: React.FC> = ({variant, title, } return ( -