Opened 12 years ago

Closed 8 years ago

#230 closed New Feature (wontfix)

Change the default <br> when pasting plain txt with CRLF

Reported by: Julian Warren Owned by:
Priority: Normal Milestone:
Component: General Version: FCKeditor 2.5
Keywords: HasPatch Cc:

Description

Sirs, I may seem a pest after my request for more flexible instance directory control but my bosses also asked me to change the paste text behaviour from inserting <br> for line breaks to in this case a <p> or maybe a <div>

I added this to FCKConfig.js (v2.4)

FCKConfig.EntityForCR = "<p>";

And then modified the following files:

fckeditorcode_ie.js approx line 31

FCKTools.HTMLEncode(A).replace(/\n/g,FCKConfig.EntityForCR)

fckeditorcode_gecko.js approx line 34

where it says FCKTools.HTMLEncode just after that tack this at the end A=A.blah bit

A=A.replace(/\n/g,FCKConfig.EntityForCR);return A;

Heaven knows it works! Is this a good idea? Probably not!!

Julian

Change History (3)

comment:1 Changed 12 years ago by Frederico Caldeira Knabben

Type: BugNew Feature

comment:2 Changed 11 years ago by Wojciech Olchawa

Keywords: HasPatch added

comment:3 Changed 8 years ago by Wiktor Walc

Resolution: wontfix
Status: newclosed
Version: FCKeditor 2.5

As the author stated, "Is this a good idea? Probably not!!".

This feature could exist only as an external plugin, because it makes sense only in a specific scenario and even there it will not work for some users as expected.

Note: See TracTickets for help on using tickets.
© 2003 – 2019 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy