Opened 8 years ago

Closed 8 years ago

#2163 closed New Feature (fixed)

Enable HTML output

Reported by: alfonsoml Owned by: alfonsoml
Priority: Normal Milestone: FCKeditor 2.6.1
Component: Core : Output Data Version: FCKeditor 2.4
Keywords: Review+ Cc:

Description

XHTML is not for everyone. If someone sets a HTML 4 doctype we must assume that he really wants HTML not XHTML.

Attachments (2)

2163.patch (943 bytes) - added by alfonsoml 8 years ago.
Proposed SVN patch
2163_2.patch (2.1 KB) - added by alfonsoml 8 years ago.
Revised patch

Download all attachments as: .zip

Change History (7)

Changed 8 years ago by alfonsoml

Proposed SVN patch

comment:1 Changed 8 years ago by alfonsoml

  • Keywords Review? added

I've set the regexp inline as /DTD HTML/ because the existing Html4DocType would match only HTML 4.0 documents in transitional mode, excluding all the 4.01 and also all the 4.0 in strict mode.

If anyone thinks it's better I can move this new regexp to FCKRegexLib.HTMLDocType for example. I just wanted to make a quick change.

comment:2 Changed 8 years ago by fredck

  • Keywords Review- added; Review? removed
  • Milestone set to FCKeditor 2.6.1

Sounds like a smart solution for it Alfonso. People will enjoy it out there.

Moving the regex to FCKRegexLib would be good, as well as adding a changelog entry for it.

Changed 8 years ago by alfonsoml

Revised patch

comment:3 Changed 8 years ago by alfonsoml

  • Keywords Review? added; Review- removed

comment:4 Changed 8 years ago by fredck

  • Keywords Review+ added; Review? removed

comment:5 Changed 8 years ago by alfonsoml

  • Resolution set to fixed
  • Status changed from new to closed

Fixed with [2022]

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