From d52a302bc0adc669ca425c3f541352dde74321a4 Mon Sep 17 00:00:00 2001
From: Dan Stillman <dstillman@zotero.org>
Date: Sun, 5 Nov 2006 09:46:56 +0000
Subject: [PATCH] A few more tiny glitches with search date age mode

---
 chrome/content/zotero/bindings/searchtextbox.xml | 3 ++-
 chrome/content/zotero/bindings/zoterosearch.xml  | 8 ++++++--
 2 files changed, 8 insertions(+), 3 deletions(-)

diff --git a/chrome/content/zotero/bindings/searchtextbox.xml b/chrome/content/zotero/bindings/searchtextbox.xml
index bbdd3bcd3c..2bd94654ae 100644
--- a/chrome/content/zotero/bindings/searchtextbox.xml
+++ b/chrome/content/zotero/bindings/searchtextbox.xml
@@ -195,10 +195,11 @@
 						var menulist = document.getAnonymousNodes(this)[0].firstChild.nextSibling;
 						var menupopup = menulist.firstChild;
 						
+						var selectThis = 0;
 						for (var i=0; i<menupopup.childNodes.length; i++){
 							if (menupopup.childNodes[i].value == units)
 							{
-								var selectThis = i;
+								selectThis = i;
 								break;
 							}
 						}
diff --git a/chrome/content/zotero/bindings/zoterosearch.xml b/chrome/content/zotero/bindings/zoterosearch.xml
index 06fa306dc4..ee8fc61fc8 100644
--- a/chrome/content/zotero/bindings/zoterosearch.xml
+++ b/chrome/content/zotero/bindings/zoterosearch.xml
@@ -287,7 +287,6 @@
 							{
 								// If switching from menu to textbox, clear value
 								if (this.id('valuefield').hidden){
-									this.value = '';
 									this.id('valuefield').value = '';
 								}
 								// If switching between textbox conditions, get loaded value for new one
@@ -321,6 +320,10 @@
 					// Textbox + units dropdown for isInTheLast operator
 					else if (operatorsList.value=='isInTheLast')
 					{
+						// If switching from text field, clear value
+						if (this.id('value-date-age').hidden){
+							this.value = '';
+						}
 						this.id('valuefield').hidden = true;
 						this.id('valuemenu').hidden = true;
 						this.id('value-date-age').hidden = false;
@@ -329,7 +332,8 @@
 					// Textbox
 					else
 					{
-						if (!this.id('value-date-age').hidden){
+						// If switching from date age, clear value
+						if (this.id('valuefield').hidden){
 							this.value = '';
 						}
 						this.id('valuefield').hidden = false;