From adce3ba75655c74a90b7cece4a34196ecdaf492c Mon Sep 17 00:00:00 2001
From: Dan Scott <dscott@laurentian.ca>
Date: Mon, 19 Sep 2011 19:43:28 -0400
Subject: [PATCH] Remove "Expand all extras" as there are fewer extras to
 expand

Cleaner.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Thomas Berezansky <tsbere@mvlc.org>
---
 .../templates/opac/parts/record/authors.tt2   |  2 +-
 .../templates/opac/parts/record/extras.tt2    |  1 -
 .../templates/opac/parts/record/summary.tt2   | 34 ++++---------------
 Open-ILS/web/css/skin/default/opac/style.css  | 23 ++++++-------
 4 files changed, 19 insertions(+), 41 deletions(-)

diff --git a/Open-ILS/src/templates/opac/parts/record/authors.tt2 b/Open-ILS/src/templates/opac/parts/record/authors.tt2
index 304e272df9..e2e749d9ae 100644
--- a/Open-ILS/src/templates/opac/parts/record/authors.tt2
+++ b/Open-ILS/src/templates/opac/parts/record/authors.tt2
@@ -44,7 +44,7 @@ BLOCK build_author_links;
 END;
 %]
 
-<div class='rdetail_extras_div'>
+<div class='rdetail_authors_div'>
 [% FOREACH author IN authors;
     NEXT UNLESS author.xpath; 
     links = PROCESS build_author_links(xpath=author.xpath, label=author.label);
diff --git a/Open-ILS/src/templates/opac/parts/record/extras.tt2 b/Open-ILS/src/templates/opac/parts/record/extras.tt2
index c6e403e2ec..4a473c3660 100644
--- a/Open-ILS/src/templates/opac/parts/record/extras.tt2
+++ b/Open-ILS/src/templates/opac/parts/record/extras.tt2
@@ -17,7 +17,6 @@
         extras = [
             {name => 'summaryplus',  label => l('Summaries &amp; More'), hide => hide_summary}, 
             {name => 'contents',  label => l('Contents'), hide => !attrs.contents},
-            {name => 'series',   label => l('Series')},
             {name => 'annotation', label => l('Annotation'), hide => 1}, 
             {name => 'awards',  label => l('Awards, Reviews, & Suggested Reads')}, 
             {name => 'excerpt',  label => l('Excerpt'), hide => 1},
diff --git a/Open-ILS/src/templates/opac/parts/record/summary.tt2 b/Open-ILS/src/templates/opac/parts/record/summary.tt2
index a77e3e1281..267f0efdf5 100644
--- a/Open-ILS/src/templates/opac/parts/record/summary.tt2
+++ b/Open-ILS/src/templates/opac/parts/record/summary.tt2
@@ -38,7 +38,7 @@
 </div>
 
 <div id='rdetail_title_div'>
-    <h1 id='rdetail_title'>[% attrs.title_extended | html %]</h1><br />
+    <h1 id='rdetail_title'>[% attrs.title_extended | html %]</h1>
     [%- INCLUDE "opac/parts/record/authors.tt2" %]
     [%- IF attrs.format_icon %]
     <div style="float:right;margin-right:17px;">
@@ -54,15 +54,16 @@
     %]
         <div id='rdetail_openurl'>
             <strong class='rdetail_openurl_title'>[% l("Electronic resources") %]</strong>
+            <table><tbody>
     [%-
             FOR res IN sfx;
     %]
-            <div class="rdetail_openurl_entry" style="margin-left: 1em;">
-                <strong><a href="[% res.target_url %]">
-                        [% res.public_name %]</a></strong>
-                <span>[% res.target_coverage %]</td>
-            </div>
+            <tr>
+                <td class='rdetail_openurl_entry'><a href="[% res.target_url %]">[% res.public_name %]</a></td>
+                <td>[% res.target_coverage %]</td>
+            </tr>
         [%- END %]
+        </tbody></table>
     [%- END %]
     [%- IF sfx.size && sfx.0 != '' %]
         </div>    
@@ -228,24 +229,3 @@
 
 <h2 class='rdetail_related_series'>[% l('Search for related items by series') %]</h2>
 [%- INCLUDE "opac/parts/record/series.tt2" %]
-
-<div>
-    [%- IF CGI.param('expand') == 'all' %]
-        <img src="[% ctx.media_prefix %]/images/minus_sign.png" />
-        <a href="[% mkurl('', {}, ['expand']) %]">[% l('Collapse all tabs') %]</a>
-    [%- ELSE %]
-        <img src="[% ctx.media_prefix %]/images/plus_sign.png" />
-        <a href="[% mkurl('', {expand => 'all'}) %]">[% l('Expand all tabs') %]</a>
-    [%- END %]
-</div>
-
-<div id="rdetail_extras_expand" class="hide_me">
-    <a href="#"><img
-        src="[% ctx.media_prefix %]/images/plus_sign.png" /></a>
-    <a style="position:relative;top:-3px;" href="#">[% l('Expand all tabs') %]</a>
-</div>
-
-<div id="rdetail_extras_collapse" class="hide_me">
-    <a href="#"><img src="[% ctx.media_prefix %]/images/plus_sign.png" /></a>
-    <a style="position:relative;top:-3px;" href="#">[% l('Collapse all tabs') %]</a>
-</div>
diff --git a/Open-ILS/web/css/skin/default/opac/style.css b/Open-ILS/web/css/skin/default/opac/style.css
index 2b747ebff5..540d14728b 100644
--- a/Open-ILS/web/css/skin/default/opac/style.css
+++ b/Open-ILS/web/css/skin/default/opac/style.css
@@ -11,11 +11,6 @@ img {
 
 a {
 	color: #003399;
-	text-decoration: none;
-}
-
-a:hover {
-	text-decoration: underline;
 }
 
 #search-wrapper input[type=text] {
@@ -54,7 +49,7 @@ h1 {
 h2 {
 	margin:0;
 	margin-bottom: 5px;
-	font-size: 14px;
+	font-size: 16px;
 	font-weight:bold;
 }
 
@@ -355,10 +350,6 @@ div.select-wrapper:hover {
 	margin-top: 15px;
 }
 
-#rdetail_title {
-	font-size: 18px;
-}
-
 #rdetail_image { border: none; }
 #rdetail_image_cell {
 	padding-top: 3px;
@@ -1056,8 +1047,7 @@ a.opac-button {
 
 #rdetail_record_details {
     clear: both;
-    margin-top: 1.5em;
-    padding-top: 1.5em;
+    margin-top: 1em;
 }
 
 .rdetail_subject_type {
@@ -1077,3 +1067,12 @@ a.opac-button {
 .rdetail_related_series {
     margin-top: 1.5em;
 }
+
+#rdetail_openurl {
+    margin-top: 1em;
+}
+
+.rdetail_openurl_entry {
+    margin-left: 1em;
+    padding-left: 1em;
+}
-- 
2.43.2