From e7635b436baa3a7d3f47f3e6fc416c4dd0cf2b43 Mon Sep 17 00:00:00 2001
From: Jeff Godin <jgodin@tadl.org>
Date: Mon, 23 Apr 2012 04:30:04 -0400
Subject: [PATCH] Change the display of subjects in the sidebar

For the sidebar display of subjects, use one link per subject
item, not the "increasingly specific" multiple link style of
the record detail summary.

Also minor other formatting changes, such as using <li>. Change
to slot.parentNode reference for unHideMe.

Signed-off-by: Jeff Godin <jgodin@tadl.org>
---
 .../web/opac/skin/tadlv4/xml/common/sidebar.xml    | 30 +++++++++++-----------
 1 file changed, 15 insertions(+), 15 deletions(-)

diff --git a/Open-ILS/web/opac/skin/tadlv4/xml/common/sidebar.xml b/Open-ILS/web/opac/skin/tadlv4/xml/common/sidebar.xml
index 78bef51eb4..e3785f5729 100644
--- a/Open-ILS/web/opac/skin/tadlv4/xml/common/sidebar.xml
+++ b/Open-ILS/web/opac/skin/tadlv4/xml/common/sidebar.xml
@@ -59,32 +59,32 @@
                 <div>
                 <h4>Related Subjects</h4>
                 </div>
-                <div type='opac/slot-data' query='datafield[tag^="6"]' class='rdetail_item'>
+                <div type='opac/slot-data' query='datafield[tag^="6"]'>
                     <script type='opac/slot-format'><![CDATA[
                         var cgi = new CGI();
                         var other_params = [ 'd', 'l', 'r', 'av', 's', 'sd', 'ol' ];
-                        var total = '';
-                        var output = [];
+                        var subj_display = '';
+                        var subj_search = '';
+                        var subj_line = '';
                         var list = dojo.query( 'subfield', item );
                         for (var i = 0; i < list.length; i++) {
                             var sub_code = list[i].getAttribute('code');
                             if ( sub_code.match(/[a-z]/i) ) {
-                                var current = '';
                                 if ( sub_code.match(/[vxyz]/) ) {
-                                    current += '&#x2d;&#x2d; ';
+                                    subj_display += '&#x2d;&#x2d; ';
                                 }
-                                total += BT.textContent(list[i]) + ' ';
-                                current += '<a href="rresult.xml?rt=subject&tp=subject&t=' + total.replace(/\s+$/, '');
-                                for (var p in other_params) {
-                                    if (cgi.param(other_params[p]))
-                                        current += '&' + other_params[p] + '=' + cgi.param(other_params[p]);
-                                }
-                                current += '">' + BT.textContent(list[i]) + '</a>'
-                                output.push(current);
+                                subj_display += BT.textContent(list[i]) + ' ';
+                                subj_search += BT.textContent(list[i]) + ' ';
                             }
                         }
-                        unHideMe($('sidebarSubjects'));
-                        return '<span>' + output.join(' ') + '</span><br/>';
+                        subj_line += '<li><a href="rresult.xml?rt=subject&tp=subject&t=' + subj_search.replace(/\s+$/, '');
+                        for (var p in other_params) {
+                            if (cgi.param(other_params[p]))
+                                subj_line += '&' + other_params[p] + '=' + cgi.param(other_params[p]);
+                        }
+                        subj_line += '">' + subj_display + '</a></li>';
+                        unHideMe(slot.parentNode);
+                        return '<span>' + subj_line + '</span>';
                     ]]></script>
                 </div>
     </div>
-- 
2.11.0