#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)
Change History (7)
comment:1 Changed 13 years ago by
| Owner: | set to haysmark |
|---|
comment:2 Changed 13 years ago by
comment:3 Changed 13 years ago by
| Status: | new → assigned |
|---|
Changed 13 years ago by
| Attachment: | 4797.patch added |
|---|
Fixes #4797. A disabled DateTextBox? shouldn't open.
comment:5 Changed 13 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
(In [10991]) Fixes #4797: DateTextBox?.setDisabled(true) doesnt fully disable in IE7 Patch from Mark Hays.
comment:6 Changed 10 years ago by
| Component: | Dijit → Dijit - Form |
|---|
Note: See
TracTickets for help on using
tickets.
On it.