[FX-4717] Delete icon property (#4530)

.changeset/quiet-coins-switch.md

@@ -0,0 +1,8 @@
+---
+'@toptal/picasso-list': major
+'@toptal/picasso': major
+---
+
+### ListItem
+
+- breaking change: `icon` property no longer exists. If your project uses it, please contact the BASE Team to confirm that it should be added to the component.

packages/base/List/src/ListItem/ListItem.tsx

@@ -3,7 +3,6 @@ import type { ReactNode } from 'react'
 import type { BaseProps } from '@toptal/picasso-shared'
 import { Container } from '@toptal/picasso-container'
 import { Typography } from '@toptal/picasso-typography'
-import { usePropDeprecationWarning } from '@toptal/picasso-utils'
 import { ArrowLongRight16, Check16 } from '@toptal/picasso-icons'
 import { twJoin } from '@toptal/picasso-tailwind-merge'
 
@@ -27,30 +26,16 @@ export type Props = BaseProps & {
   children: ReactNode
   index?: number
   variant?: Variant
-  /**
-   * @deprecated [FX-4717] if you need a custom icon that is not available on the prop `type`, please contact the BASE team to add it to the theme
-   **/
-  icon?: ReactNode
   /** Style of the bullet/ordinal */
   type?: ListItemType
   isLastElement?: boolean
 }
 
 export const ListItem = (props: Props) => {
   const { styleType: parentType } = useListContext()
-  const {
-    children,
-    type,
-    icon = resolveIcon(type ?? parentType),
-    'data-testid': testId,
-  } = props
+  const { children, type, 'data-testid': testId } = props
 
-  // TODO: [FX-4717]
-  usePropDeprecationWarning({
-    props,
-    componentName: ListItem.name,
-    name: 'icon',
-  })
+  const icon = resolveIcon(type ?? parentType)
 
   return (
     <li