Opened 11 years ago
Closed 11 years ago
#11626 closed Bug (fixed)
Tableresize sets invalid width
| Reported by: | Wiktor Walc | Owned by: | Piotr Jasiun |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 4.4.1 |
| Component: | Core : Tables | Version: | |
| Keywords: | HasPatch | Cc: |
Description (last modified by )
When resizing a column to the maximum available width (by shrinking a column next to it), the width of the second column is set to 0, which is an invalid number.
When 0px is used, the column behaves like it did have width style set at all, while the fact is that user decided to resize it to the minimum.
Setting 1px will preserve a narrow size of the column.
1) The moment of resizing a column
2) The second column has width set to 0px
<tr> <td style="width: 238px;"> </td> <td style="width: 0px;"> </td> <td> </td> <td> </td> </tr>
3) The second column has width set to 1px
<tr> <td style="width: 237px;"> </td> <td style="width: 1px;"> </td> <td> </td> <td> </td> </tr>
Attachments (4)
Change History (17)
Changed 11 years ago by
| Attachment: | table-resize-cropped.png added |
|---|
Changed 11 years ago by
| Attachment: | tableresize-0-width.png added |
|---|
Changed 11 years ago by
| Attachment: | tableresize-1px-width.png added |
|---|
comment:1 Changed 11 years ago by
| Description: | modified (diff) |
|---|
comment:2 Changed 11 years ago by
| Status: | new → confirmed |
|---|
comment:3 Changed 11 years ago by
comment:5 Changed 11 years ago by
| Keywords: | HasPatch added |
|---|
comment:6 Changed 11 years ago by
| Milestone: | → CKEditor 4.4.1 |
|---|
comment:7 Changed 11 years ago by
| Owner: | set to Piotr Jasiun |
|---|---|
| Status: | confirmed → assigned |
comment:8 Changed 11 years ago by
Thank you for your changes vlcekmi3. They work fine. Changes in t/11626.
comment:9 Changed 11 years ago by
| Status: | assigned → review |
|---|
comment:10 Changed 11 years ago by
| Status: | review → review_failed |
|---|
This patch was also submitted in pull request - please use it: https://github.com/ckeditor/ckeditor-dev/pull/87
comment:12 Changed 11 years ago by
| Status: | review → review_passed |
|---|
I force pushed rebased branches with a simplified comment.
comment:13 Changed 11 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | review_passed → closed |
Merged into master with hash git:41e48b3
hi, i'lll do that if its still unassigned