feat: Remove isListItem prop to ActionsMenuItems

cozy · Nov 28, 2024 · 687b4f4 · 687b4f4
1 parent b65fee1
commit 687b4f4
Show file tree

Hide file tree

Showing 7 changed files with 6 additions and 11 deletions.
diff --git a/src/modules/actions/index.jsx b/src/modules/actions/index.jsx
@@ -246,7 +246,7 @@ export const openExternalLink = ({ t, isSharingShortcutCreated, link }) => {
     },
     Component: forwardRef(function OpenExternalLink(props, ref) {
       return (
-        <ActionsMenuItem isListItem {...props} ref={ref}>
+        <ActionsMenuItem {...props} ref={ref}>
           <ListItemIcon>
             <Icon icon={icon} />
           </ListItemIcon>

diff --git a/src/modules/drive/Toolbar/components/AddMenuItem.jsx b/src/modules/drive/Toolbar/components/AddMenuItem.jsx
@@ -29,7 +29,6 @@ const AddMenuItem = ({ onClick }) => {
   return (
     <ActionsMenuItem
       ref={anchorRef}
-      isListItem
       disabled={isDisabled || isOffline}
       onClick={handleClick}
       {...a11y}

diff --git a/src/modules/drive/Toolbar/components/DownloadButtonItem.jsx b/src/modules/drive/Toolbar/components/DownloadButtonItem.jsx
@@ -21,7 +21,7 @@ const DownloadButtonItem = ({ files }) => {
   }
 
   return (
-    <ActionsMenuItem isListItem onClick={handleClick}>
+    <ActionsMenuItem onClick={handleClick}>
       <ListItemIcon>
         <Icon icon={DownloadIcon} />
       </ListItemIcon>

diff --git a/src/modules/drive/Toolbar/delete/DeleteItem.jsx b/src/modules/drive/Toolbar/delete/DeleteItem.jsx
@@ -19,11 +19,7 @@ const DeleteItem = ({ t, isSharedWithMe, trashFolder, displayedFolder }) => {
   const label = isSharedWithMe ? t('toolbar.leave') : t('toolbar.trash')
 
   return (
-    <ActionsMenuItem
-      data-testid="fil-action-delete"
-      isListItem
-      onClick={handleClick}
-    >
+    <ActionsMenuItem data-testid="fil-action-delete" onClick={handleClick}>
       <ListItemIcon>
         <Icon icon={TrashIcon} color="var(--errorColor)" />
       </ListItemIcon>

diff --git a/src/modules/drive/Toolbar/selectable/SelectableItem.jsx b/src/modules/drive/Toolbar/selectable/SelectableItem.jsx
@@ -15,7 +15,7 @@ const SelectableItem = ({ onClick }) => {
   const { t } = useI18n()
 
   return (
-    <ActionsMenuItem isListItem onClick={onClick}>
+    <ActionsMenuItem onClick={onClick}>
       <ListItemIcon>
         <Icon icon={CheckSquareIcon} />
       </ListItemIcon>

diff --git a/src/modules/drive/Toolbar/share/ShareItem.jsx b/src/modules/drive/Toolbar/share/ShareItem.jsx
@@ -24,7 +24,7 @@ const ShareItem = ({ displayedFolder }) => {
   return (
     <SharedDocument docId={displayedFolder.id}>
       {({ isSharedWithMe, recipients, link }) => (
-        <ActionsMenuItem onClick={share} isListItem>
+        <ActionsMenuItem onClick={share}>
           <ListItemIcon>
             <Icon icon={ShareIcon} />
           </ListItemIcon>

diff --git a/src/modules/public/OpenExternalLinkItem.jsx b/src/modules/public/OpenExternalLinkItem.jsx
@@ -24,7 +24,7 @@ const OpenExternalLinkItem = ({ link, isSharingShortcutCreated }) => {
   })
 
   return (
-    <ActionsMenuItem isListItem onClick={handleClick}>
+    <ActionsMenuItem onClick={handleClick}>
       <ListItemIcon>
         <Icon icon={icon} />
       </ListItemIcon>