Ticket #1989 (closed Bug: fixed)

Opened 6 years ago

Last modified 6 years ago

xhtml source formatting works only in IE

Reported by: banexbg Owned by: martinkou
Priority: High Milestone: FCKeditor 2.6
Component: Core : Output Data Version: FCKeditor 2.6 Beta
Keywords: Confirmed Firefox Review+ Cc:

Description

In fckeditor 2.6 beta if you look at the source in firefox (on windows and mac os) and safari it is HTML not XHTML, but if you take a look at the same editor in Internet Explorer it is as it supposed to be XHTML (<br />, <img ... />...). It is not only happening on my implementation of fckeditor it is also present in your demo page, and included samples. I'm using php implementation.

Attachments

1989.patch (2.8 KB) - added by martinkou 6 years ago.

Change History

comment:1 Changed 6 years ago by fredck

  • Priority changed from Normal to High
  • Keywords Confirmed Firefox added
  • Version set to FCKeditor 2.6 Beta
  • Component changed from General to Core : Output Data
  • Milestone set to FCKeditor 2.6

Confirmed with FF.

comment:2 Changed 6 years ago by martinkou

  • Owner set to martinkou
  • Status changed from new to assigned

comment:3 Changed 6 years ago by martinkou

Both this ticket and #1981 are caused by the change to fckxhtml_gecko.js in [1192].

comment:4 Changed 6 years ago by martinkou

Also, the changes in fcktools_gecko.js contribute to the bug.

Changed 6 years ago by martinkou

comment:5 Changed 6 years ago by martinkou

  • Keywords Review? added

comment:6 Changed 6 years ago by fredck

  • Keywords Review+ added; Review? removed

Nice trick Martin! I was sure this bug was coming from that change.

This one should also fix #1955 and #1981, as you have already noted.

comment:7 Changed 6 years ago by martinkou

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

Fixed with [1691].

Click here for more info about our SVN system.

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