Opened 7 years ago

Closed 7 years ago

#5690 closed Bug (fixed)

When editing a table cell with a width="xxx%" attribute, the cell properties dialog shows pixels instead of percentage

Reported by: jonathonsim Owned by: Saare
Priority: Normal Milestone: CKEditor 3.4
Component: General Version:
Keywords: Confirmed Cc:


If your source contains a table cell with a percentage width attribute eg:

<td width="50%">

and you right click > cell > cell properties the resulting dialog will show:

Width: 50 pixels

ie it will select "pixels" rather than "percentage" in the "widthType" dropdown.

Saving this dialog will then insert

<td style="width: 50px">

which is obviously wrong.

You can reproduce this on the nightly build (edit your source and insert a percentage width attribute and reproduce the steps described above)

The attached patch fixes this issue

Attachments (2)

table-cell-width-fix.diff (603 bytes) - added by jonathonsim 7 years ago.
5690.patch (676 bytes) - added by Saare 7 years ago.

Download all attachments as: .zip

Change History (6)

Changed 7 years ago by jonathonsim

comment:1 Changed 7 years ago by alfonsoml

  • Keywords HasPatch added
  • Milestone set to CKEditor 3.4

Changed 7 years ago by Saare

comment:2 Changed 7 years ago by Saare

  • Keywords Confirmed Review? added; HasPatch removed
  • Owner set to Saare
  • Status changed from new to assigned

comment:3 Changed 7 years ago by fredck

  • Status changed from review to review_passed

comment:4 Changed 7 years ago by Saare

  • Resolution set to fixed
  • Status changed from review_passed to closed

Fixed with [5726].

Note: See TracTickets for help on using tickets.
© 2003 – 2016 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy