Opened 8 years ago

Closed 8 years ago

#8348 closed Bug (fixed)

Editor not resizing properly while table with width:500% added

Reported by: Krzysztof Studnik Owned by: Frederico Caldeira Knabben
Priority: Normal Milestone: CKEditor 3.6.2
Component: UI : Skins Version: 3.6.2
Keywords: Webkit Cc:

Description

Safari & Chrome

TC

  • open editor and add Table with width set to 500%
  • resize width of editor to make it smaller

Result

Editor body is not properly resized, editor frame is shrinking, while edit area and toolbars do not change.

Does not happen in 3.6.1

Attachments (3)

zrzut_ekranu-29.png (37.0 KB) - added by Krzysztof Studnik 8 years ago.
8348.patch (3.9 KB) - added by Garry Yao 8 years ago.
8348_2.patch (1.3 KB) - added by Frederico Caldeira Knabben 8 years ago.

Download all attachments as: .zip

Change History (10)

Changed 8 years ago by Krzysztof Studnik

Attachment: zrzut_ekranu-29.png added

comment:1 Changed 8 years ago by Jakub Ś

Status: newconfirmed

Reproducible from rev [7213]

comment:2 Changed 8 years ago by Frederico Caldeira Knabben

Milestone: CKEditor 3.6.2

Changed 8 years ago by Garry Yao

Attachment: 8348.patch added

comment:3 Changed 8 years ago by Garry Yao

Owner: set to Garry Yao
Status: confirmedreview

Changed 8 years ago by Frederico Caldeira Knabben

Attachment: 8348_2.patch added

comment:4 Changed 8 years ago by Frederico Caldeira Knabben

Owner: changed from Garry Yao to Frederico Caldeira Knabben

I think I've found a solution which is much similar to [7213], but seems to not have performance impact.

comment:5 Changed 8 years ago by Frederico Caldeira Knabben

Btw, the past patch passes the related test:
http://ckeditor.t/tt/6212/1.html

comment:6 Changed 8 years ago by Garry Yao

Status: reviewreview_passed

comment:7 Changed 8 years ago by Frederico Caldeira Knabben

Resolution: fixed
Status: review_passedclosed

Fixed with [7266].

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