Opened 6 years ago

Closed 5 years ago

Last modified 5 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 6 years ago.
chrome_resize_error2.png (80.0 KB) - added by Anna Tomanek 6 years ago.
chrome_resize_error3.png (131.7 KB) - added by Anna Tomanek 6 years ago.
safari_resize_error1.png (48.4 KB) - added by Anna Tomanek 6 years ago.
7360.patch (1.5 KB) - added by Garry Yao 6 years ago.
7360_2.patch (812 bytes) - added by Garry Yao 5 years ago.

Download all attachments as: .zip

Change History (21)

Changed 6 years ago by Anna Tomanek

Attachment: chrome_resize_error1.png added

Changed 6 years ago by Anna Tomanek

Attachment: chrome_resize_error2.png added

Changed 6 years ago by Anna Tomanek

Attachment: chrome_resize_error3.png added

Changed 6 years ago by Anna Tomanek

Attachment: safari_resize_error1.png added

comment:1 Changed 6 years ago by Anna Tomanek

Status: newconfirmed

comment:2 Changed 6 years ago by Anna Tomanek

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

comment:3 Changed 6 years ago by Jakub Ś

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

comment:4 Changed 6 years ago by Garry Yao

Component: GeneralUI : Toolbar

comment:5 Changed 6 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 6 years ago by Garry Yao

Attachment: 7360.patch added

comment:6 Changed 6 years ago by Jakub Ś

#6195 was marekd as duplicate.

comment:7 Changed 6 years ago by Jakub Ś

#8568 was marked as duplicate.

comment:8 Changed 5 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 5 years ago by Jakub Ś

#8779 was marked as duplicate.

comment:10 Changed 5 years ago by Garry Yao

#8071 is a DUP too.

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

Changed 5 years ago by Garry Yao

Attachment: 7360_2.patch added

comment:11 Changed 5 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 5 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 5 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 5 years ago by Garry Yao

Resolution: fixed
Status: review_passedclosed

Fixed with [7387].

comment:15 Changed 5 years ago by Julian Picht

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

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