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 )
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)
Change History (8)
comment:1 Changed 12 years ago by
| Description: | modified (diff) |
|---|---|
| Summary: | Editor: focusOnLoad attribute does not work on IE6 → Editor: focusOnLoad attribute does not work on IE |
comment:2 Changed 12 years ago by
| Milestone: | 1.1 → 1.3 |
|---|---|
| Priority: | high → normal |
| severity: | major → minor |
comment:3 Changed 12 years ago by
| Milestone: | 1.3 → 1.1 |
|---|
comment:4 Changed 12 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
(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
| Resolution: | fixed |
|---|---|
| Status: | closed → reopened |
Needs to be fixed for 1.0.x as well.
comment:6 Changed 12 years ago by
| Owner: | changed from liucougar to Douglas Hays |
|---|---|
| Status: | reopened → new |
comment:7 Changed 12 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
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.