Ticket #4952 (closed Bug: fixed)

Opened 5 years ago

Last modified 5 years ago

[IE] Editing table creates a new table

Reported by: damo Owned by: garry.yao
Priority: Normal Milestone: CKEditor 3.1
Component: General Version: 3.0.2
Keywords: IBM Confirmed IE Review+ Cc:

Description (last modified by fredck) (diff)

To reproduce:

  1. Open nightly demo
  2. Create default table
  3. Edit the table

Notice that a new table is created.


4952.patch (809 bytes) - added by garry.yao 5 years ago.

Change History

comment:1 Changed 5 years ago by tobiasz.cudnik

  • Keywords Pending added

If step 3 "Edit the table" was meant by using table toolbar button, then it looks like duplicate of #4635 "Unable to open property dialog for existing table from toolbar" which is invalid, due to possibility of nested tables.

comment:2 Changed 5 years ago by fredck

  • Priority changed from High to Normal
  • Milestone CKEditor 3.1 deleted

Other than #4635, it may be a duplicate of #4948. Can you confirm it?

comment:3 Changed 5 years ago by damo

Step 3. Should be refined. It means right-click on any cell in the newly created table and select "Table Properties".

I think the problem is down to selections. When you first left-click into the cell then right-click, it works OK. But if you have the cursor outside the table and then you right-click into a cell, the correct context menu (and properties dialog) comes up but the cursor moves out of the table. So when you click OK, the dialog creates a new table, usually above the table that was being edited.

Reproduced in IE 7.

comment:4 Changed 5 years ago by fredck

  • Keywords Confirmed IE added; Pending removed
  • Version set to 3.0.2
  • Milestone set to CKEditor 3.1

Confirmed regression with IE8 also. This is definitely a selection issue. No problem with FF.

comment:5 Changed 5 years ago by fredck

Issue introduced with [4521].

Changed 5 years ago by garry.yao

comment:6 Changed 5 years ago by garry.yao

  • Owner set to garry.yao
  • Keywords Review? added
  • Status changed from new to assigned

The patch simply revert [4521] as #4041 is not any more reproduced, probably fixed by recent selection related fixings.

comment:7 Changed 5 years ago by fredck

  • Keywords Review+ added; Review? removed
  • Description modified (diff)

comment:8 Changed 5 years ago by garry.yao

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

Fixed with [4876].

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