[open-ils-commits] r11467 - trunk/Open-ILS/xul/staff_client/server/patron

[svn at svn.open-ils.org]

Author: miker
Date: 2008-12-08 23:01:11 -0500 (Mon, 08 Dec 2008)
New Revision: 11467

Modified:
   trunk/Open-ILS/xul/staff_client/server/patron/summary.js
   trunk/Open-ILS/xul/staff_client/server/patron/summary_overlay.xul
Log:
use grid instead of hbox

Modified: trunk/Open-ILS/xul/staff_client/server/patron/summary.js
===================================================================
--- trunk/Open-ILS/xul/staff_client/server/patron/summary.js	2008-12-09 03:52:35 UTC (rev 11466)
+++ trunk/Open-ILS/xul/staff_client/server/patron/summary.js	2008-12-09 04:01:11 UTC (rev 11467)
@@ -104,23 +104,23 @@
 								var penalties = obj.patron.standing_penalties();
 								for (var i = 0; i < penalties.length; i++) {
 
-									var b = document.createElement('hbox');
-									var r = document.createElement('button');
-									var x = document.createElement('label');
+									var row = document.createElement('row');
+									var label = document.createElement('label');
+									var button = document.createElement('button');
 
 									//x.setAttribute('value',penalties[i].penalty_type());
-									x.setAttribute('value',penalties[i].standing_penalty().name());
-									b.appendChild(x);
+									label.setAttribute('value',penalties[i].standing_penalty().label());
+									row.appendChild(label);
 
                                     // XXX check a permission here? How to fire the remove action ??? XXX
-									r.setAttribute('label', $("patronStrings").getString('staff.patron.summary.standing_penalty.remove'));
-									b.appendChild(r);
+									button.setAttribute('label', $("patronStrings").getString('staff.patron.summary.standing_penalty.remove'));
+									row.appendChild(button);
 
-                                    if (penalties[i].standing_penalty().block_list().match(/RENEW/)) addCSSClass(b,'PENALTY_RENEW');
-                                    if (penalties[i].standing_penalty().block_list().match(/HOLD/)) addCSSClass(b,'PENALTY_HOLD');
-                                    if (penalties[i].standing_penalty().block_list().match(/CIRC/)) addCSSClass(b,'PENALTY_CIRC');
+                                    if (penalties[i].standing_penalty().block_list().match(/RENEW/)) addCSSClass(label,'PENALTY_RENEW');
+                                    if (penalties[i].standing_penalty().block_list().match(/HOLD/)) addCSSClass(label,'PENALTY_HOLD');
+                                    if (penalties[i].standing_penalty().block_list().match(/CIRC/)) addCSSClass(label,'PENALTY_CIRC');
 
-									e2.appendChild(b);
+									e2.appendChild(row);
                                     e2.parentNode.hidden = false;
 								}
 							};

Modified: trunk/Open-ILS/xul/staff_client/server/patron/summary_overlay.xul
===================================================================
--- trunk/Open-ILS/xul/staff_client/server/patron/summary_overlay.xul	2008-12-09 03:52:35 UTC (rev 11466)
+++ trunk/Open-ILS/xul/staff_client/server/patron/summary_overlay.xul	2008-12-09 04:01:11 UTC (rev 11467)
@@ -21,7 +21,12 @@
 	</groupbox>
 	<groupbox id="PatronSummaryStanding_groupbox" flex="0" hidden="true" class="standing">
 		<caption id="pssgbc" label="&staff.patron.summary_overlay.pssgbc.standing.label;" class="shrinkable_groupbox"/>
-		<vbox id="patron_standing_penalties"/>
+		<grid id="patron_standing_penalties">
+            <columns>
+                <column/>
+                <column/>
+            </columns>
+        </grid>
 	</groupbox>
 	<groupbox id="PatronSummaryStatus_groupbox" flex="0" class="status"/>
 	<!--