Ticket #479 (closed Bug: fixed)

Opened 1 year ago

Last modified 3 months ago

o:p tags cause linebreak problems in IE6/7

Reported by: neym Assigned to: martinkou
Priority: Normal Milestone: FCKeditor 2.6
Component: UI : Enter Key Version:
Keywords: Confirmed IE Review+ Cc:

Description

Occurs in IE6 and IE7. Does not occur in Firefox.

How to reproduce:

In source mode, add the content: <o:p>Some Content</o:p>

Exit source mode. Press Enter.

You will find that no linebreak occurs. When you return to the source view, you will see a </o:p><o:p> was added. This occurs even if the <o:p></o:p> is between <p></p> tags.

And further, MS Word sometimes leaves a trailing <o:p></o:p> at the end of lines when importing. This results in people asking, "Why is my enter key not making a line break?"

Attachments

479.patch (1.2 kB) - added by martinkou on 02/20/08 08:27:04.
479_2.patch (1.2 kB) - added by martinkou on 02/20/08 09:45:12.

Change History

05/30/07 13:00:51 changed by fredck

  • keywords changed from o:p to Confirmed IE.
  • milestone set to FCKeditor 2.6.

Confirmed with IE6. Ok with FF2.

With the SVN trunk, an error is thrown instead.

Of course, we recommend users to use "Paste from Word" to remove such kind of Word tags on pasting.

02/18/08 07:40:31 changed by martinkou

  • owner set to martinkou.
  • status changed from new to assigned.

02/20/08 08:27:04 changed by martinkou

  • attachment 479.patch added.

02/20/08 08:27:19 changed by martinkou

  • keywords changed from Confirmed IE to Confirmed IE Review?.

02/20/08 09:38:02 changed by fredck

  • keywords changed from Confirmed IE Review? to Confirmed IE Review-.

Even if scopeName is an IE only beast, it doesn't tell us that the scope in not present in the name. We should instead specifically restrict the check to IE, as it may happen that other browsers (typically Opera) may implement this property anytime in the future, potentially breaking our code: 

if ( FCKBrowserInfo.IsIE && e.scopeName != 'HTML' )

02/20/08 09:44:15 changed by martinkou

  • keywords changed from Confirmed IE Review- to Confirmed IE Review?.

Updated patch to add browser check.

02/20/08 09:45:12 changed by martinkou

  • attachment 479_2.patch added.

02/20/08 10:22:06 changed by fredck

  • keywords changed from Confirmed IE Review? to Confirmed IE Review+.

02/20/08 11:00:48 changed by martinkou

  • status changed from assigned to closed.
  • resolution set to fixed.

Fixed with [1569].

Click here for more info about our SVN system.