Opened 4 years ago

Closed 3 years ago

#8888 closed Bug (fixed)

CK Editor dialogs do not show completely in small window

Reported by: satya Owned by: a.nowodzinski
Priority: Normal Milestone: CKEditor 4.0.1
Component: UI : Dialogs Version: 3.0
Keywords: IBM Cc: damo, tmonahan

Description

To reproduce the defect:

  1. Open any CK Editor sample.
  1. Re-size browser window to minimum.
  1. Click on Table icon.

Issue: Table Properties dialog comes up but we can't see some of the fields, even when we use the browser scroll bar we can't see the fields that are out of view.

I have attached a screenshot of the issue.

It happens for all the big diloags (Image, Flash,IFrame, Paste dialogs)

Attachments (2)

Table dialog when browser minimised.jpg (19.2 KB) - added by satya 4 years ago.
8888.patch (4.5 KB) - added by garry.yao 4 years ago.

Download all attachments as: .zip

Change History (14)

Changed 4 years ago by satya

comment:1 Changed 4 years ago by j.swiderski

  • Keywords IBM added; ibm removed
  • Status changed from new to confirmed
  • Version set to 3.0

Confirmed in all browsers from CKEditor 3.0

Just wondering what could we do here? Perhaps this could be one of:

  1. Scrolling dialog when browser scrolls but only when browser viewport is smaller than dialog width/height - not sure if this is possible
  2. Resizing the dialog when it is bigger than the viewport and adding scrollbar inside it

Changed 4 years ago by garry.yao

comment:2 Changed 4 years ago by garry.yao

  • Component changed from General to UI : Dialogs
  • Milestone set to CKEditor 3.6.5
  • Owner set to garry.yao
  • Status changed from confirmed to review

Fixed position is not anymore suitable for small viewport, thus the patch propose a check on dialog show, potentially switch to absolute position in such case, and to re-layout the dialog on window resize.

comment:3 Changed 4 years ago by fredck

  • Status changed from review to review_passed

comment:4 follow-up: Changed 4 years ago by garry.yao

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

Fixed with [7612].

comment:5 in reply to: ↑ 4 Changed 4 years ago by satya

Replying to garry.yao:

Fixed with [7612].

This is still an issue in IE7, IE8 Compatability view & IE9 Compatability view. Can you please re-open the ticket

comment:6 Changed 3 years ago by j.swiderski

  • Resolution fixed deleted
  • Status changed from closed to reopened

IE7-IE8 - I can confirm this problem in CKE 3.x. If you open dialog and scroll, dialog will not move but if you open dialog, move it up (grab and move up) and then scroll, the dialog will also scroll.
It is also possible to lose dialog in that way - if you grab it and try to move it dialog may dissapear.

IE9-IE10 - I can confirm this problem in CKE 3.x. If you open dialog and scroll or open, drag dialog and scroll, dialog will not move.

comment:7 Changed 3 years ago by j.swiderski

  • Milestone CKEditor 3.6.5 deleted
  • Status changed from reopened to confirmed

comment:8 Changed 3 years ago by Reinmar

  • Milestone set to CKEditor 4.0.1

comment:9 Changed 3 years ago by a.nowodzinski

  • Owner changed from garry.yao to a.nowodzinski
  • Status changed from confirmed to assigned

comment:10 Changed 3 years ago by a.nowodzinski

  • Status changed from assigned to review

Created a 1 LOC fix for this issue. Unfortunately this is a rendering bug in IE like #6068 or http://murphy.cz/ie7-hover-ghosts-bug/ and most likely it has nothing to do with dialog code.

comment:11 Changed 3 years ago by Reinmar

  • Status changed from review to review_passed

R+.

Extracted another issue: #9850.

comment:12 Changed 3 years ago by Reinmar

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

Masterised git:3059c62.

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