From 060b54baacbcc527e2783de80d23043696a43179 Mon Sep 17 00:00:00 2001
From: milanmajchrak <milan.majchrak@dataquest.sk>
Date: Wed, 7 May 2025 13:39:28 +0200
Subject: [PATCH 1/4] Do not use the clarin item view box for the bulk access

---
 .../object-list/object-list.component.html      |  4 ++--
 .../shared/object-list/object-list.component.ts | 17 ++++++++++++++++-
 2 files changed, 18 insertions(+), 3 deletions(-)

diff --git a/src/app/shared/object-list/object-list.component.html b/src/app/shared/object-list/object-list.component.html
index 33caa182f18..0bbf45d36d0 100644
--- a/src/app/shared/object-list/object-list.component.html
+++ b/src/app/shared/object-list/object-list.component.html
@@ -16,14 +16,14 @@
         (prev)="goPrev()"
         (next)="goNext()">
     <ul *ngIf="objects?.hasSucceeded" class="list-unstyled" [ngClass]="{'ml-4': selectable}">
-      <div *ngIf="context === 'search'">
+      <div *ngIf="showClarinViewBox()">
         <ds-clarin-item-box-view
           *ngFor="let object of objects?.payload?.page; let i = index; let last = last"
           [object]="object"
           [isSearchResult]="true">
         </ds-clarin-item-box-view>
       </div>
-      <div *ngIf="context !== 'search'">
+      <div *ngIf="!showClarinViewBox()">
         <li *ngFor="let object of objects?.payload?.page; let i = index; let last = last" class="mt-4 mb-4 d-flex" [class.border-bottom]="hasBorder && !last"  [attr.data-test]="'list-object' | dsBrowserOnly">
           <ds-selectable-list-item-control *ngIf="selectable" [index]="i"
                                            [object]="object"
diff --git a/src/app/shared/object-list/object-list.component.ts b/src/app/shared/object-list/object-list.component.ts
index 5161b754598..7e0a18b5bfe 100644
--- a/src/app/shared/object-list/object-list.component.ts
+++ b/src/app/shared/object-list/object-list.component.ts
@@ -1,4 +1,12 @@
-import { ChangeDetectionStrategy, Component, EventEmitter, Input, Output, ViewEncapsulation } from '@angular/core';
+import {
+  ChangeDetectionStrategy,
+  Component,
+  EventEmitter,
+  Input,
+  OnInit,
+  Output,
+  ViewEncapsulation
+} from '@angular/core';
 import { SortDirection, SortOptions } from '../../core/cache/models/sort-options.model';
 import { PaginatedList } from '../../core/data/paginated-list.model';
 import { RemoteData } from '../../core/data/remote-data';
@@ -227,4 +235,11 @@ export class ObjectListComponent {
       this.next.emit(true);
   }
 
+  /**
+   * Show the search results in the Clarin view box or use the default view
+   */
+  showClarinViewBox() {
+    return this.context === 'search' && this.selectionConfig?.listId !== 'bulk-access-list';
+  }
+
 }

From aa85b4e2f91b8338d58433c93d283a4cc0df5a8a Mon Sep 17 00:00:00 2001
From: milanmajchrak <milan.majchrak@dataquest.sk>
Date: Wed, 7 May 2025 13:44:28 +0200
Subject: [PATCH 2/4] Removed unused import

---
 src/app/shared/object-list/object-list.component.ts | 10 +---------
 1 file changed, 1 insertion(+), 9 deletions(-)

diff --git a/src/app/shared/object-list/object-list.component.ts b/src/app/shared/object-list/object-list.component.ts
index 7e0a18b5bfe..b70d43f459d 100644
--- a/src/app/shared/object-list/object-list.component.ts
+++ b/src/app/shared/object-list/object-list.component.ts
@@ -1,12 +1,4 @@
-import {
-  ChangeDetectionStrategy,
-  Component,
-  EventEmitter,
-  Input,
-  OnInit,
-  Output,
-  ViewEncapsulation
-} from '@angular/core';
+import { ChangeDetectionStrategy, Component, EventEmitter, Input, Output, ViewEncapsulation } from '@angular/core';
 import { SortDirection, SortOptions } from '../../core/cache/models/sort-options.model';
 import { PaginatedList } from '../../core/data/paginated-list.model';
 import { RemoteData } from '../../core/data/remote-data';

From 9c3b843cbc4aeef99b939dac660ecedd464f8c5f Mon Sep 17 00:00:00 2001
From: milanmajchrak <milan.majchrak@dataquest.sk>
Date: Tue, 13 May 2025 08:26:49 +0200
Subject: [PATCH 3/4] Clarify the `showClarinViewBox` is boolean

---
 src/app/shared/object-list/object-list.component.ts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/app/shared/object-list/object-list.component.ts b/src/app/shared/object-list/object-list.component.ts
index b70d43f459d..6b11bbf3646 100644
--- a/src/app/shared/object-list/object-list.component.ts
+++ b/src/app/shared/object-list/object-list.component.ts
@@ -230,7 +230,7 @@ export class ObjectListComponent {
   /**
    * Show the search results in the Clarin view box or use the default view
    */
-  showClarinViewBox() {
+  showClarinViewBox(): boolean {
     return this.context === 'search' && this.selectionConfig?.listId !== 'bulk-access-list';
   }
 

From 0f668fa27aa5d2e454f7dc1f76065d0c9479f4e5 Mon Sep 17 00:00:00 2001
From: milanmajchrak <milan.majchrak@dataquest.sk>
Date: Wed, 14 May 2025 12:25:14 +0200
Subject: [PATCH 4/4] Use the constant for the hardcoded bulk access list id

---
 src/app/shared/object-list/object-list.component.ts | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/src/app/shared/object-list/object-list.component.ts b/src/app/shared/object-list/object-list.component.ts
index 6b11bbf3646..f66e75dc5a6 100644
--- a/src/app/shared/object-list/object-list.component.ts
+++ b/src/app/shared/object-list/object-list.component.ts
@@ -19,6 +19,11 @@ import { CollectionElementLinkType } from '../object-collection/collection-eleme
   animations: [fadeIn]
 })
 export class ObjectListComponent {
+  /**
+   * The id of the list used for bulk access
+   */
+  BULK_ACCESS_LIST_ID = 'bulk-access-list';
+
   /**
    * The view mode of the this component
    */
@@ -231,7 +236,6 @@ export class ObjectListComponent {
    * Show the search results in the Clarin view box or use the default view
    */
   showClarinViewBox(): boolean {
-    return this.context === 'search' && this.selectionConfig?.listId !== 'bulk-access-list';
+    return this.context === 'search' && this.selectionConfig?.listId !== this.BULK_ACCESS_LIST_ID;
   }
-
 }