Ticket #3401 (closed New Feature: fixed)

Opened 17 months ago

Last modified 9 months ago

V3 - Implement baseHref functionality

Reported by: arczi Owned by: garry.yao
Priority: Normal Milestone: CKEditor 3.1
Component: General Version: SVN (CKEditor) - OLD
Keywords: Confirmed Review+ Cc:

Description

We have base tag config but it works in preview only.

Attachments

3401.patch Download (21.8 KB) - added by garry.yao 10 months ago.
3401_2.patch Download (2.9 KB) - added by garry.yao 9 months ago.
3401_3.patch Download (2.0 KB) - added by garry.yao 9 months ago.
3401_4.patch Download (3.0 KB) - added by garry.yao 9 months ago.

Change History

Changed 17 months ago by arczi

  • summary changed from V3 - Implement basetag funcionality to V3 - Implement basetag functionality

Changed 14 months ago by fredck

What do you mean with "basetag"? Is it the "baseHref" setting?

Changed 14 months ago by arczi

Yes. I mean baseHref setting.

Changed 14 months ago by fredck

  • milestone changed from CKEditor 3.x to CKEditor 3.1

Changed 13 months ago by arczi

  • keywords Confirmed added

Changed 12 months ago by alfonsoml

  • summary changed from V3 - Implement basetag functionality to V3 - Implement baseHref functionality

Changed 11 months ago by garry.yao

  • owner set to garry.yao
  • status changed from new to assigned

Depend on #4067.

Changed 10 months ago by garry.yao

Changed 10 months ago by garry.yao

  • keywords Review? added

Include the latest patch at #4067 for easy reviewing.

Changed 10 months ago by garry.yao

#4664 has been marked as a DUP.

Changed 10 months ago by fredck

  • keywords Review- added; Review? removed

Let's have #4067 closed first at this point, having a new patch here for this tickets needs only.

Changed 9 months ago by garry.yao

  • keywords Review? added; Review- removed
  • version set to SVN (CKEditor)

Review has to be conducted with the  fullpage branch.

Changed 9 months ago by fredck

  • keywords Review- added; Review? removed

The basehref must also be active in the previews of the image and flash dialogs.

Changed 9 months ago by fredck

The initial support for this feature has already been committed into the 3.1.x branch with [4636]. The fix for the above issue is still missing though, and this should be done through patches in this ticket now.

Changed 9 months ago by garry.yao

Changed 9 months ago by garry.yao

  • keywords Review? added; Review- removed
  • status changed from assigned to new

Changed 9 months ago by garry.yao

Changed 9 months ago by garry.yao

  • status changed from new to assigned

3401_3 is unobtrusive while 3401_2.patch are just safer.

Changed 9 months ago by fredck

  • keywords Review- added; Review? removed

The 3401_2.patch is the way to go for me, but it throws "Permission denied" in IE when inserting an image, going to source and back, and opening the image dialog again. I think the previous previewImage DOM object is being used, but the editor.document has been destroyed and recreated.

3401_3.patch is not good instead. It's not working on all cases.

The following are the TCs I'm using: 

URL: logo.gif
config.baseHref = 'http://www.google.com/intl/en_ALL/images/';

URL: ../images/logo.gif
config.baseHref = 'http://www.google.com/intl/en_ALL/images/';

URL: /intl/en_ALL/images/logo.gif
config.baseHref = 'http://www.google.com/intl/en_ALL/images/';

URL: http://www.google.com/intl/en_ALL/images/logo.gif
config.baseHref = 'http://www.yahoo.com/';

Changed 9 months ago by garry.yao

Changed 9 months ago by garry.yao

  • keywords Review? added; Review- removed

"Permission denied" in IE...I think...the editor.document has been destroyed and recreated.

True, but should be IE6 only, where cross document element is treated as a security risk.

3401_3.patch is not good instead.

I failed to take considering of the 3rd TCs.

Manual Test added at :
 http://ckeditor.t/tt/3401/1.html.

Changed 9 months ago by fredck

  • keywords Review+ added; Review? removed

Changed 9 months ago by garry.yao

  • status changed from assigned to closed
  • resolution set to fixed

Fixed with [4665] at 3.1.x branch.

Note: See TracTickets for help on using tickets.