Ticket #2390 (closed Bug: fixed)

Opened 5 months ago

Last modified 4 months ago

[IE7] Indentation doesn't work on <p> tag the first time it's clicked, but works the second time.

Reported by: Adam.Skevy Owned by: fredck
Priority: High Milestone: FCKeditor 2.6.3
Component: General Version: FCKeditor 2.6.2
Keywords: Confirmed IE Review+ Cc:

Description

This bug happens both on my personal site as well as the demo FCKEditor at http://www.fckeditor.net/demo, so I know it's a editor-wide issue. Also, it only happens on IE7.

When you type any amount of text (longer than one line, and consisting of more than one paragraph tag), try to indent any part of the text, and then either view the source or submit the form that FCKEditor is part of, it removes the indenting that you performed. If you do the same thing as above, but do not submit the form, and simply remove the indentation and re-indent the text, it performs as expected.

Example and How to Reproduce:

Type the following text into an empty FCKEditor window in Internet Explorer 7 (a good editor to test on is the demo editor, because it is a default installation):

Test Line 1 [PRESS ENTER]
Test Line 2 [PRESS ENTER]
Test Line 3 [PRESS ENTER]
Test Line 4 [PRESS ENTER]

Now indent lines 2 and 3. View the source. Notice that there is no "style='margin-left:40px'" as an attribute of any of the <p> tags.

Attachments

2390.patch (1.5 KB) - added by martinkou 4 months ago.
2390_2.patch (2.3 KB) - added by fredck 4 months ago.

Change History

Changed 4 months ago by martinkou

  • keywords Confirmed IE added; indent indentation p margin removed
  • owner set to martinkou
  • status changed from new to assigned

Changed 4 months ago by martinkou

Changed 4 months ago by martinkou

  • keywords Review? added

Changed 4 months ago by fredck

  • keywords Review- added; Review? removed
  • owner changed from martinkou to fredck
  • status changed from assigned to new

This issue is not limited to <p> elements. It happens with any kind of element (just try EnterMode='div').

I've worked on a new patch for it.

Changed 4 months ago by fredck

Changed 4 months ago by fredck

  • keywords Review? added; Review- removed
  • status changed from new to assigned

Changed 4 months ago by martinkou

  • keywords Review+ added; Review? removed

Changed 4 months ago by fredck

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

Fixed with [2280].

Note: See TracTickets for help on using tickets.