Ticket #479 (closed Bug: fixed)

Opened 3 years ago

Last modified 2 years ago

o:p tags cause linebreak problems in IE6/7

Reported by: neym Owned by: 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 Download (1.2 KB) - added by martinkou 2 years ago.
479_2.patch Download (1.2 KB) - added by martinkou 2 years ago.

Change History

Changed 3 years ago by fredck

  • keywords Confirmed IE added; o:p removed
  • 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.

Changed 2 years ago by martinkou

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

Changed 2 years ago by martinkou

Changed 2 years ago by martinkou

  • keywords Review? added

Changed 2 years ago by fredck

  • keywords Review- added; Review? removed

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' ) 

Changed 2 years ago by martinkou

  • keywords Review? added; Review- removed

Updated patch to add browser check.

Changed 2 years ago by martinkou

Changed 2 years ago by fredck

  • keywords Review+ added; Review? removed

Changed 2 years ago by martinkou

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

Fixed with [1569].

Click here for more info about our SVN system.

Note: See TracTickets for help on using tickets.