#2886: Implement logic for handling bogus BR nodes.
-----------------------+----------------------------------------------------
Reporter: martinkou | Owner: fredck
Type: Task | Status: new
Priority: Normal | Milestone: CKEditor 3.0
Component: General | Version: SVN
Keywords: Review- |
-----------------------+----------------------------------------------------
Changes (by fredck):
* keywords: Review? => Review-
* owner: martinkou => fredck
* status: assigned => new
Comment:
I've thoroughly though about this issue since the start of the V3 coding.
Actually, we always had problems with unneeded <br> and .
The solution here should be quite simply. It's enough to understand that
<br>, if placed at the end of a block, is used exclusively to set the
"line" height. It guarantees that a line has some content, even if
invisible and not selectable.
So, if we are sure the block already has content, we can safely discard
the <br>. Otherwise, we just leave it as is.
The proposed patch, other than taking a different approach (with ),
is also making the <br> cleanup inside the DOM, which is wrong. A
getData() call should never change the DOM. If changes are to be done,
they must go in the code that generates the output.
--
Ticket URL: <http://dev.fckeditor.net/ticket/2886#comment:3>
FCKeditor <http://www.fckeditor.net/>
The text editor for Internet
------------------------------------------------------------------------------
Create and Deploy Rich Internet Apps outside the browser with Adobe(R)AIR(TM)
software. With Adobe AIR, Ajax developers can use existing skills and code to
build responsive, highly engaging applications that combine the power of local
resources and data with the reach of the web. Download the Adobe AIR SDK and
Ajax docs to start building applications today-http://p.sf.net/sfu/adobe-com
_______________________________________________
FCKeditor-Trac mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/fckeditor-trac
