#12254 closed defect (fixed)
IE9: dijit._editor.RichText: error in IE-specific code branch
Reported by: | Kenneth G. Franqueiro | Owned by: | |
---|---|---|---|
Priority: | high | Milestone: | 1.5.2 |
Component: | Editor | Version: | 1.6.0b1 |
Keywords: | ie9 editor | Cc: | |
Blocked By: | Blocking: |
Description (last modified by )
This is observable in themeTester. As soon as it loads, two errors occur with the message "SCRIPT5022: DOM Exception: INVALID_CHARACTER_ERR (5)". Both are traceable to dijit._editor.RichText
line 746:
var tabStop = (this.tabStop = dojo.doc.createElement('<div tabIndex=-1>'));
This code is apparently relevant to #4996, and despite the fact that the string being passed to createElement doesn't seem remotely valid, it doesn't cause problems in any versions of IE previous to 9. I'm not sure whether limiting this code path to IE < 9 would introduce a regression of #4996 in IE9, nor whether revising the code to createElement('div')
(which fixes the error) and setting the tabIndex separately would cause regressions in general.
Attachments (1)
Change History (7)
comment:1 Changed 10 years ago by
Description: | modified (diff) |
---|
comment:2 Changed 10 years ago by
Description: | modified (diff) |
---|
comment:3 Changed 10 years ago by
Changed 10 years ago by
Attachment: | 12254.diff added |
---|
Patch to use dojo.create instead of nonstandard usage of createElement; tested in IE6, 8, 9RC.
comment:4 Changed 10 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
comment:5 Changed 10 years ago by
Milestone: | tbd → 1.6 |
---|
bulk update: bugs fixed in past few months, presumably milestone is 1.6
comment:9 Changed 9 years ago by
Milestone: | 1.6 → 1.5.2 |
---|
Although the [12842] commit didn't include test cases, they were later added to Editor_a11y.html, the ""tabbing"" section of that test file. So it should be easy to test that tabbing is working after changing the condition from
dojo.isIE
todojo.isIE < 9
, or evendojo.isIE < 8
.