[open-ils-commits] r13201 - trunk/Open-ILS/xul/staff_client/server/cat (phasefx)

[svn at svn.open-ils.org]

Author: phasefx
Date: 2009-05-17 23:53:24 -0400 (Sun, 17 May 2009)
New Revision: 13201

Modified:
   trunk/Open-ILS/xul/staff_client/server/cat/z3950.js
   trunk/Open-ILS/xul/staff_client/server/cat/z3950.xul
Log:
make clicking on the grippy in the z39.50 client and pressing the Hide/Show Top Pane button do the same thing/play well together.  The text 'Hide' versus 'Show' can get out of sync, though.  Could punt and relabel it 'Toggle'

Modified: trunk/Open-ILS/xul/staff_client/server/cat/z3950.js
===================================================================
--- trunk/Open-ILS/xul/staff_client/server/cat/z3950.js	2009-05-17 08:58:05 UTC (rev 13200)
+++ trunk/Open-ILS/xul/staff_client/server/cat/z3950.js	2009-05-18 03:53:24 UTC (rev 13201)
@@ -202,9 +202,9 @@
                             ['command'],
                             function() {
                                 var x = document.getElementById('top_pane');
-                                x.hidden = ! x.hidden;
+                                document.getElementById('splitter_grippy2').doCommand();
 								var n = obj.controller.view.toggle_form_btn;
-                                if (x.hidden) {
+                                if (x.collapsed) {
                                     n.setAttribute('image',"/xul/server/skin/media/images/down_arrow.gif");
                                     n.setAttribute('label',$("catStrings").getString('staff.cat.z3950.unhide_top_pane.label'));
                                     n.setAttribute('accesskey',$("catStrings").getString('staff.cat.z3950.unhide_top_pane.accesskey'));
@@ -215,6 +215,22 @@
                                 }
                             },
                         ],
+                        'splitter_grippy2' : [
+                            ['click'],
+                            function() {
+                                var x = document.getElementById('top_pane');
+                                var n = obj.controller.view.toggle_form_btn;
+                                if (x.collapsed) {
+                                    n.setAttribute('image',"/xul/server/skin/media/images/down_arrow.gif");
+                                    n.setAttribute('label',$("catStrings").getString('staff.cat.z3950.unhide_top_pane.label'));
+                                    n.setAttribute('accesskey',$("catStrings").getString('staff.cat.z3950.unhide_top_pane.accesskey'));
+                                } else {
+                                    n.setAttribute('image',"/xul/server/skin/media/images/up_arrow.gif");
+                                    n.setAttribute('label',$("catStrings").getString('staff.cat.z3950.hide_top_pane.label'));
+                                    n.setAttribute('accesskey',$("catStrings").getString('staff.cat.z3950.hide_top_pane.accesskey'));
+                                }
+                            }
+                        ],
 						'service_rows' : [
 							['render'],
 							function(e) {

Modified: trunk/Open-ILS/xul/staff_client/server/cat/z3950.xul
===================================================================
--- trunk/Open-ILS/xul/staff_client/server/cat/z3950.xul	2009-05-17 08:58:05 UTC (rev 13200)
+++ trunk/Open-ILS/xul/staff_client/server/cat/z3950.xul	2009-05-18 03:53:24 UTC (rev 13201)
@@ -131,7 +131,7 @@
                         <button id="search" label="&staff.cat.z3950.search.label;" accesskey="&staff.cat.z3950.search.accesskey;"/>
                     </hbox>
                 </groupbox>
-                <splitter id="x_splitter" collapse="after" persist="state hidden"><grippy/></splitter>
+                <splitter id="x_splitter" collapse="after" persist="state hidden"><grippy id="splitter_grippy1"/></splitter>
                 <groupbox id="x_splitter2" persist="width" flex="1">
                     <caption label="&staff.cat.z3950.service_credentials.label;"/>
                     <grid flex="1">
@@ -159,7 +159,7 @@
                     </hbox>
                 </groupbox>
             </hbox>
-            <splitter id="z_splitter" collapse="before" persist="state hidden"><grippy/></splitter>
+            <splitter id="z_splitter" collapse="before" persist="state hidden"><grippy id="splitter_grippy2"/></splitter>
             <groupbox id="z_splitter2" persist="height" flex="1">
                 <caption label="&staff.cat.z3950.results_caption.label;"/>
                 <hbox>