Opened 10 years ago
Closed 9 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 10 years ago by
| Attachment: | table-resize-cropped.png added |
|---|
Changed 10 years ago by
| Attachment: | tableresize-0-width.png added |
|---|
Changed 10 years ago by
| Attachment: | tableresize-1px-width.png added |
|---|
comment:1 Changed 10 years ago by
| Description: | modified (diff) |
|---|
comment:2 Changed 10 years ago by
| Status: | new → confirmed |
|---|
comment:3 Changed 10 years ago by
comment:5 Changed 10 years ago by
| Keywords: | HasPatch added |
|---|
comment:6 Changed 10 years ago by
| Milestone: | → CKEditor 4.4.1 |
|---|
comment:7 Changed 9 years ago by
| Owner: | set to Piotr Jasiun |
|---|---|
| Status: | confirmed → assigned |
comment:8 Changed 9 years ago by
Thank you for your changes vlcekmi3. They work fine. Changes in t/11626.
comment:9 Changed 9 years ago by
| Status: | assigned → review |
|---|
comment:10 Changed 9 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 9 years ago by
| Status: | review → review_passed |
|---|
I force pushed rebased branches with a simplified comment.
comment:13 Changed 9 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