Tidy up types and JSDocs

packages/components/src/dropdown-menu/v2/README.md

@@ -1,6 +1,6 @@
 # `DropdownMenu` (v2)
 
-`DropdownMenu` aisplays a menu to the user (such as a set of actions or functions) triggered by a button.
+`DropdownMenu` displays a menu to the user (such as a set of actions or functions) triggered by a button.
 
 
 ## Design guidelines

packages/components/src/dropdown-menu/v2/index.tsx

@@ -19,7 +19,7 @@ import * as DropdownMenuStyled from './styles';
 import type {
 	DropdownMenuProps,
 	DropdownSubMenuProps,
-	DropdownItemProps,
+	DropdownMenuItemProps,
 	DropdownMenuLabelProps,
 	DropdownMenuGroupProps,
 	DropdownMenuCheckboxItemProps,
@@ -34,6 +34,10 @@ const SUB_MENU_OFFSET_SIDE = 12;
 // Opposite amount of the top padding of the menu item
 const SUB_MENU_OFFSET_ALIGN = -8;
 
+/**
+ * `DropdownMenu` displays a menu to the user (such as a set of actions
+ * or functions) triggered by a button.
+ */
 export const DropdownMenu = ( {
 	// Root props
 	defaultOpen,
@@ -149,7 +153,7 @@ export const DropdownMenuGroup = ( props: DropdownMenuGroupProps ) => (
 
 export const DropdownMenuItem = forwardRef(
 	(
-		{ children, prefix, suffix, ...props }: DropdownItemProps,
+		{ children, prefix, suffix, ...props }: DropdownMenuItemProps,
 		forwardedRef: React.ForwardedRef< any >
 	) => {
 		return (

packages/components/src/dropdown-menu/v2/types.ts

@@ -54,15 +54,13 @@ export type DropdownMenuProps = {
 	 */
 	alignOffset?: DropdownMenuPrimitive.DropdownMenuContentProps[ 'alignOffset' ];
 	/**
-	 * The contents rendered inside the trigger
+	 * The trigger button.
 	 */
 	trigger: React.ReactNode;
 	/**
 	 * The contents of the dropdown
 	 */
 	children: React.ReactNode;
-
-	// TODO: slot name ?
 };
 
 export type DropdownSubMenuTriggerProps = {
@@ -77,7 +75,7 @@ export type DropdownSubMenuTriggerProps = {
 	/**
 	 * The contents of the item's suffix.
 	 *
-	 * By default, it is the standard icon for a submenu trigger.
+	 * @default The standard chevron icon for a submenu trigger.
 	 */
 	suffix?: React.ReactNode;
 };
@@ -97,8 +95,6 @@ export type DropdownSubMenuProps = {
 	 * Event handler called when the open state of the submenu changes.
 	 */
 	onOpenChange?: DropdownMenuPrimitive.DropdownMenuSubProps[ 'onOpenChange' ];
-
-	// TODO: conflict/duplication when using `asChild` and <DropdownMenuItem /> ?
 	/**
 	 * When `true`, prevents the user from interacting with the item.
 	 */
@@ -110,18 +106,17 @@ export type DropdownSubMenuProps = {
 	 */
 	textValue?: DropdownMenuPrimitive.DropdownMenuSubTriggerProps[ 'textValue' ];
 	/**
-	 * The contents rendered inside the trigger
+	 * The contents rendered inside the trigger. The trigger should be
+	 * an instance of the `DropdownSubMenuTriggerProps` component.
 	 */
 	trigger: React.ReactNode;
 	/**
 	 * The contents of the dropdown sub menu
 	 */
 	children: React.ReactNode;
-
-	// TODO: slot name ?
 };
 
-export type DropdownItemProps = {
+export type DropdownMenuItemProps = {
 	/**
 	 * When true, prevents the user from interacting with the item.
 	 *