Remove redundant ARIA roles (#6562)

* Remove redundant ARIA roles

The following elements already have intrinsic ARIA roles, so specifying them
again with role="..." is unnecessary.

- <table> has intrinsic role "table"
- <tr> has intrinsic role "row"
- <td> has intrinsic role "cell"
- <hr> has intrinsic role "separator"

* docs: add changelog

Co-authored-by: Segun Adebayo <joseshegs@gmail.com>

.changeset/brown-hairs-behave.md

@@ -0,0 +1,6 @@
+---
+"@chakra-ui/menu": patch
+"@chakra-ui/table": patch
+---
+
+Remove redundant ARIA roles

packages/components/menu/src/menu-divider.tsx

@@ -10,7 +10,6 @@ export const MenuDivider: React.FC<MenuDividerProps> = (props) => {
   const styles = useMenuStyles()
   return (
     <chakra.hr
-      role="separator"
       aria-orientation="horizontal"
       className={cx("chakra-menu__divider", className)}
       {...rest}

packages/components/table/src/table.tsx

@@ -33,7 +33,6 @@ export const Table = forwardRef<TableProps, "table">((props, ref) => {
   return (
     <TableStylesProvider value={styles}>
       <chakra.table
-        role="table"
         ref={ref}
         __css={styles.table}
         className={cx("chakra-table", className)}

packages/components/table/src/td.tsx

@@ -13,7 +13,6 @@ export const Td = forwardRef<TableCellProps, "td">(
 
     return (
       <chakra.td
-        role="gridcell"
         {...rest}
         ref={ref}
         __css={styles.td}

packages/components/table/src/tr.tsx

@@ -5,5 +5,5 @@ export interface TableRowProps extends HTMLChakraProps<"tr"> {}
 export const Tr = forwardRef<TableRowProps, "tr">((props, ref) => {
   const styles = useTableStyles()
 
-  return <chakra.tr role="row" {...props} ref={ref} __css={styles.tr} />
+  return <chakra.tr {...props} ref={ref} __css={styles.tr} />
 })