Opened 17 years ago

Closed 17 years ago

Last modified 17 years ago

#114 closed Bug (fixed)

Respect FCKConfig.ProcessHTMLEntities when FCKConfig.FillEmptyBlocks is enabled

Reported by: joelwreed@… Owned by:
Priority: Normal Milestone: FCKeditor 2.4.1
Component: General Version: FCKeditor 2.4
Keywords: Cc:

Description

In the 2.4 version of editor/_source/internals/fckxhtml.js when FCKConfig.FillEmptyBlocks is enabled, the _AppendChildNodes function will insert   in empty blocks. HOWEVER, when FCKConfig.ProcessHTMLEntities is false we should insert #160 instead.

The insertion of   will generate invalid XHTML, whereas using &#160 will generate valid XHTML.

A patch has been attached.

Attachments (1)

ticket-114.patch (623 bytes) - added by joelwreed@… 17 years ago.
patch

Download all attachments as: .zip

Change History (3)

Changed 17 years ago by joelwreed@…

Attachment: ticket-114.patch added

patch

comment:1 in reply to:  description Changed 17 years ago by Frederico Caldeira Knabben

Keywords: nbsp ProcessHTMLEntities FillEmptyBlocks removed
Resolution: fixed
Status: newclosed

Replying to joelwreed@gmail.com:

In the 2.4 version of editor/_source/internals/fckxhtml.js when FCKConfig.FillEmptyBlocks is enabled, the _AppendChildNodes function will insert   in empty blocks. HOWEVER, when FCKConfig.ProcessHTMLEntities is false we should insert #160 instead.

Your assumption is correct, as this is the way we do to preserve subsequent white spaces.

Fixed with [167].

The insertion of   will generate invalid XHTML, whereas using &#160 will generate valid XHTML.

I'm not sure about it: http://www.w3.org/TR/xhtml1/dtds.html#a_dtd_XHTML-1.0-Strict#H-A2

Wasn't you talking about XML?

comment:2 Changed 17 years ago by Frederico Caldeira Knabben

Milestone: FCKeditor 2.4FCKeditor 2.4.1
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