Opened 6 years ago

Last modified 6 years ago

#11750 confirmed Bug

Iframe Dialog Scrollbar problem.

Reported by: Jakub Ś Owned by:
Priority: Normal Milestone:
Component: General Version: 4.3.2
Keywords: IE Cc:

Description

Problem can be reproduced in IE8-10 (works in IE11).

  1. Add attached insertVariable plugin (best in CKEditor dev)
  2. Add below code in HTML page:
    CKEDITOR.replace( 'editor1', {
    	extraPlugins: 'insertVariable'
    });
    
  3. Load the page and open insertVariable Dialog.

Result: in IE8-10 there is always vertical scrollbar. This is happening because below TD element exceeds size of wrapper div:

<td class="cke_dialog_ui_vbox_child" role="presentation" style="width: 100%; height: 100%;">
<div class="cke_dialog_ui_vbox cke_dialog_page_contents" id="cke_111_uiElement" role="tabpanel" aria-hidden="false" aria-labelledby="cke_iframe_112" style="width: 100%; height: 100%;" name="iframe">

This scrolbarr is not a big issue with small content but you you use large contant you will get double vertical scrollbar (Please see attached iframeDialog.png)


One solution to this problem is adding below rule in ckeditor/skins/moono/dialog_ie.css:

.cke_dialog_page_contents {
	overflow:hidden;
}

The above rule solves the problem (please note that kama skin uses this class in dialog_iequirks.css) but if someone uses ony iframe in his dialog it would be nice idea to to have paddings removed on cke_dialog_contents_body. Perhaps we could introduce separate class for iframeDialog and 'contents body' or simply modify both?

Attachments (2)

iframeDialog.png (156.5 KB) - added by Jakub Ś 6 years ago.
insertVariable.zip (1.9 KB) - added by Jakub Ś 6 years ago.

Download all attachments as: .zip

Change History (3)

Changed 6 years ago by Jakub Ś

Attachment: iframeDialog.png added

Changed 6 years ago by Jakub Ś

Attachment: insertVariable.zip added

comment:1 Changed 6 years ago by Jakub Ś

Status: newconfirmed
Version: 4.3.2

Problem can be reproduced from CKEditor 4.3.2.

Another size and iframedialog issue is: #11034. Perhaps it would be worth checking both of them.

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