[ACS-8371] fix viewer preview

Alfresco · Aug 8, 2024 · f236312 · f236312
1 parent 2bc4e15
commit f236312
Show file tree

Hide file tree

Showing 5 changed files with 4 additions and 30 deletions.
diff --git a/projects/aca-content/assets/app.extensions.json b/projects/aca-content/assets/app.extensions.json
@@ -1217,16 +1217,6 @@
             "visible": "canInfoPreview"
           }
         },
-        {
-          "id": "app.sidebar.infoDetails",
-          "order": 150,
-          "title": "Info Details",
-          "icon": "info_outline",
-          "color": "accent",
-          "rules": {
-            "visible": "showInfoSelectionButton"
-          }
-        },
         {
           "id": "app.toolbar.aspects",
           "order": 160,

diff --git a/projects/aca-content/src/lib/aca-content.module.ts b/projects/aca-content/src/lib/aca-content.module.ts
@@ -173,7 +173,6 @@ export class ContentServiceExtensionModule {
       canEditAspects: rules.canEditAspects,
       canShowExpand: rules.canShowExpand,
       canInfoPreview: rules.canInfoPreview,
-      showInfoSelectionButton: rules.showInfoSelectionButton,
 
       'app.selection.canDelete': rules.canDeleteSelection,
       'app.selection.file.canUnlock': rules.canUnlockFile,

diff --git a/projects/aca-content/src/lib/components/search/search-results/search-results.component.scss b/projects/aca-content/src/lib/components/search/search-results/search-results.component.scss
@@ -92,6 +92,10 @@ aca-search-results {
       display: flex;
       justify-content: flex-start;
       flex-basis: 55%;
+
+      .adf-viewer {
+        position: relative;
+      }
     }
 
     &--preview-toolbar {

diff --git a/projects/aca-shared/rules/src/app.rules.spec.ts b/projects/aca-shared/rules/src/app.rules.spec.ts
@@ -1418,23 +1418,6 @@ describe('app.evaluators', () => {
     });
   });
 
-  describe('showInfoSelectionButton', () => {
-    it('should return false when user is not in search results page', () => {
-      context.navigation.url = '/trashcan/test';
-      expect(app.showInfoSelectionButton(context)).toBeFalse();
-    });
-
-    it('should return false when user is in preview', () => {
-      context.navigation.url = '/search/preview/test';
-      expect(app.showInfoSelectionButton(context)).toBeFalse();
-    });
-
-    it('should return true when user is in search results page and not in preview', () => {
-      context.navigation.url = '/search/test';
-      expect(app.showInfoSelectionButton(context)).toBeTrue();
-    });
-  });
-
   describe('isSmartFolder', () => {
     it('should return false when there is no selection', () => {
       context.selection.isEmpty = true;

diff --git a/projects/aca-shared/rules/src/app.rules.ts b/projects/aca-shared/rules/src/app.rules.ts
@@ -552,8 +552,6 @@ export const canInfoPreview = (context: RuleContext): boolean => {
   return isSearchResult && isNotMultiselect && isFileSelected && isPreview;
 };
 
-export const showInfoSelectionButton = (context: RuleContext): boolean => navigation.isSearchResults(context) && !navigation.isPreview(context);
-
 /**
  * Checks if the file can be opened with MS Office
  * JSON ref: `aos.canOpenWithOffice`