Opened 8 years ago

Closed 8 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 8 years ago.
5577_2.patch (3.4 KB) - added by brooks 8 years ago.
5577_3.patch (3.1 KB) - added by brooks 8 years ago.

Download all attachments as: .zip

Change History (12)

comment:1 Changed 8 years ago by brooks

confirmed in IE,Firefox,Chrome on win7

comment:2 Changed 8 years ago by Garry Yao

Keywords: Confirmed added
Version: SVN (CKEditor)3.0

Cursor has to be managed after column deletion also.

comment:3 Changed 8 years ago by brooks

Owner: set to brooks
Status: newassigned

comment:4 Changed 8 years ago by brooks

Version: 3.0SVN (CKEditor)

Changed 8 years ago by brooks

Attachment: 5577.patch added

comment:5 Changed 8 years ago by brooks

Keywords: Review? added; Confirmed removed

comment:6 Changed 8 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 8 years ago by brooks

Attachment: 5577_2.patch added

Changed 8 years ago by brooks

Attachment: 5577_3.patch added

comment:7 Changed 8 years ago by brooks

Keywords: Review? added; Review- removed

delete some unneeded optimized code from 5577_2.patch

comment:8 Changed 8 years ago by Garry Yao

Keywords: Confirmed Review+ added; Review? removed

comment:9 Changed 8 years ago by brooks

Resolution: fixed
Status: assignedclosed

fixed with 5418 5418

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