Opened 12 years ago

Closed 12 years ago

Last modified 10 years ago

#3784 closed defect (fixed)

FF: Empty validating text box; Esc doesn't work the first time

Reported by: ptbrunet Owned by: Douglas Hays
Priority: high Milestone: 0.9
Component: Dijit Version: 0.9
Keywords: Cc:
Blocked By: Blocking:

Description

Using .../dijit/tests/form/test_validate.html tab down to the edit box labeled "Trac ticket 1651:", enter a number such as 1111, press Esc, the field doesn't revert to it's prior empty non-value. Tab away and back again, enter 1112, press Esc, the number reverts back to 1111. This is on FF.

Change History (5)

comment:1 Changed 12 years ago by bill

Owner: changed from bill to haysmark

comment:2 Changed 12 years ago by haysmark

See #1651.

That last textbox in test_validate is very special because its value is null. Consequently, its "last value" is also null. Clearly the undefined test in onKeyPress in _FormWidget will prevent the escape key from working in this specific instance.

Do we need the undefined test in _FormWidget? I don't know if it's safe to remove it.

comment:3 Changed 12 years ago by haysmark

Owner: changed from haysmark to bill

So we could do a null test in TextBox? postMixin, but then we would have to add additional code to DateTextBox? to handle the "". Think on it and reassign to me when you decide how/if TextBox? should handle null values.

comment:4 Changed 12 years ago by Douglas Hays

Owner: changed from bill to Douglas Hays

comment:5 Changed 12 years ago by Douglas Hays

Resolution: fixed
Status: newclosed

(In [9945]) Fixes #3784. If null, set to ""

Note: See TracTickets for help on using tickets.