Record details / Related items tweaks, buttonize hold/list links in search results

- Made record details section be table-based rather than list-based
- tweaked spacing and margins for same
- made hold/list links in search results be buttons
- margins and spacing

Signed-off-by: William Rockwood <wrockwood@tadl.org>

diff --git a/css/tadl.css b/css/tadl.css
index 9bbc856..9979451 100644 (file)
--- a/css/tadl.css
+++ b/css/tadl.css
@@ -530,13 +530,13 @@ div.format_icon {
 }
 
 .results_aux_utils {
-       padding-left: 17px;
+       float:right;
+       padding-left: 27px;
        padding-bottom: 6px;
-       padding-right: 50px;
+       padding-right: 10px;
 }
 
 .result_util {
-    border-bottom: 1px dotted #ccc;
     padding-top: 6px;
 }
 

diff --git a/templates_tadlskin/opac/parts/record/subjects.tt2 b/templates_tadlskin/opac/parts/record/subjects.tt2
index 52d9ac8..1c40043 100644 (file)
--- a/templates_tadlskin/opac/parts/record/subjects.tt2
+++ b/templates_tadlskin/opac/parts/record/subjects.tt2
@@ -48,26 +48,26 @@
     END 
 %]
 
-[%  BLOCK render_all_subjects;
-    FOREACH subj IN subjects;
+[%  BLOCK render_all_subjects; %]
+        <table class='rdetail_subject'>
+            <tbody>
+[%  FOREACH subj IN subjects;
         content = PROCESS render_subject(xpath=subj.xpath);
         IF content.match('\S');
 %]
-        <table class='rdetail_subject'>
-            <tbody>
                 <tr>
                     <td class='rdetail_subject_type'>[% subj.label %]</td>
                     <td class='rdetail_subject_value' itemprop='keywords'>[% content %]</td>
                 </tr>
-            </tbody>
-        </table>
         [%- END; %]
     [%- END; %]
+            </tbody>
+        </table>
 [%- END %]
 
 [%-  subject_html = PROCESS render_all_subjects;
     IF subject_html.length > 0;
 %]
-<h2 class='rdetail_related_subjects'>[% l('Search for related items by subject') %]</h2>
+<h2 class='rdetail_related_subjects'>[% l('Related items') %]</h2>
 [%- subject_html %]
 [%- END %]

diff --git a/templates_tadlskin/opac/parts/record/summary.tt2 b/templates_tadlskin/opac/parts/record/summary.tt2
index 696f29d..4b981b1 100644 (file)
--- a/templates_tadlskin/opac/parts/record/summary.tt2
+++ b/templates_tadlskin/opac/parts/record/summary.tt2
@@ -109,7 +109,6 @@
 num_uris = merged_uris.size;
 IF num_uris > 0;
 -%]
-<h2 class="rdetail_uris">[% l("Electronic resources") %]</h2>
 <div class="rdetail_uris">
     [%- IF num_uris > 1 %]<ul>[% END %]
     [%- FOR uri IN merged_uris %]
@@ -132,45 +131,45 @@ IF num_uris > 0;
 
 <div class="padding-ten float-left">
 <h2 id='rdetail_record_details'>[% l("Record details") %]</h2>
-<ul>
+<table><tbody>
     [%- IF attrs.isbns.0; FOR isbn IN attrs.isbns %]
-    <li class='rdetail_isbns'>
-        <strong class='rdetail_label'>[% l('ISBN:'); %]</strong>
-        <span class='rdetail_value' itemprop='isbn'>[% isbn | html  %]</span>
-    </li>
+        <tr>
+        <td><strong class='rdetail_label'>[% l('ISBN:'); %]</strong></td>
+        <td><span class='rdetail_value' itemprop='isbn'>[% isbn | html  %]</span></td>
+        </tr>
         [%- END %]
     [%- END %]
     [%- IF attrs.issns.0; FOR issn IN attrs.issns %]
-    <li class='rdetail_issns'>
-        <strong class='rdetail_label'>[% l('ISSN:'); %]</strong>
-        <span class='rdetail_value'>[% issn | html  %]</span>
-    </li>
+        <tr>
+        <td><strong class='rdetail_label'>[% l('ISSN:'); %]</strong></td>
+        <td><span class='rdetail_value'>[% issn | html  %]</span></td>
+        </tr>
         [%- END %]
     [%- END %]
     [%- IF attrs.phys_desc %]
