Opened 14 years ago
Last modified 13 years ago
#7980 confirmed Bug
JS error when switching to source mode under specific conditions
Reported by: | Freddie Bingham | Owned by: | |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | General | Version: | 3.0 |
Keywords: | Cc: |
Description
Humor me with this one.
This involves a nested form and nested forms are not valid. Regardless, that does not explain why I am having this problem. You know it is possible to start with a single form, nest a form via javascript (Move Editor below), extract data via javascript, and be perfectly well with it. It only happens in IE7 and IE8, IE9 is happy.
- Load the sample
- Press "Create Editor".
- Press "Move Editor".
- Press "Source"
This recreates a scenario that I am having issues with. Here you will receive a JS error emanating from the richcombo plugin. The following event is fired on mode changes and it is the 'setValue' call below that is the cause of the problem.
editor.on( 'mode', function() { this.setState( this.modes[ editor.mode ] ? CKEDITOR.TRISTATE_OFF : CKEDITOR.TRISTATE_DISABLED ); this.setValue( '' ); }, this );
Attachments (1)
Change History (3)
Changed 14 years ago by
comment:1 Changed 14 years ago by
Status: | new → confirmed |
---|---|
Version: | → 3.0 |
Since it is invalid HTML, browser can interpret it differently. I'm not convinced if CKEditor is the one to blame.
Reproducible in IE6, IE7 and IE8.
The JS error that is thrown is:
Message: Unknown runtime error
Line: 379
Char: 4
URI: /ckeditor/_source/core/dom/element.js
comment:2 Changed 13 years ago by
I think the reason for this bug may have been described in #8590.
This may be because IE6-8 are the only browsers which don't remove the nested form tags which as @fbingha has already mentioned is invalid HTML.
Sample file, modified from the sample in the download. Just overwrite the existing file of the same name and open in IE8