From 50e9c3a5e85175eae3b496353c1bc8a5de819956 Mon Sep 17 00:00:00 2001
From: Jeff Davis <jdavis@sitka.bclibraries.ca>
Date: Thu, 16 Jun 2022 15:49:04 -0700
Subject: [PATCH] LP#1982217: Fix ebook API display in Bootstrap OPAC

  - Make e-item checkout/hold buttons look like regular buttons.
  - Make ebook "holdings" table look like the regular holdings table.

Signed-off-by: Jeff Davis <jeff.davis@bc.libraries.coop>
---
 .../opac/parts/ebook_api/avail.tt2                 |  2 +-
 .../opac/parts/record/summary.tt2                  | 26 +++++++++-------------
 .../opac/parts/result/table.tt2                    | 10 +++++----
 3 files changed, 17 insertions(+), 21 deletions(-)

diff --git a/Open-ILS/src/templates-bootstrap/opac/parts/ebook_api/avail.tt2 b/Open-ILS/src/templates-bootstrap/opac/parts/ebook_api/avail.tt2
index 9df0465cc4..5d53c1d497 100755
--- a/Open-ILS/src/templates-bootstrap/opac/parts/ebook_api/avail.tt2
+++ b/Open-ILS/src/templates-bootstrap/opac/parts/ebook_api/avail.tt2
@@ -23,7 +23,7 @@ IF args.ebook;
 -%]
 <div id="[% ebook.rec_id %]" class="ebook_avail hidden">
     <div id="[% ebook.ebook_id %]" class="[% ebook.vendor %]_avail">
-        <table id="[% ebook.rec_id %]_ebook_holdings" class="result_holdings_table hidden">
+        <table id="[% ebook.rec_id %]_ebook_holdings" class="result_table_title_cell container-fluid table table-hover mt-4 miniTable holdingsTable hidden">
             <thead>
                 <tr>
                     <th>[% l('Available Formats') %]</th>
diff --git a/Open-ILS/src/templates-bootstrap/opac/parts/record/summary.tt2 b/Open-ILS/src/templates-bootstrap/opac/parts/record/summary.tt2
index 94ae7f5344..c1cd146bbd 100755
--- a/Open-ILS/src/templates-bootstrap/opac/parts/record/summary.tt2
+++ b/Open-ILS/src/templates-bootstrap/opac/parts/record/summary.tt2
@@ -372,22 +372,16 @@ ctx.metalinks.push('
                 [%- END -%]
 
                 [%- IF ebook_api.enabled == 'true' && args.ebook %]
-              
-                    <div id="[%- ctx.bre_id -%]_ebook_checkout" class="rdetail_aux_utils ebook_action hidden">
-                        <a class="btn btn-action no-dec" role="button" href="[% mkurl(ctx.opac_root _ '/myopac/ebook_checkout',
-                    {title => args.ebook.ebook_id, vendor => args.ebook.vendor, action => 'checkout'}, stop_parms) %]"
-                            rel="nofollow" vocab="">
-                            <span class="place_hold"><i class="fas fa-check" aria-hidden="true"></i> [% l('Check Out E-Item') %]</span></a>
-                    </div>
-               
-               
-                    <div id="[%- ctx.bre_id -%]_ebook_place_hold" class="rdetail_aux_utils ebook_action hidden">
-                        <a class="btn btn-action no-dec" role="button" href="[% mkurl(ctx.opac_root _ '/myopac/ebook_place_hold',
-                    {title => args.ebook.ebook_id, vendor => args.ebook.vendor, action => 'place_hold'}, stop_parms) %]"
-                            rel="nofollow" vocab="">
-                            <span class="place_hold"><i class="fas fa-check" aria-hidden="true"></i> [% l('Place Hold on E-Item') %]</span></a>
-                    </div>
-               
+                    <a id="[%- ctx.bre_id -%]_ebook_checkout"
+                        class="btn btn-action no-dec hidden" role="button" href="[% mkurl(ctx.opac_root _ '/myopac/ebook_checkout',
+                        {title => args.ebook.ebook_id, vendor => args.ebook.vendor, action => 'checkout'}, stop_parms) %]"
+                        rel="nofollow" vocab="">
+                        <span class="place_hold"><i class="fas fa-check" aria-hidden="true"></i> [% l('Check Out E-Item') %]</span></a>
+                    <a id="[%- ctx.bre_id -%]_ebook_place_hold"
+                        class="btn btn-action no-dec hidden" role="button" href="[% mkurl(ctx.opac_root _ '/myopac/ebook_place_hold',
+                        {title => args.ebook.ebook_id, vendor => args.ebook.vendor, action => 'place_hold'}, stop_parms) %]"
+                        rel="nofollow" vocab="">
+                        <span class="place_hold"><i class="fas fa-check" aria-hidden="true"></i> [% l('Place Hold on E-Item') %]</span></a>
                 [%- END -%]
 
 
diff --git a/Open-ILS/src/templates-bootstrap/opac/parts/result/table.tt2 b/Open-ILS/src/templates-bootstrap/opac/parts/result/table.tt2
index d01c1722d4..c46e1cda33 100755
--- a/Open-ILS/src/templates-bootstrap/opac/parts/result/table.tt2
+++ b/Open-ILS/src/templates-bootstrap/opac/parts/result/table.tt2
@@ -400,18 +400,20 @@
                                         <!-- eBook Buttons -->
                                         [%- IF ebook_api.enabled == 'true' && args.ebook %]
                                       
-                                            <a href="[% mkurl(ctx.opac_root _ '/myopac/ebook_checkout',
+                                            <a id="[%- rec.id -%]_ebook_checkout"
+                                                href="[% mkurl(ctx.opac_root _ '/myopac/ebook_checkout',
                                                 {title => args.ebook.ebook_id, vendor => args.ebook.vendor, action => 'checkout'},
                                                 ['query','tag','subfield','term','_special','sort','page']) %]"
                                                 [% html_text_attr('title', l('Check Out [_1]', attrs.title)) %]
-                                                    class="btn btn-action" role="button" rel="nofollow" vocab=""><i class="fas fa-atlas" aria-hidden="true"></i> <span class="result_place_hold">[% l('Check Out E-Item') %]</span></a>
+                                                    class="btn btn-action hidden" role="button" rel="nofollow" vocab=""><i class="fas fa-atlas" aria-hidden="true"></i> <span class="result_place_hold">[% l('Check Out E-Item') %]</span></a>
                                        
                                        
-                                            <a href="[% mkurl(ctx.opac_root _ '/myopac/ebook_place_hold',
+                                            <a id="[%- rec.id -%]_ebook_place_hold"
+                                                href="[% mkurl(ctx.opac_root _ '/myopac/ebook_place_hold',
                                                 {title => args.ebook.ebook_id, vendor => args.ebook.vendor, action => 'place_hold'},
                                                 ['query','tag','subfield','term','_special','sort','page']) %]"
                                                 [% html_text_attr('title', l('Place Hold on [_1]', attrs.title)) %]
-                                                    class="btn btn-action" role="button" rel="nofollow" vocab=""><i class="fas fa-book-reader" aria-hidden="true"></i> <span class="result_place_hold">[% l('Place Hold on E-Item') %]</span></a>
+                                                    class="btn btn-action hidden" role="button" rel="nofollow" vocab=""><i class="fas fa-book-reader" aria-hidden="true"></i> <span class="result_place_hold">[% l('Place Hold on E-Item') %]</span></a>
                                        
                                         [%- END -%]
 
-- 
2.11.0