Opened 16 years ago
Closed 16 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:
- use table plugin to insert a default table
2. selected multiple cell in the same column
- right click to open context menu -> column-> delete columns
Attachments (3)
Change History (12)
comment:1 Changed 16 years ago by
comment:2 Changed 16 years ago by
| Keywords: | Confirmed added |
|---|---|
| Version: | SVN (CKEditor) → 3.0 |
Cursor has to be managed after column deletion also.
comment:3 Changed 16 years ago by
| Owner: | set to brooks |
|---|---|
| Status: | new → assigned |
comment:4 Changed 16 years ago by
| Version: | 3.0 → SVN (CKEditor) |
|---|
Changed 16 years ago by
| Attachment: | 5577.patch added |
|---|
comment:5 Changed 16 years ago by
| Keywords: | Review? added; Confirmed removed |
|---|
comment:6 Changed 16 years ago by
| 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 16 years ago by
| Attachment: | 5577_2.patch added |
|---|
Changed 16 years ago by
| Attachment: | 5577_3.patch added |
|---|
comment:7 Changed 16 years ago by
| Keywords: | Review? added; Review- removed |
|---|
delete some unneeded optimized code from 5577_2.patch
comment:8 Changed 16 years ago by
| Keywords: | Confirmed Review+ added; Review? removed |
|---|
comment:9 Changed 16 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
fixed with 5418 5418

confirmed in IE,Firefox,Chrome on win7