#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)
Change History (15)
Changed 14 years ago by
| Attachment: | activeX_testcase.html added |
|---|
comment:1 Changed 14 years ago by
| Status: | new → confirmed |
|---|---|
| Version: | 3.5.3 → 3.0 |
comment:2 Changed 14 years ago by
| Resolution: | → invalid |
|---|---|
| Status: | confirmed → closed |
This should be fixed by the app instead of editor, check transparent flash option.
comment:3 Changed 14 years ago by
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 14 years ago by
| Resolution: | invalid |
|---|---|
| Status: | closed → reopened |
I'm not sure we're able to workaround this problem, but this is certainly a valid issue.
comment:5 follow-up: 6 Changed 14 years ago by
| Component: | General → UI : Dialogs |
|---|---|
| Status: | reopened → confirmed |
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 Changed 14 years ago by
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?
Changed 14 years ago by
| Attachment: | 7619.patch added |
|---|
comment:8 Changed 14 years ago by
| Owner: | set to Garry Yao |
|---|---|
| Status: | confirmed → review |
comment:9 Changed 14 years ago by
| Milestone: | → CKEditor 3.6.2 |
|---|---|
| Status: | review → review_passed |
comment:10 follow-up: 11 Changed 14 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | review_passed → closed |
Fixed with [7201].
comment:11 follow-up: 12 Changed 14 years ago by
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 14 years ago by
| Attachment: | 7619error.JPG added |
|---|
comment:12 Changed 14 years ago by
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.
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.