Opened 11 years ago

Last modified 11 years ago

#11283 closed Bug

Error on particular content. — at Version 2

Reported by: Marcus Bointon Owned by:
Priority: Normal Milestone: CKEditor 4.3.2
Component: General Version: 4.3
Keywords: Cc:

Description (last modified by Piotrek Koszuliński)

This bug was originally reported here.

Here's a really simple, minimal example of a CKEditor page, using a fresh download of 4.3, and it works fine:

<!DOCTYPE html>
<html>
<head>
    <title></title>
</head>
<body>
<script type="text/javascript" src="/ckeditor/ckeditor.js"></script>
<textarea id="html_header" name="html_header" rows="8" cols="60">&lt;div id="header" style="background-color: purple; width: 100%; height:100px;"&gt;&lt;img alt="email header" height="63" src="http://www.example.net/content/images/header.gif" style="border: 0px;" width="600"&gt;&lt;/div&gt;</textarea>
<script type="text/javascript">
    CKEDITOR.replace('html_header', {
        "baseHref":"http://example.com/",
        "toolbarStartupExpanded":true,
        "customConfig":false
    });
</script>
</body>
</html>

Now add 'text-align: center;' to the inline style in the textarea content so it becomes this:

<!DOCTYPE html>
<html>
<head>
    <title></title>
</head>
<body>
<script type="text/javascript" src="/ckeditor/ckeditor.js"></script>
<textarea id="html_header" name="html_header" rows="8" cols="60">&lt;div id="header" style="text-align: center;background-color: purple; width: 100%; height:100px;"&gt;&lt;img alt="email header" height="63" src="http://www.example.net/content/images/header.gif" style="border: 0px;" width="600"&gt;&lt;/div&gt;</textarea>
<script type="text/javascript">
    CKEDITOR.replace('html_header', {
        "baseHref":"http://example.com/",
        "toolbarStartupExpanded":true,
        "customConfig":false
    });
</script>
</body>
</html>

CKEditor now fails to run on load, giving this error, and leaving the editor unresponsive:

[Error] TypeError: 'null' is not an object (evaluating 'a.previous=this.previous')
	replaceWith (ckeditor.js, line 249)
	upcast (ckeditor.js, line 1016)
	(anonymous function) (ckeditor.js, line 979)
	forEach (ckeditor.js, line 264)
	forEach (ckeditor.js, line 264)
	(anonymous function) (ckeditor.js, line 978)
	j (ckeditor.js, line 10)
	(anonymous function) (ckeditor.js, line 12)
	fire (ckeditor.js, line 13)
	toHtml (ckeditor.js, line 285)
	setData (ckeditor.js, line 763)
	(anonymous function) (ckeditor.js, line 327)
	j (ckeditor.js, line 10)
	(anonymous function) (ckeditor.js, line 12)
	fire (ckeditor.js, line 13)
	setData (ckeditor.js, line 239)
	b (ckeditor.js, line 759)
	(anonymous function) (ckeditor.js, line 761)
	setMode (ckeditor.js, line 313)
	(anonymous function) (ckeditor.js, line 308)
	j (ckeditor.js, line 10)
	(anonymous function) (ckeditor.js, line 12)
	fire (ckeditor.js, line 13)
	fireOnce (ckeditor.js, line 12)
	fireOnce (ckeditor.js, line 13)
	(anonymous function) (ckeditor.js, line 234)
	l (ckeditor.js, line 214)
	u (ckeditor.js, line 214)
	s (ckeditor.js, line 214)
	(anonymous function) (ckeditor.js, line 215)

If I take out the text-align style, it works again. I've no idea why it should break like that, I'm assuming it's a bug.

I attach my build-config.js file.

To reproduce

  1. Download 11283.html to samples/.
  2. Run it.

Change History (4)

Changed 11 years ago by Marcus Bointon

Attachment: build-config.js added

comment:1 Changed 11 years ago by Piotrek Koszuliński

Description: modified (diff)
Version: 4.3

Changed 11 years ago by Piotrek Koszuliński

Attachment: 11283.html added

comment:2 Changed 11 years ago by Piotrek Koszuliński

Description: modified (diff)
Milestone: CKEditor 4.3.2
Status: newconfirmed
Version: 4.3

Confirmed with 11283.html. Image widget incorrectly recognises or handles centered image.

Note: See TracTickets for help on using tickets.
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy