Opened 12 years ago

Closed 12 years ago

#10123 closed Bug (fixed)

[Chrome] Insert image dialog layout broken in v25.0.1364.97

Reported by: Olek Nowodziński Owned by: Olek Nowodziński
Priority: Normal Milestone: CKEditor 4.0.2
Component: UI : Dialogs Version: 4.0.2
Keywords: Cc:

Description (last modified by Olek Nowodziński)

Latest version of Google Chrome has problem with correct rendering of image dialog. Other dialog windows may also be affected.

Attachments (2)

insertimagechrome2425.png (122.2 KB) - added by Olek Nowodziński 12 years ago.
24 vs 25
10123_TC.html (1.1 KB) - added by Frederico Caldeira Knabben 12 years ago.
Test page.

Download all attachments as: .zip

Change History (10)

Changed 12 years ago by Olek Nowodziński

Attachment: insertimagechrome2425.png added

24 vs 25

comment:1 Changed 12 years ago by Olek Nowodziński

24 vs 25

comment:2 Changed 12 years ago by Olek Nowodziński

Description: modified (diff)

comment:3 Changed 12 years ago by Piotrek Koszuliński

Status: newconfirmed

comment:4 Changed 12 years ago by Jakub Ś

Affected dialogs are: link, table, image, image dialog, iframe.

Changed 12 years ago by Frederico Caldeira Knabben

Attachment: 10123_TC.html added

Test page.

comment:5 Changed 12 years ago by Frederico Caldeira Knabben

This is a WebKit bug. I've opened a ticket here:
https://bugs.webkit.org/show_bug.cgi?id=110868

I created a reduced test case for it (attachment:10123_TC.html). I was able to confirm the issue with both Chrome 25.0.1364.99 and Safari 6.0.2 (8536.26.17).

comment:6 Changed 12 years ago by Olek Nowodziński

Owner: set to Olek Nowodziński
Status: confirmedreview

I suggest a fix in t/10123 that seems to solve the problem and not to destroy other dialogs in v<=25. Careful testing is recommended.

comment:7 Changed 12 years ago by Piotrek Koszuliński

Status: reviewreview_passed

Ugly fix for ugly bug.

comment:8 Changed 12 years ago by Olek Nowodziński

Resolution: fixed
Status: review_passedclosed

Ugly fix git:6a3500067ca merged into master.

Note: See TracTickets for help on using tickets.
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy