Opened 6 years ago

Closed 6 years ago

#6289 closed Bug (fixed)

Deleting nested table removes the parent cell

Reported by: yiminghe Owned by: Saare
Priority: Normal Milestone: CKEditor 3.4.2
Component: General Version: 3.3
Keywords: IE Cc:

Description (last modified by krst)

note:only occurs in IE series

steps:

  1. insert a table A
  2. insert a new table B to A's one empty cell
  3. delete table B through B's contextmenu "删除表格(delete table)"
  4. A's corresponding cell is deleted too,that's wrong

should relate to :

If the table's parent has only one child, remove it,except body,as well.( #5416 )

i think "delete table" should except td as well ?

Attachments (3)

before_table.png (7.9 KB) - added by yiminghe 6 years ago.
contextmenu show
after_table.png (3.2 KB) - added by yiminghe 6 years ago.
after delete table ,parent cell is deleted too
6289.patch (1.3 KB) - added by Saare 6 years ago.

Download all attachments as: .zip

Change History (10)

Changed 6 years ago by yiminghe

contextmenu show

Changed 6 years ago by yiminghe

after delete table ,parent cell is deleted too

comment:1 Changed 6 years ago by krst

  • Description modified (diff)
  • Keywords IE added
  • Status changed from new to confirmed

comment:2 Changed 6 years ago by krst

  • Description modified (diff)

comment:3 Changed 6 years ago by Saare

  • Milestone set to CKEditor 3.4.2
  • Summary changed from delete table error to Deleting nested table removes the parent cell

#6295 is a DUP.

comment:4 Changed 6 years ago by Saare

  • Owner set to Saare
  • Status changed from confirmed to assigned

Changed 6 years ago by Saare

comment:5 Changed 6 years ago by Saare

  • Status changed from assigned to review
  • Version changed from 3.4 to 3.3

comment:6 Changed 6 years ago by garry.yao

  • Status changed from review to review_passed

comment:7 Changed 6 years ago by Saare

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

Fixed with [5902].

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