From bb1908a1449ecf3f4185ef4a077b2255d4a604d6 Mon Sep 17 00:00:00 2001
From: phasefx <phasefx@dcc99617-32d9-48b4-a31d-7c20da2025e4>
Date: Thu, 5 Mar 2009 17:51:40 +0000
Subject: [PATCH] fix patron summary toggle.  some tweaking with persist, which
 isn't working

git-svn-id: svn://svn.open-ils.org/ILS/trunk@12427 dcc99617-32d9-48b4-a31d-7c20da2025e4
---
 Open-ILS/xul/staff_client/server/patron/display.js       |  9 +++++++++
 Open-ILS/xul/staff_client/server/patron/display.xul      |  2 ++
 .../xul/staff_client/server/patron/display_horiz.xul     |  2 ++
 .../staff_client/server/patron/display_horiz_overlay.xul | 16 ++++++++--------
 .../xul/staff_client/server/patron/display_overlay.xul   | 12 ++++++------
 5 files changed, 27 insertions(+), 14 deletions(-)

diff --git a/Open-ILS/xul/staff_client/server/patron/display.js b/Open-ILS/xul/staff_client/server/patron/display.js
index 32fad47ce2..de499a95ce 100644
--- a/Open-ILS/xul/staff_client/server/patron/display.js
+++ b/Open-ILS/xul/staff_client/server/patron/display.js
@@ -113,6 +113,15 @@ patron.display.prototype = {
                             }
                         }
                     ],
