Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#11625 closed defect (fixed)

DateTextBox incorrectly sets today's date after applying [22665]

Reported by: Douglas Hays Owned by: bill
Priority: high Milestone: 1.6
Component: Dijit - Form Version: 1.5
Keywords: Cc: ben hockey
Blocked By: Blocking:

Description (last modified by Douglas Hays)

Starting with [22665], DateTextBox? is sometimes setting the current date when the value should be blank.
Load dijit/tests/test_Dialog.html
Click the "Show Dialog" button.
Click the center of the date textbox in the popup dialog.
Click the whitespace to the right and today's date shows up in the textbox when it should be blank.

A separate issue is that if using IE6 to test, hover the mouse over 1 of the other textboxes and the value disappears, and then reappears when hovering over the date textbox.

Change History (5)

comment:1 Changed 11 years ago by bill

Description: modified (diff)

Thanks, I'll fix this, dropDownDefaultValue needs to be change to control currentFocus, rather than value. IOW, when the DateTextBox is blank and the Calendar drop down is opened it shouldn't show any date as selected.

comment:2 Changed 11 years ago by Douglas Hays

Description: modified (diff)

This appears to be fixed by [22763].

comment:3 Changed 11 years ago by bill

Resolution: fixed
Status: newclosed

Hmm, so it is. Anyway I'll still checkin the change I listed above, as I think it makes things better.

comment:4 Changed 11 years ago by bill

(In [22772]) Modifications to DateTimeTextBox and Calendar so that dropDownDefaultValue controls the Calendar's currentFocus, rather than it's value. Calendar by default now has no selected date, although focus (by default) goes to today's date.

All of this code is complicated by invalid dates sometimes being represented as null, and sometimes as Date(""), the NaN of dates. Hopefully that will be simplified in 2.0.

Refs #11625, #10822 ([22665]), !strict.

comment:5 Changed 11 years ago by bill

Component: DijitDijit - Form
Note: See TracTickets for help on using tickets.