Show reviews/anotes by default if found

If the added content handler found reviews or author notes,
show that information by default (first reviews, then author
notes if there were no reviews).

If neither are found, show nothing by default -- but things
like Table of Contents or Excerpts will still be available
by clicking the proper link.

Signed-off-by: Jeff Godin <jgodin@tadl.org>

diff --git a/Open-ILS/web/opac/skin/tadlv3/js/rdetail.js b/Open-ILS/web/opac/skin/tadlv3/js/rdetail.js
index db7e25a..58b310c 100644 (file)
--- a/Open-ILS/web/opac/skin/tadlv3/js/rdetail.js
+++ b/Open-ILS/web/opac/skin/tadlv3/js/rdetail.js
@@ -881,6 +881,8 @@ function rdetailShowCNBrowse( cn, loc, depth, fromOnclick ) {
 
 function rdetailhandleAC(data) {
 
+    var found_review = false;
+
        if( data.summary.html ) {
                $('rdetail_summary_div').innerHTML = data.summary.html;
                unHideMe($('rdetail_summary_link'));
@@ -888,7 +890,9 @@ function rdetailhandleAC(data) {
 
        if( data.reviews.html ) {
                $('rdetail_review_container').innerHTML = data.reviews.html;
+        found_review = true;
                unHideMe($('rdetail_reviews_link'));
+        rdetailShowExtra('reviews');
        }
 
        if( data.toc.html ) {
@@ -904,6 +908,9 @@ function rdetailhandleAC(data) {
        if( data.anotes.html ) {
                $('rdetail_anotes_div').innerHTML = data.anotes.html;
                unHideMe($('rdetail_anotes_link'));
+        if (!found_review) {
+            rdetailShowExtra('anotes');
+        }
        }
 }
 

diff --git a/Open-ILS/web/opac/skin/tadlv3/xml/rdetail/rdetail_extras.xml b/Open-ILS/web/opac/skin/tadlv3/xml/rdetail/rdetail_extras.xml
index c9f563e..4003f97 100644 (file)
--- a/Open-ILS/web/opac/skin/tadlv3/xml/rdetail/rdetail_extras.xml
+++ b/Open-ILS/web/opac/skin/tadlv3/xml/rdetail/rdetail_extras.xml
@@ -4,12 +4,6 @@
        <table class='rdetails_extra_links'>
                <thead>
                        <tr>
-                               <td id='rdetail_viewcn_link' class='rdetail_extras_td'
-                                       style='padding-right: 15px; padding-left: 15px;' >
-                                       <a href='javascript:rdetailShowExtra("cn");' 
-                                               class='classic_link'>&rdetail.extras.browser;</a>
-                               </td>
-
                                <td id='rdetail_summary_link' class='hide_me rdetail_extras_td'
                                        style='padding-right: 15px; padding-left: 15px;' >
                                        <a href='javascript:rdetailShowExtra("summary");' 
@@ -22,6 +16,12 @@
                                                class='classic_link'>&rdetail.extras.reviews;</a>
                                </td>
 
+                               <td id='rdetail_anotes_link' class='hide_me rdetail_extras_td'
+                                       style='padding-right: 15px; padding-left: 15px;' >
+                                       <a href='javascript:rdetailShowExtra("anotes");' 
+                                               class='classic_link'>&rdetail.extras.author.notes;</a>
+                               </td>
+
                                <td id='rdetail_toc_link' class='hide_me rdetail_extras_td'
                                        style='padding-right: 15px; padding-left: 15px;' >
                                        <a href='javascript:rdetailShowExtra("toc");' 
@@ -40,18 +40,18 @@
                                                class='classic_link'>&rdetail.extras.preview;</a>
                                </td>
 
-                               <td id='rdetail_anotes_link' class='hide_me rdetail_extras_td'
-                                       style='padding-right: 15px; padding-left: 15px;' >
-                                       <a href='javascript:rdetailShowExtra("anotes");' 
-                                               class='classic_link'>&rdetail.extras.author.notes;</a>
-                               </td>
-
                                <td id='rdetail_annotation_link' class='hide_me rdetail_extras_td'
                                        style='padding-right: 15px; padding-left: 15px;' >
                                        <a href='javascript:rdetailShowExtra("annotation");' 
                                                class='classic_link'>&rdetail.extras.annotation;</a>
                                </td>
 
+                               <td id='rdetail_viewcn_link' class='rdetail_extras_td'
+                                       style='padding-right: 15px; padding-left: 15px;' >
+                                       <a href='javascript:rdetailShowExtra("cn");' 
+                                               class='classic_link'>&rdetail.extras.browser;</a>
+                               </td>
+
                                <td id='rdetail_viewmarc_link' class='rdetail_extras_td'
                                        style='padding-right: 15px; padding-left: 15px;' >
                                        <a href='javascript:rdetailShowExtra("marc");'