Opened 12 years ago

Closed 12 years ago

#5968 closed defect (fixed)

Editor: focusOnLoad attribute does not work on IE

Reported by: Douglas Hays Owned by: Douglas Hays
Priority: high Milestone: 1.1
Component: Editor Version: 1.1b1
Keywords: Cc: alex
Blocked By: Blocking:

Description (last modified by Douglas Hays)

Editor is supposed to focus the text when focusOnLoad=true. This works on FF2 and Safari but not IE. To recreate, run dijit/test_Editor.html in IE and notice that the new automated test (first editor) does not have any checkboxes checked. Click the Automated Text text (ie. manually focus) and the checkboxes fill in. Tested and failed on both IE 6 and 7.

Attachments (1)

5968_10x.patch (2.4 KB) - added by Douglas Hays 12 years ago.
Patch for 1.0.x.

Download all attachments as: .zip

Change History (8)

comment:1 Changed 12 years ago by Douglas Hays

Description: modified (diff)
Summary: Editor: focusOnLoad attribute does not work on IE6Editor: focusOnLoad attribute does not work on IE

comment:2 Changed 12 years ago by bill

Milestone: 1.11.3
Priority: highnormal
severity: majorminor

Not sure why we even have this parameter. Just checked with Doug; he says that Editor.focus() does work (when done later, after the editor is instantiated), so it's just an issue w/this param.

comment:3 Changed 12 years ago by Douglas Hays

Milestone: 1.31.1

comment:4 Changed 12 years ago by Douglas Hays

Resolution: fixed
Status: newclosed

(In [12747]) Fixes #5968. On IE, the focus was being set to each Editor as it was rendered which was trumping the focusOnLoad. I made each editor temporarily unselectable in IE which prevented incorrect focusing. This also fixes the onChange problem in IE were the first onChange did not fire because the automatic focus was not detected by the widget.

comment:5 Changed 12 years ago by Douglas Hays

Resolution: fixed
Status: closedreopened

Needs to be fixed for 1.0.x as well.

comment:6 Changed 12 years ago by Douglas Hays

Owner: changed from liucougar to Douglas Hays
Status: reopenednew

comment:7 Changed 12 years ago by Douglas Hays

Resolution: fixed
Status: newclosed

(In [12813]) Fixes #5968 for 1.0.x !strict. See [12747] for details.

Changed 12 years ago by Douglas Hays

Attachment: 5968_10x.patch added

Patch for 1.0.x.

Note: See TracTickets for help on using tickets.