From b92b5a6c569345607b2dd81fa9feaec5f8a42ab5 Mon Sep 17 00:00:00 2001
From: phasefx <phasefx@dcc99617-32d9-48b4-a31d-7c20da2025e4>
Date: Mon, 11 Jul 2005 08:23:16 +0000
Subject: [PATCH] refactor and patron selection feedback

git-svn-id: svn://svn.open-ils.org/ILS/trunk@1122 dcc99617-32d9-48b4-a31d-7c20da2025e4
---
 .../content/evergreen/patron/patron_search_form.js |   2 +
 .../evergreen/patron/patron_search_form.xul        |   5 +-
 .../patron/patron_search_form_overlay.xul          | 140 +++++++++++----------
 3 files changed, 78 insertions(+), 69 deletions(-)

diff --git a/Evergreen/staff_client/chrome/content/evergreen/patron/patron_search_form.js b/Evergreen/staff_client/chrome/content/evergreen/patron/patron_search_form.js
index e46c160c01..e52dfd846a 100644
--- a/Evergreen/staff_client/chrome/content/evergreen/patron/patron_search_form.js
+++ b/Evergreen/staff_client/chrome/content/evergreen/patron/patron_search_form.js
@@ -15,6 +15,8 @@ function patron_search_form_init(p) {
 					p.w.crazy_search_hash); },
 			false);
 
+	p.w.selection_canvas = get_widget( p.w.document, p.selection_canvas );
+
 	var search_command = p.w.document.getElementById('cmd_search');
 	var clear_command = p.w.document.getElementById('cmd_clear');
 
diff --git a/Evergreen/staff_client/chrome/content/evergreen/patron/patron_search_form.xul b/Evergreen/staff_client/chrome/content/evergreen/patron/patron_search_form.xul
index 6316e7a6e7..b76c6f3db2 100644
--- a/Evergreen/staff_client/chrome/content/evergreen/patron/patron_search_form.xul
+++ b/Evergreen/staff_client/chrome/content/evergreen/patron/patron_search_form.xul
@@ -7,7 +7,10 @@
 <page id="patron_search_form_win"
 	orient="vertical" style="overflow: auto" width="800" height="500"
 	sizemode="maximized" persist="width height" 
-	onload="params.w = window; params.app = 'PatronSearchForm'; mw.OpenILS_init(params);"
+	onload="params.w = window; 
+		params.app = 'PatronSearchForm'; 
+		params.selection_canvas = 'PatronSearchForm_selection';
+		mw.OpenILS_init(params);"
 	onunload="mw.OpenILS_exit(params);"
 	xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul">
 
diff --git a/Evergreen/staff_client/chrome/content/evergreen/patron/patron_search_form_overlay.xul b/Evergreen/staff_client/chrome/content/evergreen/patron/patron_search_form_overlay.xul
index ec2c3051f7..1b3b29be21 100644
--- a/Evergreen/staff_client/chrome/content/evergreen/patron/patron_search_form_overlay.xul
+++ b/Evergreen/staff_client/chrome/content/evergreen/patron/patron_search_form_overlay.xul
@@ -16,80 +16,84 @@
 </commandset>
 
 <!-- main widget for patron_search_form -->
-<box id="PatronSearchForm_main" flex="0" orient="horizontal">
+<box id="PatronSearchForm_main" flex="1" orient="vertical">
 
-	<!-- group 0 = user  group 1 = address  group 2 = phone, ident -->
-
-	<grid id="psg">
-		<columns id="psc">
-			<column id="psc1"/>
-			<column id="psc2" flex="1"/>
-		</columns>
-		<rows id="psr">
-			<row id="psr1">
-				<label id="psl1" control="pst1" value="&patron_search_form.family_name.label;" accesskey="&patron_search_form.family_name.accesskey;"/>
-				<textbox id="pst1" field="family_name" group="0"/>
-			</row>
-			<row id="psr2">
-				<label id="psl2" control="pst2" value="&patron_search_form.first_given_name.label;" accesskey="&patron_search_form.first_given_name.accesskey;"/>
-				<textbox id="pst2" field="first_given_name" group="0"/>
-			</row>
-			<row id="psr3">
-				<label id="psl3" control="pst3" value="&patron_search_form.second_given_name.label;" accesskey="&patron_search_form.second_given_name.accesskey;"/>
-				<textbox id="pst3" field="second_given_name" group="0"/>
-			</row>
-			<row id="psr4">
-				<label id="psl4" control="pst4" value="&patron_search_form.email.label;" accesskey="&patron_search_form.email.accesskey;"/>
-				<textbox id="pst4" field="email" group="0"/>
-			</row>
-			<row id="psr5">
-				<label id="psl5" control="pst5" value="&patron_search_form.phone.label;" accesskey="&patron_search_form.phone.accesskey;"/>
-				<textbox id="pst5" field="phone" group="2"/>
-			</row>
-			<row id="psr6">
-				<label id="psl6" control="pst6" value="&patron_search_form.ident.label;" accesskey="&patron_search_form.ident.accesskey;"/>
-				<textbox id="pst6" field="ident" group="2"/>
-			</row>
-			<row id="psr6a">
-				<label id="psl6a" value=" "/>
-			</row>
-			<row id="psr7">
-				<label id="psl7" control="pst7" value="&patron_search_form.street1.label;" accesskey="&patron_search_form.street1.accesskey;"/>
-				<textbox id="pst7" field="street1" group="1"/>
-			</row>
-			<row id="psr8">
-				<label id="psl8" control="pst8" value="&patron_search_form.street2.label;" accesskey="&patron_search_form.street2.accesskey;"/>
-				<textbox id="pst8" field="street2" group="1"/>
-			</row>
-			<row id="psr9">
-				<label id="psl9" control="pst9" value="&patron_search_form.city.label;" accesskey="&patron_search_form.city.accesskey;"/>
-				<textbox id="pst9" field="city" group="1"/>
-			</row>
-			<row id="psr10">
-				<label id="psl10" control="pst10" value="&patron_search_form.state.label;" accesskey="&patron_search_form.state.accesskey;"/>
-				<textbox id="pst10" field="state" group="1"/>
-			</row>
-			<row id="psr11">
-				<label id="psl11" control="pst11" value="&patron_search_form.post_code.label;" accesskey="&patron_search_form.post_code.accesskey;"/>
-				<textbox id="pst11" field="post_code" group="1"/>
-			</row>
-			<row id="psr12">
-				<spacer id="pss12"/>
-				<button id="psb12" label="&patron_search_form.search.label;" accesskey="&patron_search_form.search.accesskey;" key="ps_search" command="cmd_search"/>
-			</row>
-			<row id="psr13">
-				<spacer id="pss13"/>
-				<button id="psb13" label="&patron_search_form.clear.label;" accesskey="&patron_search_form.clear.accesskey;" key="ps_clear" command="cmd_clear"/>
-			</row>
-		</rows>
-	</grid>
+	<groupbox id="psgf_gb">
+		<caption id="psgf_gbc" label='&patron_search_form.caption;' />
+		<grid id="psg" />
+	</groupbox>
+	<box id="PatronSearchForm_selection" orient="vertical" flex="1" />
 
 </box>
 
