Opened 6 years ago

Closed 6 years ago

#5577 closed Bug (fixed)

delete table column throws error

Reported by: brooks Owned by: brooks
Priority: Normal Milestone: CKEditor 3.3
Component: General Version: SVN (CKEditor) - OLD
Keywords: Confirmed Review+ Cc:

Description

reproduce steps:

  1. use table plugin to insert a default table

2. selected multiple cell in the same column

  1. right click to open context menu -> column-> delete columns

Attachments (3)

5577.patch (1.8 KB) - added by brooks 6 years ago.
5577_2.patch (3.4 KB) - added by brooks 6 years ago.
5577_3.patch (3.1 KB) - added by brooks 6 years ago.

Download all attachments as: .zip

Change History (12)

comment:1 Changed 6 years ago by brooks

confirmed in IE,Firefox,Chrome on win7

comment:2 Changed 6 years ago by garry.yao

  • Keywords Confirmed added
  • Version changed from SVN (CKEditor) to 3.0

Cursor has to be managed after column deletion also.

comment:3 Changed 6 years ago by brooks

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

comment:4 Changed 6 years ago by brooks

  • Version changed from 3.0 to SVN (CKEditor)

Changed 6 years ago by brooks

comment:5 Changed 6 years ago by brooks

  • Keywords Review? added; Confirmed removed

comment:6 Changed 6 years ago by garry.yao

  • Keywords Review- added; Review? removed

On a 3 col table, with cells of the last column selected, the cursor position after 'Delete Column' is put into the first column instead of the second column.

Changed 6 years ago by brooks

Changed 6 years ago by brooks

comment:7 Changed 6 years ago by brooks

  • Keywords Review? added; Review- removed

delete some unneeded optimized code from 5577_2.patch

comment:8 Changed 6 years ago by garry.yao

  • Keywords Confirmed Review+ added; Review? removed

comment:9 Changed 6 years ago by brooks

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

fixed with 5418 5418

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