♻️(frontend) search on all docs if no children

When searching for documents, if no children are
found, the search will now include all documents
instead of just those with children.

Author: AntoLC

src/frontend/apps/impress/src/features/docs/doc-management/hooks/index.ts

@@ -1,4 +1,5 @@
 export * from './useCollaboration';
 export * from './useCopyDocLink';
+export * from './useDocUtils';
 export * from './useIsCollaborativeEditable';
 export * from './useTrans';

src/frontend/apps/impress/src/features/docs/doc-management/hooks/useDocUtils.tsx

@@ -0,0 +1,15 @@
+import { Doc } from '@/docs/doc-management';
+
+export const useDocUtils = (doc: Doc) => {
+  return {
+    isParent: doc.depth === 1, // it is a parent
+    isChild: doc.depth > 1, // it is a child
+    hasChildren: doc.numchild > 0, // it has children
+    isDesyncronized: !!(
+      doc.ancestors_link_reach &&
+      doc.ancestors_link_role &&
+      (doc.computed_link_reach !== doc.ancestors_link_reach ||
+        doc.computed_link_role !== doc.ancestors_link_role)
+    ),
+  } as const;
+};

src/frontend/apps/impress/src/features/docs/doc-share/components/DocVisibility.tsx

@@ -22,9 +22,9 @@ import {
   LinkReach,
   LinkRole,
   getDocLinkReach,
+  useDocUtils,
   useUpdateDocLink,
 } from '@/docs/doc-management';
-import { useTreeUtils } from '@/docs/doc-tree';
 import { useResponsiveStore } from '@/stores';
 
 import { useTranslatedShareSettings } from '../hooks/';
@@ -47,7 +47,7 @@ export const DocVisibility = ({ doc, canEdit = true }: DocVisibilityProps) => {
   const [docLinkRole, setDocLinkRole] = useState<LinkRole>(
     doc.computed_link_role ?? LinkRole.READER,
   );
-  const { isDesyncronized } = useTreeUtils(doc);
+  const { isDesyncronized } = useDocUtils(doc);
 
   const { linkModeTranslations, linkReachChoices, linkReachTranslations } =
     useTranslatedShareSettings();

src/frontend/apps/impress/src/features/docs/doc-tree/components/DocTreeItemActions.tsx

@@ -10,17 +10,17 @@ import { useTranslation } from 'react-i18next';
 import { css } from 'styled-components';
 
 import { Box, BoxButton, Icon } from '@/components';
-
 import {
   Doc,
   ModalRemoveDoc,
   Role,
   useCopyDocLink,
-} from '../../doc-management';
+  useDocUtils,
+} from '@/docs/doc-management';
+
 import { useCreateChildrenDoc } from '../api/useCreateChildren';
 import { useDetachDoc } from '../api/useDetach';
 import MoveDocIcon from '../assets/doc-extract-bold.svg';
-import { useTreeUtils } from '../hooks';
 
 type DocTreeItemActionsProps = {
   doc: Doc;
@@ -42,7 +42,7 @@ export const DocTreeItemActions = ({
   const deleteModal = useModal();
 
   const copyLink = useCopyDocLink(doc.id);
-  const { isCurrentParent } = useTreeUtils(doc);
+  const { isParent } = useDocUtils(doc);
   const { mutate: detachDoc } = useDetachDoc();
   const treeContext = useTreeContext<Doc>();
 
@@ -71,7 +71,7 @@ export const DocTreeItemActions = ({
       icon: <Icon iconName="link" $size="24px" />,
       callback: copyLink,
     },
-    ...(!isCurrentParent
+    ...(!isParent
       ? [
           {
             label: t('Move to my docs'),

src/frontend/apps/impress/src/features/docs/doc-tree/hooks/index.ts

@@ -1,3 +1,2 @@
 export * from './api';
-export * from './hooks';
 export * from './utils';

src/frontend/apps/impress/src/features/docs/doc-tree/hooks/useTreeUtils.tsx

@@ -3,6 +3,7 @@ import { useRouter } from 'next/router';
 import { PropsWithChildren, useCallback, useState } from 'react';
 
 import { Box, Icon, SeparatedSection } from '@/components';
+import { Doc, useDocStore, useDocUtils } from '@/docs/doc-management';
 import { DocSearchModal, DocSearchTarget } from '@/docs/doc-search/';
 import { useAuth } from '@/features/auth';
 import { useCmdK } from '@/hook/useCmdK';
@@ -12,10 +13,10 @@ import { useLeftPanelStore } from '../stores';
 import { LeftPanelHeaderButton } from './LeftPanelHeaderButton';
 
 export const LeftPanelHeader = ({ children }: PropsWithChildren) => {
+  const { currentDoc } = useDocStore();
+  const isDoc = !!currentDoc;
   const router = useRouter();
   const { authenticated } = useAuth();
-  const isDoc = router.pathname === '/docs/[id]';
-
   const [isSearchModalOpen, setIsSearchModalOpen] = useState(false);
 
   const openSearchModal = useCallback(() => {
@@ -77,16 +78,45 @@ export const LeftPanelHeader = ({ children }: PropsWithChildren) => {
         </SeparatedSection>
         {children}
       </Box>
-      {isSearchModalOpen && (
-        <DocSearchModal
+      {isSearchModalOpen && isDoc && (
+        <LeftPanelSearchModalFilter
           onClose={closeSearchModal}
           isOpen={isSearchModalOpen}
-          showFilters={isDoc}
-          defaultFilters={{
-            target: isDoc ? DocSearchTarget.CURRENT : undefined,
-          }}
+          doc={currentDoc}
         />
       )}
+      {isSearchModalOpen && !isDoc && (
+        <DocSearchModal onClose={closeSearchModal} isOpen={isSearchModalOpen} />
+      )}
     </>
   );
 };
+
+interface LeftPanelSearchModalProps {
+  doc: Doc;
+  isOpen: boolean;
+  onClose: () => void;
+}
+
+const LeftPanelSearchModalFilter = ({
+  doc,
+  isOpen,
+  onClose,
+}: LeftPanelSearchModalProps) => {
+  const { hasChildren, isChild } = useDocUtils(doc);
+  const isWithChildren = isChild || hasChildren;
+
+  let defaultFilters = DocSearchTarget.ALL;
+  if (isWithChildren) {
+    defaultFilters = DocSearchTarget.CURRENT;
+  }
+
+  return (
+    <DocSearchModal
+      onClose={onClose}
+      isOpen={isOpen}
+      showFilters={true}
+      defaultFilters={{ target: defaultFilters }}
+    />
+  );
+};