+<!-- group 0 = user  group 1 = address  group 2 = phone, ident -->
+
+<grid id="psg">
+	<columns id="psc">
+		<column id="psc1"/>
+		<column id="psc2" flex="1"/>
+	</columns>
+	<rows id="psr">
+		<row id="psr1">
+			<label id="psl1" control="pst1" value="&patron_search_form.family_name.label;" accesskey="&patron_search_form.family_name.accesskey;"/>
+			<textbox id="pst1" field="family_name" group="0"/>
+		</row>
+		<row id="psr2">
+			<label id="psl2" control="pst2" value="&patron_search_form.first_given_name.label;" accesskey="&patron_search_form.first_given_name.accesskey;"/>
+			<textbox id="pst2" field="first_given_name" group="0"/>
+		</row>
+		<row id="psr3">
+			<label id="psl3" control="pst3" value="&patron_search_form.second_given_name.label;" accesskey="&patron_search_form.second_given_name.accesskey;"/>
+			<textbox id="pst3" field="second_given_name" group="0"/>
+		</row>
+		<row id="psr4">
+			<label id="psl4" control="pst4" value="&patron_search_form.email.label;" accesskey="&patron_search_form.email.accesskey;"/>
+			<textbox id="pst4" field="email" group="0"/>
+		</row>
+		<row id="psr5">
+			<label id="psl5" control="pst5" value="&patron_search_form.phone.label;" accesskey="&patron_search_form.phone.accesskey;"/>
+			<textbox id="pst5" field="phone" group="2"/>
+		</row>
+		<row id="psr6">
+			<label id="psl6" control="pst6" value="&patron_search_form.ident.label;" accesskey="&patron_search_form.ident.accesskey;"/>
+			<textbox id="pst6" field="ident" group="2"/>
+		</row>
+		<row id="psr6a">
+			<label id="psl6a" value=" "/>
+		</row>
+		<row id="psr7">
+			<label id="psl7" control="pst7" value="&patron_search_form.street1.label;" accesskey="&patron_search_form.street1.accesskey;"/>
+			<textbox id="pst7" field="street1" group="1"/>
+		</row>
+		<row id="psr8">
+			<label id="psl8" control="pst8" value="&patron_search_form.street2.label;" accesskey="&patron_search_form.street2.accesskey;"/>
+			<textbox id="pst8" field="street2" group="1"/>
+		</row>
+		<row id="psr9">
+			<label id="psl9" control="pst9" value="&patron_search_form.city.label;" accesskey="&patron_search_form.city.accesskey;"/>
+			<textbox id="pst9" field="city" group="1"/>
+		</row>
+		<row id="psr10">
+			<label id="psl10" control="pst10" value="&patron_search_form.state.label;" accesskey="&patron_search_form.state.accesskey;"/>
+			<textbox id="pst10" field="state" group="1"/>
+		</row>
+		<row id="psr11">
+			<label id="psl11" control="pst11" value="&patron_search_form.post_code.label;" accesskey="&patron_search_form.post_code.accesskey;"/>
+			<textbox id="pst11" field="post_code" group="1"/>
+		</row>
+		<row id="psr12">
+			<spacer id="pss12"/>
+			<button id="psb12" label="&patron_search_form.search.label;" accesskey="&patron_search_form.search.accesskey;" command="cmd_search"/>
+		</row>
+		<row id="psr13">
+			<spacer id="pss13"/>
+			<button id="psb13" label="&patron_search_form.clear.label;" accesskey="&patron_search_form.clear.accesskey;" command="cmd_clear"/>
+		</row>
+	</rows>
+</grid>
+
 <!-- Accelerator Keys (Accessor Keys are in DTD's) -->
 <keyset id="PatronSearchForm_keys">
-	<key id="ps_search" modifiers="" keycode="VK_RETURN" command="cmd_search"/>
-	<key id="ps_clear" modifiers="accel" keycode="VK_DELETE" command="cmd_clear"/>
 </keyset>
 
 </overlay>
-- 
2.11.0