Opened 6 years ago

Closed 3 years ago

#5020 closed Bug (expired)

Inserted image not being displayed correctly on IE6

Reported by: gogobu 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 6 years ago by fredck

  • Keywords image removed
  • Milestone changed from CKEditor 3.2 to CKEditor 3.3
  • Priority changed from High to Normal

comment:2 Changed 5 years ago by gogobu

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

comment:3 Changed 5 years ago by alfonsoml

  • Milestone changed from CKEditor 3.3 to CKEditor 3.4

comment:4 follow-up: Changed 5 years ago by garry.yao

  • Keywords Pending added
  • Milestone CKEditor 3.4 deleted

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 4 years ago by andynedine

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 3 years ago by j.swiderski

  • Resolution set to expired
  • Status changed from pending to closed

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 – 2015 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy