Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

#7619 closed Bug (fixed)

CKEditor dialogs are displayed behind ActiveX controls in IE

Reported by: Teresa Monahan Owned by: Garry Yao
Priority: Normal Milestone: CKEditor 3.6.2
Component: UI : Dialogs Version: 3.0
Keywords: IBM Cc: Damian, Satya Minnekanti, James Cunningham

Description

When an ActiveX control is included in a HTML page that also includes CKEditor, CKEditor dialogs are displayed behind the ActiveX control in IE. For example, copy the attached test case into the _samples directory. Open it in IE, and open the 'Insert Image' dialog. The dialog is displayed behing the activeX control.

This occurs in IE7 and IE8. In IE6, there appears to be a different issue where the activeX control does not display when the dialog is open. There seems to be some IE6 specific code in the showCover and hideCover functions in dialog\plugin.js which adds an iframe as a child of the dialog background cover which could account for this difference.

Attachments (3)

activeX_testcase.html (1.1 KB) - added by Teresa Monahan 13 years ago.
7619.patch (1.9 KB) - added by Garry Yao 13 years ago.
7619error.JPG (123.6 KB) - added by Satya Minnekanti 13 years ago.

Download all attachments as: .zip

Change History (15)

Changed 13 years ago by Teresa Monahan

Attachment: activeX_testcase.html added

comment:1 Changed 13 years ago by Jakub Ś

Status: newconfirmed
Version: 3.5.33.0

Happens in all browsers.

In FF4 only the corners are hidden.

Opera is the only browser where dialog is not covered but the youtube movie is hidden instead (which is very interesting by the way:)). I'm not sure if this is the correct behavior.

comment:2 Changed 13 years ago by Garry Yao

Resolution: invalid
Status: confirmedclosed

This should be fixed by the app instead of editor, check transparent flash option.

comment:3 Changed 13 years ago by Teresa Monahan

This is not only an issue with flash content - it happens for all activeX controls. Therefore the proposed solution is not valid. Can you please reopen this ticket to investigate further?

comment:4 Changed 13 years ago by Frederico Caldeira Knabben

Resolution: invalid
Status: closedreopened

I'm not sure we're able to workaround this problem, but this is certainly a valid issue.

comment:5 in reply to:  description ; Changed 13 years ago by Frederico Caldeira Knabben

Component: GeneralUI : Dialogs
Status: reopenedconfirmed

Replying to tmonahan:

There seems to be some IE6 specific code in the showCover and hideCover functions in dialog\plugin.js which adds an iframe as a child of the dialog background cover which could account for this difference.

In fact, this may be a solution for it.

comment:6 in reply to:  5 Changed 13 years ago by Damian

Replying to fredck:

Replying to tmonahan:

There seems to be some IE6 specific code in the showCover and hideCover functions in dialog\plugin.js which adds an iframe as a child of the dialog background cover which could account for this difference.

In fact, this may be a solution for it.

Any update on this one?

comment:7 Changed 13 years ago by Garry Yao

Ticket Test: run OR view source.

Version 0, edited 13 years ago by Garry Yao (next)

Changed 13 years ago by Garry Yao

Attachment: 7619.patch added

comment:8 Changed 13 years ago by Garry Yao

Owner: set to Garry Yao
Status: confirmedreview

comment:9 Changed 13 years ago by Frederico Caldeira Knabben

Milestone: CKEditor 3.6.2
Status: reviewreview_passed

comment:10 Changed 13 years ago by Garry Yao

Resolution: fixed
Status: review_passedclosed

Fixed with [7201].

comment:11 in reply to:  10 ; Changed 13 years ago by Satya Minnekanti

Replying to garry.yao:

Fixed with [7201].

@garry.yao This is not properly fixed. Download the attached test case,modify the src to point to nightly build. with out keeping focus in editor, click on link or image icon it strips all the CSS and you will get a javascript error.

Please re oprn this ticket. I have attached a screenshot of the error.

Changed 13 years ago by Satya Minnekanti

Attachment: 7619error.JPG added

comment:12 in reply to:  11 Changed 13 years ago by Frederico Caldeira Knabben

Replying to satya:

@garry.yao This is not properly fixed. Download the attached test case,modify the src to point to nightly build. with out keeping focus in editor, click on link or image icon it strips all the CSS and you will get a javascript error.

Please re oprn this ticket. I have attached a screenshot of the error.

I'm not able to reproduce this problem. In any case, such issue is not related to this ticket and deserves a dedicated one.

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