From 3774e6a5a5b6698e4ab9f37d45cf8aeb31325c7f Mon Sep 17 00:00:00 2001
From: Bill Erickson <berickxx@gmail.com>
Date: Fri, 9 Aug 2019 16:55:30 -0400
Subject: [PATCH] LP1839670 Angular catalog more result page info

Adds physical description, isbn, issn, upc, and publisher information to
the record result list page.

Given the extra vertical space allowed by the new data, also increase
the size of the jacket images.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
---
 .../app/staff/catalog/result/record.component.css  |  8 ++--
 .../app/staff/catalog/result/record.component.html | 44 +++++++++++++++++++---
 2 files changed, 42 insertions(+), 10 deletions(-)

diff --git a/Open-ILS/src/eg2/src/app/staff/catalog/result/record.component.css b/Open-ILS/src/eg2/src/app/staff/catalog/result/record.component.css
index 3077d9ac93..3d753f4e42 100644
--- a/Open-ILS/src/eg2/src/app/staff/catalog/result/record.component.css
+++ b/Open-ILS/src/eg2/src/app/staff/catalog/result/record.component.css
@@ -5,11 +5,11 @@
  * render the correct aspect ratio.
  */
 .record-jacket-div {
-    width: 68px;
+    width: 120px;
 }
 
 .record-jacket-div img {
-    height: 100%; 
-    max-height:80px; 
-    max-width: 54px;
+    height: 100%;
+    max-height: 158px;
+    max-width: 100px;
 }
diff --git a/Open-ILS/src/eg2/src/app/staff/catalog/result/record.component.html b/Open-ILS/src/eg2/src/app/staff/catalog/result/record.component.html
index 5ddf94e000..a27c1bdf34 100644
--- a/Open-ILS/src/eg2/src/app/staff/catalog/result/record.component.html
+++ b/Open-ILS/src/eg2/src/app/staff/catalog/result/record.component.html
@@ -10,7 +10,7 @@
       <!-- Checkbox, jacket image, and title blob live in a flex row
            because there's no way to give them col-lg-* columns that
            don't waste a lot of space. -->
-      <div class="col-lg-6 d-flex">
+      <div class="col-lg-7 d-flex">
         <label class="checkbox">
           <span class="font-weight-bold font-italic">
             {{index + 1 + searchContext.pager.offset}}.
@@ -24,13 +24,13 @@
           <ng-container *ngIf="hasMrConstituentRecords(summary)">
             <a routerLink="/staff/catalog/search"
               [queryParams]="appendFromMrParam(summary)">
-              <img src="/opac/extras/ac/jacket/small/r/{{summary.id}}"/>
+              <img src="/opac/extras/ac/jacket/medium/r/{{summary.id}}"/>
             </a>
           </ng-container>
           <ng-container *ngIf="!hasMrConstituentRecords(summary)">
               <a routerLink="/staff/catalog/record/{{summary.id}}"
                 [queryParams]="currentParams()">
-                <img src="/opac/extras/ac/jacket/small/r/{{summary.id}}"/>
+                <img src="/opac/extras/ac/jacket/medium/r/{{summary.id}}"/>
               </a>
           </ng-container>
         </div>
@@ -81,8 +81,40 @@
                 </span>
                 </ng-container>
               </ng-container>
-              <span class='pl-1'>{{summary.display.edition}}</span>
-              <span class='pl-1'>{{summary.display.pubdate}}</span>
+            </div>
+          </div>
+          <div class="row pt-2">
+            <div class="col-lg-12">
+              <ng-container *ngIf="summary.display.physical_description">
+                <!-- [].concat() to avoid modifying the summary arrays -->
+                <div class="pb-1" i18n>Phys. Desc.: 
+                  {{[].concat(summary.display.physical_description).join(', ')}}
+                </div>
+              </ng-container>
+              <ng-container *ngIf="summary.display.edition">
+                <div class="pb-1" i18n>Edition: {{summary.display.edition}}</div>
+              </ng-container>
+              <ng-container *ngIf="summary.display.publisher || summary.display.pubdate">
+                <!-- note publisher typically includes pubdate -->
+                <ng-container *ngIf="summary.display.publisher; else pubDate">
+                  <div class="pb-1" i18n>Publisher: {{summary.display.publisher}}</div>
+                </ng-container>
+                <ng-template #pubDate>
+                  <div class="pb-1" i18n>Pub Date: {{summary.display.pubdate}}</div>
+                </ng-template>
+              </ng-container>
+              <ng-container *ngIf="summary.display.isbn">
+                <div class="pb-1" i18n>ISBN: 
+                  {{[].concat(summary.display.isbn).join(', ')}}</div>
+              </ng-container>
+              <ng-container *ngIf="summary.display.upc">
+                <div class="pb-1" i18n>UPC: 
+                  {{[].concat(summary.display.upc).join(', ')}}</div>
+              </ng-container>
+              <ng-container *ngIf="summary.display.issn">
+                <div i18n>ISSN: 
+                  {{[].concat(summary.display.issn).join(', ')}}</div>
+              </ng-container>
             </div>
           </div>
         </div>
@@ -114,7 +146,7 @@
           </div>
         </div>
       </div>
-      <div class="col-lg-3">
+      <div class="col-lg-2">
         <div class="row">
           <div class="col-lg-12">
             <div class="float-right small-text-1">
-- 
2.11.0