From: Bill Erickson <berickxx@gmail.com>
Date: Sat, 9 Dec 2017 03:18:02 +0000 (-0500)
Subject: LP#626157 Ang2 experiments
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=2bb4d9221000c25365ffc13b2c35eb438c1a5a07;p=working%2FEvergreen.git

LP#626157 Ang2 experiments

Signed-off-by: Bill Erickson <berickxx@gmail.com>
---

diff --git a/Open-ILS/webby-src/src/app/staff/catalog/record/copies.component.css b/Open-ILS/webby-src/src/app/staff/catalog/record/copies.component.css
index e69de29bb2..422af1a07a 100644
--- a/Open-ILS/webby-src/src/app/staff/catalog/record/copies.component.css
+++ b/Open-ILS/webby-src/src/app/staff/catalog/record/copies.component.css
@@ -0,0 +1,7 @@
+
+.d-flex > div:not(first-child) {
+  padding-left: 3px;
+  /*
+  border-left: 1px solid rgba(0,0,0,0.2);
+  */
+}
diff --git a/Open-ILS/webby-src/src/app/staff/catalog/record/copies.component.html b/Open-ILS/webby-src/src/app/staff/catalog/record/copies.component.html
index ea269a8654..de8f32242b 100644
--- a/Open-ILS/webby-src/src/app/staff/catalog/record/copies.component.html
+++ b/Open-ILS/webby-src/src/app/staff/catalog/record/copies.component.html
@@ -10,7 +10,7 @@
 <div class='eg-copies card w-100'>
   <div class="card-header font-weight-bold d-flex bg-info">
     <div class="flex-1" i18n>Location</div>
-    <div class="flex-2" i18n>Call Number / Copy Notes</div>
+    <div class="flex-1" i18n>Call Number / Copy Notes</div>
     <div class="flex-1" i18n>Barcode</div>
     <div class="flex-1" i18n>Shelving Location</div>
     <div class="flex-1" i18n>Circulation Modifier</div>
@@ -25,13 +25,18 @@
       <li class="list-group-item" *ngFor="let copy of copies">
         <div class="d-flex">
           <div class="flex-1" i18n>{{orgName(copy.circ_lib)}}</div>
-          <div class="flex-2" i18n>
+          <div class="flex-1" i18n>
             {{copy.call_number_prefix_label}}
             {{copy.call_number_label}}
             {{copy.call_number_suffix_label}}
           </div>
-          <div class="flex-1" i18n>{{copy.barcode}}</div>
-          <div class="flex-1" i18n>{{copy.location}}</div>
+          <div class="flex-1" i18n>
+            {{copy.barcode}}
+            <a class="pl-1" href="/eg/staff/cat/item/{{copy.id}}" i18n>View</a>
+            | 
+            <a class="pl-1" href="/eg/staff/cat/item/{{copy.id}}/edit" i18n>Edit</a>
+          </div>
+          <div class="flex-1" i18n>{{copy.copy_location}}</div>
           <div class="flex-1" i18n>{{copy.circ_modifier || ''}}</div>
           <div class="flex-1" i18n>{{copy.age_protect}}</div>
           <div class="flex-1" i18n>
