Opened 10 years ago

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

Download all attachments as: .zip

Change History (12)

comment:1 Changed 10 years ago by brooks

confirmed in IE,Firefox,Chrome on win7

comment:2 Changed 10 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 10 years ago by brooks

Owner: set to brooks
Status: newassigned

comment:4 Changed 10 years ago by brooks

Version: 3.0SVN (CKEditor)

Changed 10 years ago by brooks

Attachment: 5577.patch added

comment:5 Changed 10 years ago by brooks

Keywords: Review? added; Confirmed removed

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

Attachment: 5577_2.patch added

Changed 10 years ago by brooks

Attachment: 5577_3.patch added

comment:7 Changed 10 years ago by brooks

Keywords: Review? added; Review- removed

delete some unneeded optimized code from 5577_2.patch

comment:8 Changed 10 years ago by Garry Yao

Keywords: Confirmed Review+ added; Review? removed

comment:9 Changed 10 years ago by brooks

Resolution: fixed
Status: assignedclosed

fixed with 5418 5418

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