Opened 14 years ago

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

Download all attachments as: .zip

Change History (12)

comment:1 Changed 14 years ago by brooks

confirmed in IE,Firefox,Chrome on win7

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

Owner: set to brooks
Status: newassigned

comment:4 Changed 14 years ago by brooks

Version: 3.0SVN (CKEditor)

Changed 14 years ago by brooks

Attachment: 5577.patch added

comment:5 Changed 14 years ago by brooks

Keywords: Review? added; Confirmed removed

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

Attachment: 5577_2.patch added

Changed 14 years ago by brooks

Attachment: 5577_3.patch added

comment:7 Changed 14 years ago by brooks

Keywords: Review? added; Review- removed

delete some unneeded optimized code from 5577_2.patch

comment:8 Changed 14 years ago by Garry Yao

Keywords: Confirmed Review+ added; Review? removed

comment:9 Changed 14 years ago by brooks

Resolution: fixed
Status: assignedclosed

fixed with 5418 5418

Note: See TracTickets for help on using tickets.
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy