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

diff --git a/Open-ILS/xul/staff_client/server/patron/display.js b/Open-ILS/xul/staff_client/server/patron/display.js
index 32fad47..de499a9 100644 (file)
--- 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 98cfa6c..0f6afe8 100644 (file)
--- 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 a6f5e9c..3936250 100644 (file)
--- 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 7a4d2c2..91e3ca5 100644 (file)
--- 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 5a6595e..e3c0e6a 100644 (file)
--- 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>