diff --git a/Open-ILS/webby-src/src/app/staff/catalog/record/copies.component.ts b/Open-ILS/webby-src/src/app/staff/catalog/record/copies.component.ts
index 3e80783d72..0af98a5eff 100644
--- a/Open-ILS/webby-src/src/app/staff/catalog/record/copies.component.ts
+++ b/Open-ILS/webby-src/src/app/staff/catalog/record/copies.component.ts
@@ -26,7 +26,7 @@ export class CopiesComponent implements OnInit {
     ngOnInit() { 
         if (!this.recordId) return;
         this.pager = new Pager();
-        this.pager.limit = 50; // TODO
+        this.pager.limit = 10; // TODO
         this.fetchCopies();
     }
 
diff --git a/Open-ILS/webby-src/src/app/staff/catalog/search-form.component.ts b/Open-ILS/webby-src/src/app/staff/catalog/search-form.component.ts
index 0d70807ab2..aeb07badc3 100644
--- a/Open-ILS/webby-src/src/app/staff/catalog/search-form.component.ts
+++ b/Open-ILS/webby-src/src/app/staff/catalog/search-form.component.ts
@@ -28,6 +28,10 @@ export class SearchFormComponent implements OnInit {
         this.ccvmMap = this.cat.ccvmMap;
         this.cmfMap = this.cat.cmfMap;
         this.searchContext = this.staffCat.searchContext;
+
+        // Start with advanced search options open 
+        // if any filters are active.
+        this.showAdvancedSearch = this.hasAdvancedOptions();
     }
 
     /**
@@ -35,7 +39,7 @@ export class SearchFormComponent implements OnInit {
      * or if any advanced options are selected.
      */
     showAdvanced(): boolean {
-        return this.showAdvancedSearch || this.hasAdvancedOptions();
+        return this.showAdvancedSearch;
     }
 
     hasAdvancedOptions(): boolean {
@@ -44,7 +48,7 @@ export class SearchFormComponent implements OnInit {
         let show = false;
         Object.keys(this.searchContext.ccvmFilters).forEach(ccvm => {
             if (this.searchContext.ccvmFilters[ccvm][0] != '')
-                this.showAdvancedSearch = show = true;
+                show = true;
         });
 
         return show;
diff --git a/Open-ILS/webby-src/src/app/staff/share/bib-summary.component.html b/Open-ILS/webby-src/src/app/staff/share/bib-summary.component.html
index 05737d8544..8011524479 100644
--- a/Open-ILS/webby-src/src/app/staff/share/bib-summary.component.html
+++ b/Open-ILS/webby-src/src/app/staff/share/bib-summary.component.html
@@ -1,8 +1,22 @@
 
 <div class='eg-bib-summary card w-100' *ngIf="summary">
-  <div class="card-header font-weight-bold" i18n>
-    Record Summary
-    <!-- TODO expandy at right-->
+  <div class="card-header d-flex">
+    <div class="font-weight-bold">
+      Record Summary
+    </div>
+    <div class="flex-1"></div>
+    <div>
+      <a class="with-material-icon no-href text-primary" 
+        title="Show More" i18n-title
+        *ngIf="!expandDisplay" (click)="expandDisplay=true">
+        <span class="material-icons">expand_more</span>
+      </a>
+      <a class="with-material-icon no-href text-primary" 
+        title="Show Less" i18n-title
+        *ngIf="expandDisplay" (click)="expandDisplay=false">
+        <span class="material-icons">expand_less</span>
+      </a>
+    </div>
   </div>
   <div class="card-body">
     <ul class="list-group list-group-flush">
@@ -20,7 +34,7 @@
           </div>
         </div>
       </li>
-      <li class="list-group-item">
+      <li class="list-group-item" *ngIf="expandDisplay">
         <div class="d-flex">
           <div class="flex-1 font-weight-bold" i18n>Author:</div>
           <div class="flex-3">{{summary.author}}</div>
@@ -34,7 +48,7 @@
           </div>
         </div>
       </li>
-      <li class="list-group-item">
+      <li class="list-group-item" *ngIf="expandDisplay">
         <div class="d-flex">
           <div class="flex-1 font-weight-bold" i18n>Bib Call #:</div>
           <div class="flex-3">{{summary.callNumber}}</div>
diff --git a/Open-ILS/webby-src/src/app/staff/share/bib-summary.component.ts b/Open-ILS/webby-src/src/app/staff/share/bib-summary.component.ts
index 41990af205..1dbeff64aa 100644
--- a/Open-ILS/webby-src/src/app/staff/share/bib-summary.component.ts
+++ b/Open-ILS/webby-src/src/app/staff/share/bib-summary.component.ts
@@ -22,6 +22,8 @@ export class EgBibSummaryComponent implements OnInit {
         }
     }
 
+    expandDisplay: boolean = true;
+
     constructor(
         private cat: EgCatalogService,
         private net: EgNetService,
diff --git a/Open-ILS/webby-src/src/styles.css b/Open-ILS/webby-src/src/styles.css
index daa32c16ea..b8dd531fd6 100644
--- a/Open-ILS/webby-src/src/styles.css
+++ b/Open-ILS/webby-src/src/styles.css
@@ -18,6 +18,10 @@ h3 {font-size: 1.15rem}
 h4 {font-size: 1.05rem}
 h5 {font-size: .95rem}
 
+.no-href {
+  cursor: pointer;
+}
+
 
 /** BS has flex utility classes, but none for specifying flex widths */
 .flex-1 {flex: 1}