From 9a26ea80b90f968cf72b597d60f8bfc7fcf8c901 Mon Sep 17 00:00:00 2001
From: senator <senator@dcc99617-32d9-48b4-a31d-7c20da2025e4>
Date: Wed, 22 Dec 2010 16:59:47 +0000
Subject: [PATCH] OPAC/Serials: small aesthetic change to render held issues
 more neatly

git-svn-id: svn://svn.open-ils.org/ILS/trunk@19041 dcc99617-32d9-48b4-a31d-7c20da2025e4
---
 Open-ILS/web/opac/skin/default/js/rdetail.js | 14 +++++++++-----
 1 file changed, 9 insertions(+), 5 deletions(-)

diff --git a/Open-ILS/web/opac/skin/default/js/rdetail.js b/Open-ILS/web/opac/skin/default/js/rdetail.js
index 08e8319464..b1fc03b255 100644
--- a/Open-ILS/web/opac/skin/default/js/rdetail.js
+++ b/Open-ILS/web/opac/skin/default/js/rdetail.js
@@ -1231,22 +1231,26 @@ function rdetailDrawExpandedHoldings(anchor, bibid, type) {
                 "oncomplete": function(r) {
                     try {
                         if (msg = r.recv().content()) { /* sic, assignment */
+                            if (!msg.length) return;
                             offsets[type] += msg.length;
+                            var table = dojo.create("table", null, target);
                             dojo.forEach(
                                 msg, function(o) {
-                                    dojo.create("br", null, target);
+                                    var tr = dojo.create("tr", null, table);
                                     dojo.create(
-                                        "span", {
+                                        "td", {
                                             "innerHTML": o.issuance.label(),
-                                            "style": {"padding": "0 2em"}
-                                        }, target
+                                            "style": {"paddingLeft": "3em"}
+                                        }, tr
                                     );
 
                                     if (!o.has_units) return;
                                     /* can't place holds if no units */
+                                    var td = dojo.create("td", null, tr);
                                     dojo.create(
                                         "a", {
                                             "href":"javascript:void(0);",
+                                            "style": {"marginLeft": "1.5em"},
                                             "onclick": function() {
                                                 holdsDrawEditor({
                                                     "type": "I",
@@ -1255,7 +1259,7 @@ function rdetailDrawExpandedHoldings(anchor, bibid, type) {
                                             },
                                             "innerHTML": "[" +
                                                 opac_strings.PLACE_HOLD + "]"
-                                        }, target
+                                        }, td
                                     );
                                 }
                             );
-- 
2.11.0