feat: impove attached files screen

changelogs/unreleased/88612_SearchTreeView.json

@@ -0,0 +1,5 @@
+{
+  "title": "SearchTreeView: add possibility to remove management of parent filter",
+  "type": "feat",
+  "packages": "core"
+}

changelogs/unreleased/88612_TreeView.json

@@ -0,0 +1,5 @@
+{
+  "title": "TreeView: add possibility to hide branch filter quick action",
+  "type": "feat",
+  "packages": "ui"
+}

packages/apps/dms/src/components/templates/DocumentList/DocumentList.tsx

@@ -38,9 +38,13 @@ import {File} from '../../../types';
 
 interface DocumentListProps {
   defaultParent?: any;
+  isAttachedFilesList?: boolean;
 }
 
-const DocumentList = ({defaultParent}: DocumentListProps) => {
+const DocumentList = ({
+  defaultParent,
+  isAttachedFilesList = false,
+}: DocumentListProps) => {
   const I18n = useTranslator();
   const Colors = useThemeColor();
   const navigation = useNavigation();
@@ -152,6 +156,7 @@ const DocumentList = ({defaultParent}: DocumentListProps) => {
         }
         displayBreadcrumb
         defaultParent={defaultParent}
+        manageParentFilter={!isAttachedFilesList}
       />
     </Screen>
   );

packages/apps/dms/src/screens/AttachedFilesScreen.tsx

@@ -65,7 +65,9 @@ const AttachedFilesScreen = ({navigation, route}) => {
 
   return (
     <>
-      {!isVisible && <DocumentList defaultParent={parent} />}
+      {!isVisible && (
+        <DocumentList defaultParent={parent} isAttachedFilesList />
+      )}
       <Alert
         visible={isVisible}
         title={I18n.t('Dms_NoAttachedFiles')}

packages/core/src/components/templates/SearchTreeView/SearchTreeView.tsx

@@ -70,6 +70,7 @@ interface SearchTreeViewProps {
   verticalActions?: boolean;
   displayBreadcrumb?: boolean;
   defaultParent?: any;
+  manageParentFilter?: boolean;
 }
 
 const SearchTreeView = ({
@@ -109,6 +110,7 @@ const SearchTreeView = ({
   verticalActions,
   displayBreadcrumb = false,
   defaultParent,
+  manageParentFilter = true,
 }: SearchTreeViewProps) => {
   const I18n = useTranslator();
   const dispatch = useDispatch();
@@ -200,6 +202,10 @@ const SearchTreeView = ({
   }, [defaultParent, fetchListAPI, isFocused]);
 
   const renderParentSearchBar = useCallback(() => {
+    if (!manageParentFilter) {
+      return null;
+    }
+
     return (
       <AutoCompleteSearch
         objectList={parentList}
@@ -214,6 +220,7 @@ const SearchTreeView = ({
   }, [
     displayParentSearchValue,
     fetchParentSearchAPI,
+    manageParentFilter,
     parent,
     parentList,
     searchParentPlaceholder,
@@ -271,6 +278,7 @@ const SearchTreeView = ({
       {displayBreadcrumb && (
         <Breadcrumb
           style={styles.breadcrumb}
+          disabled={!manageParentFilter}
           items={parent.map((item, index) =>
             item === EMPTY_PARENT
               ? EMPTY_PARENT
@@ -295,6 +303,7 @@ const SearchTreeView = ({
         fetchBranchData={fetchBranchData}
         branchCondition={branchCondition}
         onBranchFilterPress={handleChangeParent}
+        isBranchFilterVisible={manageParentFilter}
         moreLoading={moreLoading}
         isListEnd={isListEnd}
         translator={I18n.t}

packages/ui/__tests__/components/molecules/Breadcrumb.test.js

@@ -89,6 +89,16 @@ describe('Breadcrumb Component', () => {
     expect(nonClickableItem.prop('onPress')).toBeUndefined();
   });
 
+  it('renders correctly when disabled', () => {
+    const wrapper = shallow(<Breadcrumb {...props} disabled />);
+    const items = wrapper.find(TouchableOpacity);
+
+    items.forEach(_item => {
+      expect(_item.exists()).toBeTruthy();
+      expect(_item.prop('disabled')).toBeTruthy();
+    });
+  });
+
   it('should apply custom styles to the container when style prop is provided', () => {
     const customStyle = {
       fontSize: 25,

packages/ui/src/components/molecules/Breadcrumb/Breadcrumb.tsx

@@ -32,11 +32,17 @@ interface Item {
 
 interface BreadcrumbProps {
   style?: any;
+  disabled?: boolean;
   items: Item[];
   onHomePress: () => void;
 }
 
-const Breadcrumb = ({style, items, onHomePress}: BreadcrumbProps) => {
+const Breadcrumb = ({
+  style,
+  disabled = false,
+  items,
+  onHomePress,
+}: BreadcrumbProps) => {
   const Colors = useThemeColor();
 
   const [visibleItems, setVisibleItems] = useState(items);
@@ -86,7 +92,11 @@ const Breadcrumb = ({style, items, onHomePress}: BreadcrumbProps) => {
     <View
       style={[styles.container, style]}
       onLayout={e => setContainerWidth(e?.nativeEvent?.layout?.width)}>
-      <Icon name="house-door-fill" touchable onPress={onHomePress} />
+      <Icon
+        name="house-door-fill"
+        touchable={!disabled}
+        onPress={onHomePress}
+      />
       {visibleItems.map((item, index) => (
         <View
           style={styles.contentContainer}
@@ -96,7 +106,7 @@ const Breadcrumb = ({style, items, onHomePress}: BreadcrumbProps) => {
           <TouchableOpacity
             style={styles.textContainer}
             activeOpacity={0.7}
-            disabled={!item.onPress}
+            disabled={disabled || !item.onPress}
             onPress={item.onPress}>
             <Text>{item.title}</Text>
           </TouchableOpacity>

packages/ui/src/components/templates/TreeView/Branch.tsx

@@ -132,6 +132,7 @@ interface BranchProps {
   fetchBranchData: (idParent: number) => Promise<any>;
   branchCondition: (item: any) => boolean;
   onBranchFilterPress: (branch: any) => void;
+  isBranchFilterVisible?: boolean;
   translator: (translationKey: string) => string;
 }
 
@@ -148,6 +149,7 @@ const Branch = ({
   fetchBranchData,
   branchCondition,
   onBranchFilterPress,
+  isBranchFilterVisible,
   translator,
 }: BranchProps) => {
   const handleBranchPress = useCallback(() => {
@@ -194,6 +196,7 @@ const Branch = ({
         isOpen={isBranchOpen}
         parent={branch.item}
         onFilterPress={onBranchFilterPress}
+        isBranchFilterVisible={isBranchFilterVisible}
         infoButtonIndication={branchCardInfoButtonIndication}
         actionList={getBranchActions?.(branch)}
         translator={translator}

packages/ui/src/components/templates/TreeView/BranchCard.tsx

@@ -64,6 +64,7 @@ interface BranchActionCardProps {
   isOpen: boolean;
   parent: any;
   onFilterPress: (branch: any) => void;
+  isBranchFilterVisible?: boolean;
   infoButtonIndication: string;
   actionList?: ActionCardType[];
   translator: (translationKey: string) => string;
@@ -75,6 +76,7 @@ const BranchActionCard = ({
   isOpen,
   parent,
   onFilterPress,
+  isBranchFilterVisible,
   infoButtonIndication,
   actionList = [],
   translator,
@@ -87,6 +89,7 @@ const BranchActionCard = ({
         helper: infoButtonIndication,
         large: true,
         onPress: () => onFilterPress(parent),
+        hidden: !isBranchFilterVisible,
       }}
       translator={translator}>
       <BranchCard onPress={onPress} children={children} isOpen={isOpen} />

packages/ui/src/components/templates/TreeView/TreeView.tsx

@@ -52,6 +52,7 @@ interface TreeViewProps {
    */
   branchCondition: (item: any) => boolean;
   onBranchFilterPress: (branch: any) => void;
+  isBranchFilterVisible?: boolean;
   moreLoading: boolean;
   isListEnd: boolean;
   filter?: boolean;
@@ -75,6 +76,7 @@ const TreeView = ({
   fetchBranchData,
   branchCondition,
   onBranchFilterPress,
+  isBranchFilterVisible = true,
   moreLoading = false,
   isListEnd = false,
   filter = false,
@@ -104,6 +106,7 @@ const TreeView = ({
         fetchBranchData={fetchBranchData}
         branchCondition={branchCondition}
         onBranchFilterPress={onBranchFilterPress}
+        isBranchFilterVisible={isBranchFilterVisible}
         translator={translator}
       />
     ) : (

packages/ui/stories/molecules/Breadcrumb.stories.tsx

@@ -31,6 +31,7 @@ export default meta;
 export const Breadcrumb: Story<typeof Component> = {
   args: {
     numberItems: 5,
+    disabled: false,
   },
   argTypes: {
     items: disabledControl,