#2886: Implement logic for handling bogus BR nodes.
-------------------------------+--------------------------------------------
Reporter: martinkou | Owner: martinkou
Type: Task | Status: new
Priority: Normal | Milestone: CKEditor 3.0
Component: General | Version: SVN (FCKeditor)
Keywords: IBM 3.0RC Review? |
-------------------------------+--------------------------------------------
Changes (by martinkou):
* keywords: IBM 3.0RC Review- => IBM 3.0RC Review?
Comment:
Your test case is actually correct.
The raw HTML input is usually meant for HTML code already existing outside
of the editor. e.g. A CKEditor plugin in Drupal. So the
{{{
<p>text<br /></p>
}}}
code you have will have come from a Drupal page being edited.
And in this case, the <br /> makes no difference. Because outside of
browser editing mode, a <br /> at the tail of a block is never visible. No
matter if it's IE or Firefox. So it is correct for the editor to trim it
to
{{{
<p>text</p>
}}}
and display that in WYSIWYG mode instead.
There's a small bug in the previous patch about this case, actually.
Previously, the <br> trimming logic would be executed for non-IE browsers
only. But in this case, the <br /> should be trimmed even for IE. So in my
new patch, the trimming logic is run like this:
1. If we're converting from WYSIWYG mode data to output/Source mode data,
then the <br> would be trimmed only for non-IE browsers. The reason is
that tail <br> nodes are visible in IE editing mode.
2. If we're converting from raw HTML data to WYSIWYG data, then the <br>
would always be trimmed for any browser. The reason is that the tail <br>
is always invisible outside of editing mode in any browser.
--
Ticket URL: <http://dev.fckeditor.net/ticket/2886#comment:17>
FCKeditor <http://www.fckeditor.net/>
The text editor for Internet
------------------------------------------------------------------------------
Crystal Reports - New Free Runtime and 30 Day Trial
Check out the new simplified licensing option that enables unlimited
royalty-free distribution of the report engine for externally facing
server and web deployment.
http://p.sf.net/sfu/businessobjects
_______________________________________________
FCKeditor-Trac mailing list
FCKeditor-Trac@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/fckeditor-trac
