Opened 11 years ago

Closed 11 years ago

Last modified 8 years ago

#5701 closed defect (fixed)

DateTextBox: disabled check fails in Safari 3.0

Reported by: guest Owned by: Adam Peller
Priority: high Milestone: 1.1
Component: Dijit - Form Version: 1.0
Keywords: Cc: sebastian.schaffert@…, Douglas Hays
Blocked By: Blocking:

Description

When creating a DateTextBox?, the calendar picker does not work properly in Safari 3.0 on MacOS X 10.5. All dates are stroken and none of them can be selected (maybe a problem with constraints on the values, like min or max?). The same calendar picker works fine in Firefox 2.0.

Attachments (1)

SafariSchnappschuss001.png (34.0 KB) - added by guest 11 years ago.
Screenshot illustrating how the calendar picker disables all entries when min/max are not given in Safari

Download all attachments as: .zip

Change History (8)

comment:1 Changed 11 years ago by Adam Peller

Owner: set to Adam Peller

does this happen on the provided tests? which ones? If not, can you attach an example?

comment:2 Changed 11 years ago by guest

The problem happens e.g. with dijit/tests/form/test_DateTextBox.html in the first test field (Date (local format) ). It seems to be related to whether a min/max constraint is given (works) or is not given (does not work). Screenshot attached.

Changed 11 years ago by guest

Attachment: SafariSchnappschuss001.png added

Screenshot illustrating how the calendar picker disables all entries when min/max are not given in Safari

comment:3 Changed 11 years ago by Adam Peller

Resolution: fixed
Status: newclosed

(In [12185]) dojo.date.compare with undefined produces unpredictable results, so check inputs first. Fixes #5701 !strict

comment:4 Changed 11 years ago by Adam Peller

Cc: Douglas Hays added
Summary: DateTextBox: picker doesn't work in Safari / MacOS X 10.5DateTextBox: disabled check fails in Safari 3.0

the unpredictable part comes from what happens in compare when the input (date1) is converted to a Number and a Date. Date(Number(undefined)) produces an 'invalid date' in Mozilla, but an empty object in Safari. The compare then goes on to produce '0' and '-1' respectively, for the same inputs.

comment:5 Changed 9 years ago by Adam Peller

(In [21369]) doughays noticed that the parens were grouped incorrectly in r12185. Refs #5107. !strict

comment:6 Changed 9 years ago by Adam Peller

Browser behavior may have changed. Date(NaN) produces now, and Date() is only invalid on Mozilla, not Safari 4.0

comment:7 Changed 8 years ago by bill

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