Opened 14 years ago

Closed 12 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 14 years ago by Frederico Caldeira Knabben

Keywords: image removed
Milestone: CKEditor 3.2CKEditor 3.3
Priority: HighNormal

comment:2 Changed 14 years ago by Michael Bu

Only happens if config.baseHref was set. Could workaround by not setting config.baseHref

comment:3 Changed 14 years ago by Alfonso Martínez de Lizarrondo

Milestone: CKEditor 3.3CKEditor 3.4

comment:4 Changed 14 years ago by Garry Yao

Keywords: Pending added
Milestone: CKEditor 3.4

WFM in IE6 with the following test case:

  1. baseHref configured to "http://www.google.com/intl/en_ALL/images/";
  2. Insert a image with URL "/intl/en_ALL/images/logo.gif";

comment:5 in reply to:  4 Changed 13 years ago by Alejandro

Replying to garry.yao: I confirm this bug. This bug also happens to me with IE7

WFM in IE6 with the following test case:

  1. baseHref configured to "http://www.google.com/intl/en_ALL/images/";
  2. Insert a image with URL "/intl/en_ALL/images/logo.gif";

comment:6 Changed 12 years ago by Jakub Ś

Resolution: expired
Status: pendingclosed

I have checked the above TC and it seems to work fine even in IE6 or IE7

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