HR in input "source" completely destroys document structure

[amashtakov@…]

Hi all,

This can be easyly re-produced in current demo page (2.3.1). Please try the following:

1. Create three <p>'s with bold title an text in editor view:

<p><strong>title1<br />
</strong>text1 text1</p>
<p><strong>title2</strong><br />
text2 text2</p>
<p><strong>title3<br />
</strong>text3 text3</p>
<p>&nbsp;</p>

2. Then put <hr>'s at the beginning of title2 and title3 in editor view:

<p><strong>title1<br />
</strong>text1 text1</p>
<p><strong><hr />
title2</strong><br />
text2 text2</p>
<p><strong><hr />
title3<br />
</strong>text3 text3</p>
<p>&nbsp;</p>

3. Then switch twice between source and editor view. As a result - the structure of document is completely destroyed:

<p><strong>title1<br />
</strong>text1 text1</p>
<p><strong></strong><br />
text2 text2
<p>&nbsp;</p>
<p><strong></strong>text3 text3
<p>&nbsp;</p>
<p>&nbsp;</p>
</p>
<hr />
title3<br />
</p>
<hr />
title2

Regards, /Alexander

---

Moved from SF:
​http://sourceforge.net/tracker/index.php?func=detail&aid=1552552&group_id=75348&atid=543653

[Frederico Caldeira Knabben]

missed browser info - IE6.0

---

Moved from SF. Original poster: amashtakov

[Frederico Caldeira Knabben]

I changed the FCKConfig.UseBROnCarriageReturn to true. It says that attribute is IE only but see it working on FF. Recently, when i discovered this problem and started playing with configuration settings, i tried this issue on the demo site (2.3.2 in this case) and as you say, it failed. But now, the demo site has obviously made this same change, because no more <p></p> are present in new content using your example. the CR is <BR> and not <P> even in the demo. It has solved the issue for me. My clients were reporting things "hopping around" after saving content.

See what it does for you.

Eric

---

Moved from SF. Original poster: geotronik

[Frederico Caldeira Knabben]

Confirmed with IE6. Firefox is OK.

The <hr> is a block tag, which inside another block tags (<p>) makes IE go crazy.

[Frederico Caldeira Knabben]

This is a DUP of #202.

[Dale]

I am still seeing nearly identical behavior in IE 7.0 with build 2.4.1. Following steps 1 and 2 above results in the following code:

<p><strong>title1<br />
</strong>text1 text1</p>
<p><br />
text2 text2</p>
<p>&nbsp;</p>
<p>text3 text3</p>
<p>&nbsp;</p>
<p>&nbsp;</p>
<p>&nbsp;</p>
<hr />
<p>title3</p>
<p>&nbsp;</p>
<hr />
<p>title2</p>

I saw that this was said to be fixed in 2.4 but it does not appear to be.

[Frederico Caldeira Knabben]

It was an unfortunate regression brought with [219]. It regressed [202] too, so this one is still a DUP of it.

I've fixed it with [321].

[Dale]

I do not wish to be a pain about this, but I am still seeing the same behavior. I copied your new code over the old in fck.js and tested it, but still get the same results.

If I paste this code in the source view of the editor (using IE 7.0):

<p><strong>title1<br />
</strong>text1 text1</p>
<p><strong><hr />
title2</strong><br />
text2 text2</p>
<p><strong><hr />
title3<br />
</strong>text3 text3</p>
<p>&nbsp;</p>

And then switch to the main view I see the text as it should be.

But then if I switch back to source mode the text code gets jumbled to look like this:

<p><strong>title1<br />
</strong>text1 text1</p>
<p><br />
text2 text2
<p>&nbsp;</p>
<p>text3 text3
<p>&nbsp;</p>
<p>&nbsp;</p>
</p>
<hr />
title3<br />
</p>
<hr />
title2

[Alfonso Martínez de Lizarrondo]

I've tested the nightly build and it works fine for me in IE6 and IE7

[Frederico Caldeira Knabben]

It works for me too with the SVN version. Please download the entire SVN version for testings.

[Dale]

Replying to fredck:

It works for me too with the SVN version. Please download the entire SVN version for testings.

Ok. I downloaded and installed the entire SVN version and it worked. Thanks for the quick response on this.