-    <li id='rdetail_phys_desc'>
-        <strong class='rdetail_label'>[% l("Physical Description:") %]</strong>
-        <span class='rdetail_value'>[% attrs.phys_desc | html %]</span>
-    </li>
+        <tr>
+        <td><strong class='rdetail_label'>[% l("Physical Description:") %]</strong></td>
+        <td><span class='rdetail_value'>[% attrs.phys_desc | html %]</span></td>
+        </tr>
     [%- END %]
     [%- IF attrs.edition %]
-    <li id='rdetail_edition'>
-        <strong class='rdetail_label'>[% l("Edition:") %]</strong>
-        <span class='rdetail_value'>[% attrs.edition | html %]</span>
-    </li>
+        <tr>
+        <td><strong class='rdetail_label'>[% l("Edition:") %]</strong></td>
+        <td><span class='rdetail_value'>[% attrs.edition | html %]</span></td>
+        </tr>
     [%- END %]
     [%- IF attrs.publisher %]
-    <li id='rdetail_publisher'>
-        <strong class='rdetail_label'>[% l("Publisher:") %]</strong>
-        <span class='rdetail_value' itemprop="publisher">[%
+        <tr>
+        <td><strong class='rdetail_label'>[% l("Publisher:") %]</strong></td>
+        <td><span class='rdetail_value' itemprop="publisher">[%
             attrs.publisher | html;
         %]</span>
         [% IF attrs.pubdate; %]
         <span itemprop="datePublished">[% attrs.pubdate | html; %]</span>
-        [% END; %]
-    </li>
+        [% END; %]</td>
+        </tr>
     [%- END %]
-</ul>
+</tbody></table>
 </div>
 <div class="padding-ten float-left">
 [%- INCLUDE "opac/parts/record/subjects.tt2" %]

diff --git a/templates_tadlskin/opac/parts/result/table.tt2 b/templates_tadlskin/opac/parts/result/table.tt2
index 58e1d83..71dd9c5 100644 (file)
--- a/templates_tadlskin/opac/parts/result/table.tt2
+++ b/templates_tadlskin/opac/parts/result/table.tt2
@@ -232,25 +232,23 @@
                                                 <div class="result_table_utils_cont">
                                                     <div class="result_table_utils">
                                                         [% IF !args.ezproxy %]
-                                                        <div class="results_aux_utils place_hold"><a
+                                                        <div class="results_aux_utils result_util"><a
                                                                 href="[% mkurl(ctx.opac_root _ '/place_hold', 
                                                                     {hold_target => rec.id, hold_type => 'T', hold_source_page => mkurl()}) %]" 
-                                                                    name="place_hold_link" class="no-dec"><img
-                                                                src="[% ctx.media_prefix %]/images/green_check.png"
-                                                                alt=""/><span class="result_place_hold">[% l('Place Hold') %]</span></a>
-                                                        </div>
+                                                                    name="place_hold_link" class="tbutton small green">
+                                                                <span>[% l('Place Hold') %]</span></a>
+                                                        <!--</div>-->
                                                         [% END %]
-                                                        <div class="results_aux_utils result_util">
+                                                        <!--<div class="results_aux_utils result_util">--><br />
                                                             [%  
                                                                 operation = ctx.mylist.grep(rec.id).size ? "delete" : "add";
-                                                                label = (operation == "add") ? l("Add to my list") : l("Remove from my list");
+                                                                label = (operation == "add") ? l("Add to list") : l("Remove from list");
                                                                 href = mkurl(ctx.opac_root _ '/mylist/' _ operation, 
                                                                         {record => rec.id, anchor => 'record_' _ rec.id}, 1);
                                                             %]      
-                                                            <a href="[% href %]" class="no-dec">
-                                                                <img src="[% ctx.media_prefix %]/images/clipboard.png" alt="" />
+                                                            <a href="[% href %]" class="tbutton small tadlblue"><span>
                                                                 [% label %]
-                                                            </a>
+                                                            </span></a>
                                                         </div>
                                                         [% IF ENV.OILS_CONTENT_CAFE_USER %]
                                                         <div class="results_aux_utils result_util">