Opened 9 years ago

Last modified 8 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.

  1. Load the sample
  2. Press "Create Editor".
  3. Press "Move Editor".
  4. 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)

ajax.html (3.5 KB) - added by Freddie Bingham 9 years ago.
Sample file, modified from the sample in the download. Just overwrite the existing file of the same name and open in IE8

Download all attachments as: .zip

Change History (3)

Changed 9 years ago by Freddie Bingham

Attachment: ajax.html added

Sample file, modified from the sample in the download. Just overwrite the existing file of the same name and open in IE8

comment:1 Changed 9 years ago by Jakub Ś

Status: newconfirmed
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 8 years ago by Jakub Ś

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.

Note: See TracTickets for help on using tickets.
© 2003 – 2019 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy