Opened 13 years ago

Closed 12 years ago

Last modified 12 years ago

#7360 closed Bug (fixed)

WebKit: problems with editor interface after browser window resize

Reported by: Anna Tomanek Owned by: Garry Yao
Priority: Normal Milestone: CKEditor 3.6.3
Component: UI : Toolbar Version: 3.0
Keywords: Webkit Cc:

Description

Steps to reproduce:

  1. Open the replacebycode.html sample in Chrome or Safari.
  2. Change browser window width.
  3. The editor toolbar and right border do not reflow and the right editor border is gone. The HTML element containing the editor seems to occupy the whole window width.

Please note this happens to the first editor instance on the page with multiple editors OR to any CKEditor instance after you switch between Source and WYSIWYG view.

Attachments (6)

chrome_resize_error1.png (71.7 KB) - added by Anna Tomanek 13 years ago.
chrome_resize_error2.png (80.0 KB) - added by Anna Tomanek 13 years ago.
chrome_resize_error3.png (131.7 KB) - added by Anna Tomanek 13 years ago.
safari_resize_error1.png (48.4 KB) - added by Anna Tomanek 13 years ago.
7360.patch (1.5 KB) - added by Garry Yao 13 years ago.
7360_2.patch (812 bytes) - added by Garry Yao 12 years ago.

Download all attachments as: .zip

Change History (21)

Changed 13 years ago by Anna Tomanek

Attachment: chrome_resize_error1.png added

Changed 13 years ago by Anna Tomanek

Attachment: chrome_resize_error2.png added

Changed 13 years ago by Anna Tomanek

Attachment: chrome_resize_error3.png added

Changed 13 years ago by Anna Tomanek

Attachment: safari_resize_error1.png added

comment:1 Changed 13 years ago by Anna Tomanek

Status: newconfirmed

comment:2 Changed 13 years ago by Anna Tomanek

#6195 may be related, judging by the attached screenshot.

comment:3 Changed 13 years ago by Jakub Ś

This bug is very similar (judging by the picture) to #8071

comment:4 Changed 13 years ago by Garry Yao

Component: GeneralUI : Toolbar

comment:5 Changed 13 years ago by Garry Yao

Owner: set to Garry Yao
Status: confirmedreview

Making it worse, on other skins instead, it totally prevents the editor chrome from resizing.

Changed 13 years ago by Garry Yao

Attachment: 7360.patch added

comment:6 Changed 12 years ago by Jakub Ś

#6195 was marekd as duplicate.

comment:7 Changed 12 years ago by Jakub Ś

#8568 was marked as duplicate.

comment:8 Changed 12 years ago by Jakub Ś

Another TC:

  1. Open demo or sample page (replacebycode)
  2. On demo - Enlarge page by pressing CTRL and PLUS (+) twice. Than press CTRL and Minus twice(-)
  3. On sample - Reduce page by pressing CTRL and MINUS (-) twice. Than press CTRL and Plus twice(+)

comment:9 Changed 12 years ago by Jakub Ś

#8779 was marked as duplicate.

comment:10 Changed 12 years ago by Garry Yao

#8071 is a DUP too.

Last edited 12 years ago by Garry Yao (previous) (diff)

Changed 12 years ago by Garry Yao

Attachment: 7360_2.patch added

comment:11 Changed 12 years ago by Garry Yao

Previous patch doesn't handle tc raised by the second tc.

http://ckeditor.t/tt/7360/1.html

comment:12 Changed 12 years ago by Frederico Caldeira Knabben

Milestone: CKEditor 3.6.3
Status: reviewreview_passed

I found the issue a bit random, not happening all times. This tends to tell me that we may be just missing something in the editor structure or CSS to have this issue solved.

But, as we don't have anything better right now, let's go ahead with the proposed hack.

comment:13 Changed 12 years ago by Frederico Caldeira Knabben

Btw, the "onResize" named function is not needed. Please pass it directly to the event listener function instead when committing.

comment:14 Changed 12 years ago by Garry Yao

Resolution: fixed
Status: review_passedclosed

Fixed with [7387].

comment:15 Changed 12 years ago by Julian Picht

This patch breaks the editor in some cases. See #9160

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