Opened 8 years ago

Closed 8 years ago

#1997 closed Bug (fixed)

[IE] When you specify a title to a table, the first character is deleted

Reported by: estan Owned by: fredck
Priority: Normal Milestone: FCKeditor 2.6
Component: General Version: FCKeditor 2.6 Beta
Keywords: Confirmed IE Review+ Cc:

Description

Like the title suggests.

To reproduce, open the editor, click the button to add a table, fill for example the text 'hello' in the title field. click on OK, now you can see the table in the editor with on top the title in bold 'ello'

Attachments (2)

1997.patch (5.3 KB) - added by fredck 8 years ago.
1997_2.patch (5.3 KB) - added by fredck 8 years ago.

Download all attachments as: .zip

Change History (12)

comment:1 Changed 8 years ago by estan

I like to add that it only happens with IE(7) not with Firefox. You can even see the error in your demo http://www.fckeditor.net/demo

debugging with VS2008 gives me the problem in fckeditorcode_ie.js

After the ok click at the end of that function you have the following line: oEditor.FCK.InsertElement( table ) ; When i ask the quick watch of the table object, you can see clearly that the caption is correct. So, it must be happening when the object is converted to html code.

Good luck

comment:2 Changed 8 years ago by w.olchawa

  • Keywords Confirmed IE added; table title removed
  • Milestone FCKeditor 2.6 deleted
  • Summary changed from When you specify a title to a table, the first character is deleted to [IE] When you specify a title to a table, the first character is deleted

Confirmed both in IE6 and IE7 using 2.6 beta and the latest SVN.

comment:3 Changed 8 years ago by fredck

  • Component changed from UI : Dialogs to General
  • Milestone set to FCKeditor 2.6

(Regression) This one is ok with version 2.5.1.

comment:4 Changed 8 years ago by fredck

It seems it has been introduced with [1404].

comment:5 Changed 8 years ago by fredck

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

Changed 8 years ago by fredck

comment:6 Changed 8 years ago by fredck

  • Keywords Review? added

The root of the problem was at FCKDomTools.MoveToElementEditStart. The logic there was not catching all cases.

I've completely rewritten that function in the patch, using now the DTD to identify the elements that can be editable. This solution is much straightforward.

I've also noted that the selection was not being placed in the right position on IE when creating forms or tables. I've done a small change on fckdialog.html. I would ask Martin to be sure that that change is ok, because I'm not sure why we have placed the Selection.EnsureSelection() call inside CloseDialog (which was making EnsureSelection be called two times for the Ok button).

comment:7 Changed 8 years ago by alfonsoml

  • Keywords Review- added; Review? removed

I would rename

var editable ;

to

var editableElement ;

There's a typo in fcklistslib.js Elementst

and in the what's new: being properly handled when creating

Changed 8 years ago by fredck

comment:8 Changed 8 years ago by fredck

  • Keywords Review? added; Review- removed

Good to hear that no critical things have been found in the patch. The new one fixes all your considerations.

comment:9 Changed 8 years ago by martinkou

  • Keywords Review+ added; Review? removed

The bug was already fixed by [1795], so it cannot be reproduce with the newest SVN code anyways. The changes to MoveToElementEditStart() is still an sensible improvement to its robustness, however.

Moving the EnsureSelection() to inside Cancel() should be ok, I don't remember any particular reason I placed the EnsureSelection() inside CloseDialog(), and I wasn't able to discover any new bugs caused by the change anyway.

comment:10 Changed 8 years ago by fredck

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

Fixed with [1803]. Click here for more info about our SVN system.

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