feat: Do not display MoreMenu if has no actions

cozy · Nov 29, 2024 · a4882a1 · a4882a1
1 parent 44ce6ef
commit a4882a1
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 2 deletions.
diff --git a/src/modules/public/PublicToolbarByLink.jsx b/src/modules/public/PublicToolbarByLink.jsx
@@ -41,7 +41,7 @@ const PublicToolbarByLink = ({
   const actions = makeActions(
     [
       isMobile && download,
-      select,
+      files.length > 1 && select,
       addItems,
       isMobile && (files.length > 1 || hasWriteAccess) && hr,
       isMobile && openExternalLink

diff --git a/src/modules/public/PublicToolbarCozyToCozy.jsx b/src/modules/public/PublicToolbarCozyToCozy.jsx
@@ -29,7 +29,7 @@ const PublicToolbarCozyToCozy = ({
   const actions = makeActions(
     [
       isMobile && download,
-      select,
+      files.length > 1 && select,
       ((isMobile && files.length > 0) || files.length > 1) && hr,
       openExternalLink
     ],

diff --git a/src/modules/public/PublicToolbarMoreMenu.jsx b/src/modules/public/PublicToolbarMoreMenu.jsx
@@ -19,6 +19,8 @@ const PublicToolbarMoreMenu = ({ files, actions }) => {
     openMenu()
   }, [closeMenu, openMenu, menuIsVisible])
 
+  if (actions.length === 0) return null
+
   return (
     <>
       <div