Opened 15 years ago
Closed 13 years ago
#5020 closed Bug (expired)
Inserted image not being displayed correctly on IE6
Reported by: | Michael Bu | Owned by: | |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | UI : Dialogs | Version: | 3.1 |
Keywords: | Cc: |
Description
When inserting images using the Image plugin, if the "fileurl" string returned via "window.opener.CKEDITOR.tools.callFunction" as stated in http://docs.cksource.com/CKEditor_3.x/Developers_Guide/File_Browser_%28Uploader%29/Custom_File_Browser is not a full URL path (without the protocol and domain info), on IE6 the image src would be rendered as "http:///path_to_image" with the extra "http://" in front. Which in turn cause the image not being displayed and inserted correctly as expected.
The root cause has to do with the way those "img" elements are created in the image plugin. On IE6 this line of code in plugins/image/dialogs/image.js would always fail:
original.setAttribute( 'src', newUrl );
Change History (6)
comment:1 Changed 15 years ago by
Keywords: | image removed |
---|---|
Milestone: | CKEditor 3.2 → CKEditor 3.3 |
Priority: | High → Normal |
comment:2 Changed 15 years ago by
comment:3 Changed 15 years ago by
Milestone: | CKEditor 3.3 → CKEditor 3.4 |
---|
comment:4 follow-up: 5 Changed 15 years ago by
Keywords: | Pending added |
---|---|
Milestone: | CKEditor 3.4 |
WFM in IE6 with the following test case:
- baseHref configured to "http://www.google.com/intl/en_ALL/images/";
- Insert a image with URL "/intl/en_ALL/images/logo.gif";
comment:5 Changed 14 years ago by
Replying to garry.yao: I confirm this bug. This bug also happens to me with IE7
WFM in IE6 with the following test case:
- baseHref configured to "http://www.google.com/intl/en_ALL/images/";
- Insert a image with URL "/intl/en_ALL/images/logo.gif";
comment:6 Changed 13 years ago by
Resolution: | → expired |
---|---|
Status: | pending → closed |
I have checked the above TC and it seems to work fine even in IE6 or IE7
Only happens if config.baseHref was set. Could workaround by not setting config.baseHref