Opened 12 years ago

Closed 12 years ago

Last modified 9 years ago

#4797 closed defect (fixed)

DateTextBox.setDisabled(true) doesnt fully disable in IE7

Reported by: tk Owned by: haysmark
Priority: high Milestone: 1.0
Component: Dijit - Form Version: 0.9
Keywords: DateTextBox TimeTextBox setDisabled Cc: Adam Peller, bill, Douglas Hays, dante
Blocked By: Blocking:

Description

If you load the test page and do the following:

var a = dijit.byId('q14'); first widget on page a.setDisabled(true);

then click around on the input (setting a value before disabling gives you a better idea of this bug) outside the area where the date value is displayed, you will get the popup to continue to fire and even change the date while disabled.

Recommended fix: (not at home to test or produce patches)

_open() function in TimeTextBox?.js needs a check for this.disabled to prevent this from happening

CC'ing a few of the Dijit crew incase this can be fixed today.

Attachments (1)

4797.patch (369 bytes) - added by haysmark 12 years ago.
Fixes #4797. A disabled DateTextBox? shouldn't open.

Download all attachments as: .zip

Change History (7)

comment:1 Changed 12 years ago by bill

Owner: set to haysmark

comment:2 Changed 12 years ago by haysmark

On it.

comment:3 Changed 12 years ago by haysmark

Status: newassigned

Changed 12 years ago by haysmark

Attachment: 4797.patch added

Fixes #4797. A disabled DateTextBox? shouldn't open.

comment:4 Changed 12 years ago by haysmark

Good to go.

comment:5 Changed 12 years ago by bill

Resolution: fixed
Status: assignedclosed

(In [10991]) Fixes #4797: DateTextBox?.setDisabled(true) doesnt fully disable in IE7 Patch from Mark Hays.

comment:6 Changed 9 years ago by bill

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