From 10a10725b9a8e3dc7bc7d9091e4cfddec866da90 Mon Sep 17 00:00:00 2001 From: "jminta%gmail.com" Date: Thu, 10 Aug 2006 00:43:29 +0000 Subject: [PATCH] Bug 346318 Datepicker appears only briefly when clicking on date dropdown in event dialog r=dmose --- .../datetimepickers/datetimepickers.xml | 33 ++++++++++++------- 1 file changed, 22 insertions(+), 11 deletions(-) diff --git a/calendar/resources/content/datetimepickers/datetimepickers.xml b/calendar/resources/content/datetimepickers/datetimepickers.xml index c8c00b61e6b..298974130e7 100644 --- a/calendar/resources/content/datetimepickers/datetimepickers.xml +++ b/calendar/resources/content/datetimepickers/datetimepickers.xml @@ -95,7 +95,9 @@ onkeypress="if (event.keyCode == 13) this.kDatePicker.parseTextBoxDate(true);" xbl:inherits="disabled"> + anonid="datepopup" + onpopupshowing="onPopup()" + onpopuphiding="reshowPopup()"> @@ -120,12 +122,12 @@ this.value = new Date(); } this.kMinimonth.addEventListener("select", this.clickDate, false); - this.kMinimonth.addEventListener("monthchange", this.reshowPopup, false); - this.kMinimonth.addEventListener("popuplisthidden", this.reshowPopup, false); - this.mIsReshowing = false; ]]> + + false + @@ -167,7 +169,11 @@ this.mInPopup = false; // select all to remove cursor since can't type while popped-up this.select(); + // From this point on, until the user does something we + // care about, we shouldn't close. + this.mReallyClose = false; } + this.mIsReshowing = false; ]]> @@ -178,14 +184,18 @@ @@ -220,6 +230,7 @@ datepicker.update(new Date(aEvent.target.value), true); // select changed value so no cursor appears (can't type to it). datepicker.select(); + this.mReallyClose = true; aEvent.target.parentNode.hidePopup(); } ]]>