From 03bde9ab39d53c02153c435fbeec86c9f23614b3 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Johnny=20Marie=CC=81thoz?= <Johnny.Mariethoz@rero.ch>
Date: Wed, 8 Jan 2025 08:26:09 +0100
Subject: [PATCH] primeng: admin inventories
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Co-Authored-by: Johnny Mariéthoz <Johnny.Mariethoz@rero.ch>
Co-Authored-by: Bertrand Zuchuat <bertrand.zuchuat@rero.ch>
---
 .../issues-brief-view.component.ts            |  2 +-
 .../items-brief-view.component.html           | 18 +++++++-------
 projects/admin/src/app/routes/items-route.ts  | 24 ++++++++++++-------
 3 files changed, 26 insertions(+), 18 deletions(-)

diff --git a/projects/admin/src/app/record/brief-view/issues-brief-view/issues-brief-view.component.ts b/projects/admin/src/app/record/brief-view/issues-brief-view/issues-brief-view.component.ts
index 2e9b1246f..8cee7caaf 100644
--- a/projects/admin/src/app/record/brief-view/issues-brief-view/issues-brief-view.component.ts
+++ b/projects/admin/src/app/record/brief-view/issues-brief-view/issues-brief-view.component.ts
@@ -20,7 +20,7 @@ import { ResultItem } from '@rero/ng-core';
 import { IssueItemStatus } from '@rero/shared';
 
 @Component({
-  selector: 'admin-inventory-brief-view',
+  selector: 'admin-issues-brief-view',
   templateUrl: './issues-brief-view.component.html',
 })
 export class IssuesBriefViewComponent implements ResultItem, OnInit {
diff --git a/projects/admin/src/app/record/brief-view/items-brief-view/items-brief-view.component.html b/projects/admin/src/app/record/brief-view/items-brief-view/items-brief-view.component.html
index f5560769a..8baaa08fa 100644
--- a/projects/admin/src/app/record/brief-view/items-brief-view/items-brief-view.component.html
+++ b/projects/admin/src/app/record/brief-view/items-brief-view/items-brief-view.component.html
@@ -16,18 +16,18 @@
   along with this program.  If not, see <http://www.gnu.org/licenses/>.
 -->
 
-<h5 class="mb-0">
+<h5>
   <a [routerLink]="['/records', 'items', 'detail', record.metadata.pid]">
       {{ record.metadata.ui_title_text }}
   </a>
 </h5>
-<dl class="row mb-0 ml-1">
+<dl class="metadata">
   <!-- INHERITED CALL NUMBER -->
   @if (record | itemHoldingsCallNumber | async; as callNumber) {
     @if ((callNumber.first | json) !== '{}' || (callNumber.second | json) !== '{}') {
-      <dt class="col-sm-3 offset-sm-2 offset-md-0 label-title" translate>Call number</dt>
-      <dd class="col-sm-7 col-md-8 mb-0">
-        <shared-inherited-call-number [item]="record" context="first"></shared-inherited-call-number>
+      <dt translate>Call number</dt>
+      <dd>
+        <shared-inherited-call-number [item]="record" context="first" />
         @if (record.metadata.second_call_number) {
           | {{ record.metadata.second_call_number }}
         }
@@ -35,11 +35,11 @@ <h5 class="mb-0">
     }
   }
   @if (record.metadata.barcode) {
-    <dt class="col-sm-3 offset-sm-2 offset-md-0 label-title" translate>Barcode</dt>
-    <dd class="col-sm-7 col-md-8 mb-0">{{ record.metadata.barcode }}</dd>
+    <dt translate>Barcode</dt>
+    <dd>{{ record.metadata.barcode }}</dd>
   }
   @if (record.metadata.current_pending_requests > 0) {
-    <dt class="col-sm-3 offset-sm-2 offset-md-0 label-title" translate>Current requests</dt>
-    <dd  class="col-sm-7 col-md-8 mb-0">{{ record.metadata.current_pending_requests }}</dd>
+    <dt translate>Current requests</dt>
+    <dd>{{ record.metadata.current_pending_requests }}</dd>
   }
 </dl>
diff --git a/projects/admin/src/app/routes/items-route.ts b/projects/admin/src/app/routes/items-route.ts
index 794c3843d..4544a232d 100644
--- a/projects/admin/src/app/routes/items-route.ts
+++ b/projects/admin/src/app/routes/items-route.ts
@@ -164,35 +164,43 @@ export class ItemsRoute extends BaseRoute implements RouteInterface {
               {
                 label: _('Relevance'),
                 value: 'bestmatch',
-                defaultQuery: true
+                defaultQuery: true,
+                icon: 'fa fa-sort-amount-desc'
               },
               {
                 label: _('Barcode'),
-                value: 'barcode'
+                value: 'barcode',
+                 icon: 'fa fa-sort-alpha-asc'
               },
               {
                 label: _('Barcode (desc)'),
-                value: '-barcode'
+                value: '-barcode',
+                icon: 'fa fa-sort-alpha-desc'
               },
               {
                 label: _('Call number'),
-                value: 'call_number'
+                value: 'call_number',
+                icon: 'fa fa-sort-alpha-asc'
               },
               {
                 label: _('Call number (desc)'),
-                value: '-call_number'
+                value: '-call_number',
+                icon: 'fa fa-sort-alpha-desc'
               },
               {
                 label: _('Second call number'),
-                value: 'second_call_number'
+                value: 'second_call_number',
+                icon: 'fa fa-sort-alpha-asc'
               },
               {
                 label: _('Second call number (desc)'),
-                value: '-second_call_number'
+                value: '-second_call_number',
+                icon: 'fa fa-sort-alpha-desc'
               },
               {
                 label: _('Current requests'),
-                value: 'current_requests'
+                value: 'current_requests',
+                icon: 'fa fa-sort-amount-desc'
               }
             ]
           }