Opened 15 years ago

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

Download all attachments as: .zip

Change History (12)

comment:1 Changed 15 years ago by brooks

confirmed in IE,Firefox,Chrome on win7

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

Owner: set to brooks
Status: newassigned

comment:4 Changed 15 years ago by brooks

Version: 3.0SVN (CKEditor)

Changed 15 years ago by brooks

Attachment: 5577.patch added

comment:5 Changed 15 years ago by brooks

Keywords: Review? added; Confirmed removed

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

Attachment: 5577_2.patch added

Changed 15 years ago by brooks

Attachment: 5577_3.patch added

comment:7 Changed 15 years ago by brooks

Keywords: Review? added; Review- removed

delete some unneeded optimized code from 5577_2.patch

comment:8 Changed 15 years ago by Garry Yao

Keywords: Confirmed Review+ added; Review? removed

comment:9 Changed 15 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