[open-ils-commits] r19041 - trunk/Open-ILS/web/opac/skin/default/js (senator)

[svn at svn.open-ils.org]

Author: senator
Date: 2010-12-22 11:59:47 -0500 (Wed, 22 Dec 2010)
New Revision: 19041

Modified:
   trunk/Open-ILS/web/opac/skin/default/js/rdetail.js
Log:
OPAC/Serials: small aesthetic change to render held issues more neatly


Modified: trunk/Open-ILS/web/opac/skin/default/js/rdetail.js
===================================================================
--- trunk/Open-ILS/web/opac/skin/default/js/rdetail.js	2010-12-21 23:10:42 UTC (rev 19040)
+++ trunk/Open-ILS/web/opac/skin/default/js/rdetail.js	2010-12-22 16:59:47 UTC (rev 19041)
@@ -1231,22 +1231,26 @@
                 "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 @@
                                             },
                                             "innerHTML": "[" +
                                                 opac_strings.PLACE_HOLD + "]"
-                                        }, target
+                                        }, td
                                     );
                                 }
                             );