+                    'cmd_patron_toggle_summary' : [
+                        ['command'],
+                        function(ev) {
+                            var x = document.getElementById('left_deck_vbox'); 
+                            if (x) {
+                                x.hidden = ! x.hidden;
+                            }
+                        }
+                    ],
 					'cmd_search_form' : [
 						['command'],
 						function(ev) {
diff --git a/Open-ILS/xul/staff_client/server/patron/display.xul b/Open-ILS/xul/staff_client/server/patron/display.xul
index 98cfa6c7a3..0f6afe897f 100644
--- a/Open-ILS/xul/staff_client/server/patron/display.xul
+++ b/Open-ILS/xul/staff_client/server/patron/display.xul
@@ -32,6 +32,7 @@
 	<scripts id="openils_util_scripts"/>
 
 	<script type="text/javascript" src="/xul/server/main/JSAN.js"/>
+	<script type="text/javascript" src="/xul/server/patron/display.js"/>
 	<script>
 	<![CDATA[
 		function $(id) { return document.getElementById(id); }
@@ -104,6 +105,7 @@
 		<command id="cmd_patron_exit" />
 		<command id="cmd_patron_retrieve" />
 		<command id="cmd_patron_merge" />
+		<command id="cmd_patron_toggle_summary" />
 		<command id="cmd_search_form" />
 		<command id="cmd_verify_credentials" />
 		<command id="cmd_perm_editor" />
diff --git a/Open-ILS/xul/staff_client/server/patron/display_horiz.xul b/Open-ILS/xul/staff_client/server/patron/display_horiz.xul
index a6f5e9ca8e..393625049b 100644
--- a/Open-ILS/xul/staff_client/server/patron/display_horiz.xul
+++ b/Open-ILS/xul/staff_client/server/patron/display_horiz.xul
@@ -32,6 +32,7 @@
 	<scripts id="openils_util_scripts"/>
 
 	<script type="text/javascript" src="/xul/server/main/JSAN.js"/>
+	<script type="text/javascript" src="/xul/server/patron/display.js"/>
 	<script>
 	<![CDATA[
 		function $(id) { return document.getElementById(id); }
@@ -104,6 +105,7 @@
 		<command id="cmd_patron_exit" />
 		<command id="cmd_patron_retrieve" />
 		<command id="cmd_patron_merge" />
+		<command id="cmd_patron_toggle_summary" />
 		<command id="cmd_search_form" />
 		<command id="cmd_verify_credentials" />
 		<command id="cmd_perm_editor" />
diff --git a/Open-ILS/xul/staff_client/server/patron/display_horiz_overlay.xul b/Open-ILS/xul/staff_client/server/patron/display_horiz_overlay.xul
index 7a4d2c26c3..91e3ca57f3 100644
--- a/Open-ILS/xul/staff_client/server/patron/display_horiz_overlay.xul
+++ b/Open-ILS/xul/staff_client/server/patron/display_horiz_overlay.xul
@@ -32,13 +32,13 @@
 		<label class="hideme invalid_address_indicator" value="&staff.patron.display_overlay.invalid_address.value;"/>
 	</hbox>
 	<vbox id="PatronNotNavBar" flex="1" class="my_bg">
-        <vbox id="pdm3y" flex="1" persist="height"> 
-            <deck id="patron_left_deck"/>
-        </vbox>
-        <splitter id="pdms1y" orient="vertical" collapse="before" persist="state hidden"><grippy id="pdmsg1"/></splitter>
-        <vbox id="pdm4y" flex="8" persist="height">
-            <deck id="patron_right_deck"/>
-        </vbox>
+        <hbox id="left_deck_vbox" flex="1" persist="height"> 
+            <deck id="patron_left_deck" persist="height"/>
+        </hbox>
+        <splitter id="pdms1y" collapse="before" persist="state hidden"><grippy id="pdmsg1"/></splitter>
+        <hbox id="right_deck_vbox" flex="8" persist="height">
+            <deck id="patron_right_deck" persist="height"/>
+        </hbox>
 	</vbox>
 </vbox>
 </box>
@@ -96,7 +96,7 @@
                                     <menuitem label="&staff.patron_display.verify_password.label;" accesskey="&staff.patron_display.verify_password.accesskey;" command="cmd_verify_credentials"/>
                                     <menuitem label="&staff.main.menu.admin.user_edit.label;" accesskey="&staff.main.menu.admin.user_edit.accesskey;" command="cmd_perm_editor"/>
                                     <menuitem label="&staff.patron.display_overlay.exit.label;" accesskey="&staff.patron.display_overlay.exit.accesskey;" command="cmd_patron_exit"/>
-                                    <menuitem label="&staff.patron_display.toggle_summary.label;" accesskey="&staff.patron_display.toggle_summary.label;" oncommand="document.getElementById('patron_left_deck').hidden = ! document.getElementById('patron_left_deck').hidden;"/>
+                                    <menuitem label="&staff.patron_display.toggle_summary.label;" accesskey="&staff.patron_display.toggle_summary.label;" command="cmd_patron_toggle_summary"/>
                                 </menupopup>
                             </button>
                         </row>
diff --git a/Open-ILS/xul/staff_client/server/patron/display_overlay.xul b/Open-ILS/xul/staff_client/server/patron/display_overlay.xul
index 5a6595e999..e3c0e6a93b 100644
--- a/Open-ILS/xul/staff_client/server/patron/display_overlay.xul
+++ b/Open-ILS/xul/staff_client/server/patron/display_overlay.xul
@@ -32,12 +32,12 @@
 		<label class="hideme invalid_address_indicator" value="&staff.patron.display_overlay.invalid_address.value;"/>
 	</hbox>
 	<hbox id="PatronNotNavBar" flex="1" class="my_bg">
-        <vbox id="pdm3z" flex="1" persist="width"> 
-            <deck id="patron_left_deck"/>
+        <vbox id="left_deck_vbox" flex="1" persist="width"> 
+            <deck id="patron_left_deck" persist="width"/>
         </vbox>
-        <splitter id="pdms1z" orient="horizontal" collapse="before" persist="state hidden"><grippy id="pdmsg1"/></splitter>
-        <vbox id="pdm4z" flex="3" persist="width">
-            <deck id="patron_right_deck"/>
+        <splitter id="pdms1z" collapse="before" persist="state hidden"><grippy id="pdmsg1"/></splitter>
+        <vbox id="right_deck_vbox" flex="3" persist="width">
+            <deck id="patron_right_deck" persist="width"/>
         </vbox>
 	</hbox>
 </vbox>
@@ -96,7 +96,7 @@
                                     <menuitem label="&staff.patron_display.verify_password.label;" accesskey="&staff.patron_display.verify_password.accesskey;" command="cmd_verify_credentials"/>
                                     <menuitem label="&staff.main.menu.admin.user_edit.label;" accesskey="&staff.main.menu.admin.user_edit.accesskey;" command="cmd_perm_editor"/>
                                     <menuitem label="&staff.patron.display_overlay.exit.label;" accesskey="&staff.patron.display_overlay.exit.accesskey;" command="cmd_patron_exit"/>
-                                    <menuitem label="&staff.patron_display.toggle_summary.label;" accesskey="&staff.patron_display.toggle_summary.label;" oncommand="document.getElementById('patron_left_deck').hidden = ! document.getElementById('patron_left_deck').hidden;"/>
+                                    <menuitem label="&staff.patron_display.toggle_summary.label;" accesskey="&staff.patron_display.toggle_summary.label;" command="cmd_patron_toggle_summary"/>
                                 </menupopup>
                             </button>
                         </row>
-- 
2.11.0