Opened 14 years ago
Closed 12 years ago
#6111 closed Bug (fixed)
Splitting cell horizontally after splitting another vertically can cause incorrect colspans
Reported by: | Evan Krall | Owned by: | Martin |
---|---|---|---|
Priority: | Normal | Milestone: | CKEditor 3.6.3 |
Component: | Core : Tables | Version: | 3.1 |
Keywords: | Cc: |
Description
Steps to reproduce:
- Create a 3x3 table
- Split vertically the third cell in the second row
- Split horizontally the second cell in the third row.
Expected: 4x4 table with some cells merged Actual: 2nd cell in 2nd row has colspan=3, causing the last cells in the 2nd and 3rd row to stick out the right side of the table.
Attachments (4)
Change History (16)
Changed 14 years ago by
Attachment: | ckeditor_tableproblems.PNG added |
---|
comment:1 Changed 14 years ago by
Keywords: | table colspan rowspan removed |
---|---|
Status: | new → confirmed |
Confirmed in IE8 and FF 3.6.8. I did not test other browsers.
comment:2 Changed 14 years ago by
Owner: | set to Martin |
---|---|
Status: | confirmed → assigned |
comment:3 Changed 14 years ago by
Status: | assigned → review |
---|
comment:4 Changed 14 years ago by
Status: | review → review_failed |
---|
Patch is OK, but coding style is wrong - please use tabs for indentation.
Besides this, nice fix :)
Changed 14 years ago by
Attachment: | 6111_3.patch added |
---|
comment:5 Changed 14 years ago by
Status: | review_failed → review |
---|
comment:6 Changed 14 years ago by
Component: | General → Core : Tables |
---|
comment:8 Changed 13 years ago by
Version: | → 3.1 |
---|
comment:9 Changed 13 years ago by
Keywords: | HasPatch added |
---|
I did a quick test for patch 3 and it seems to fix the problem.
comment:11 Changed 12 years ago by
Milestone: | → CKEditor 3.6.3 |
---|---|
Status: | review → review_passed |
comment:12 Changed 12 years ago by
Keywords: | HasPatch removed |
---|---|
Resolution: | → fixed |
Status: | review_passed → closed |
Fixed with [7409].
Step-by-step instructions for reproduction