Opened 5 years ago

Closed 4 years ago

Last modified 4 years ago

#7360 closed Bug (fixed)

WebKit: problems with editor interface after browser window resize

Reported by: Anna 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 5 years ago.
chrome_resize_error2.png (80.0 KB) - added by Anna 5 years ago.
chrome_resize_error3.png (131.7 KB) - added by Anna 5 years ago.
safari_resize_error1.png (48.4 KB) - added by Anna 5 years ago.
7360.patch (1.5 KB) - added by garry.yao 5 years ago.
7360_2.patch (812 bytes) - added by garry.yao 4 years ago.

Download all attachments as: .zip

Change History (21)

Changed 5 years ago by Anna

Changed 5 years ago by Anna

Changed 5 years ago by Anna

Changed 5 years ago by Anna

comment:1 Changed 5 years ago by Anna

  • Status changed from new to confirmed

comment:2 Changed 5 years ago by Anna

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

comment:3 Changed 5 years ago by j.swiderski

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

comment:4 Changed 5 years ago by garry.yao

  • Component changed from General to UI : Toolbar

comment:5 Changed 5 years ago by garry.yao

  • Owner set to garry.yao
  • Status changed from confirmed to review

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

Changed 5 years ago by garry.yao

comment:6 Changed 4 years ago by j.swiderski

#6195 was marekd as duplicate.

comment:7 Changed 4 years ago by j.swiderski

#8568 was marked as duplicate.

comment:8 Changed 4 years ago by j.swiderski

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 4 years ago by j.swiderski

#8779 was marked as duplicate.

comment:10 Changed 4 years ago by garry.yao

#8071 is a DUP too.

Last edited 4 years ago by garry.yao (previous) (diff)

Changed 4 years ago by garry.yao

comment:11 Changed 4 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 4 years ago by fredck

  • Milestone set to CKEditor 3.6.3
  • Status changed from review to review_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 4 years ago by fredck

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

comment:14 Changed 4 years ago by garry.yao

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

Fixed with [7387].

comment:15 Changed 4 years ago by picht

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

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