Opened 18 years ago

Closed 10 years ago

#625 closed New Feature (fixed)

Relative URLs

Reported by: calophi@… Owned by:
Priority: Normal Milestone:
Component: General Version:
Keywords: SF HasPatch Cc: Alfonso Martínez de Lizarrondo

Description

It would be nice if there was a way to put in a relative URL from the link dialog box without having to select the protocol type as "other". Perhaps adding "Relative URL" to the list of URL types along with email and anchor would be appropriate, and just have it bring up a div similar to the normal URL but without the protocol?


Moved from SF:
http://sourceforge.net/tracker/index.php?func=detail&aid=1171898&group_id=75348&atid=543656

Attachments (1)

fck_link.patch (717 bytes) - added by David Zahler 16 years ago.
Patch to add optional argument in SetUrl method. Argument allows setting protocol in link dialog when using custom link browser.

Download all attachments as: .zip

Change History (7)

comment:1 Changed 18 years ago by Martin Kou

Reporter: changed from Martin Kou to calophi@…

I've just changed the default option in fck_link.html and it works fine for me:

<select id="cmbLinkProtocol">
	<option value="http://">http://</option>
	<option value="https://">https://</option>
	<option value="ftp://">ftp://</option>
	<option value="news://">news://</option>
	<option value="" fckLang="DlgLnkProtoOther"
selected="selected">&lt;other&gt;</option>
</select>

No other changes are neccesary and I think that this is a better default as pasting a full link changes the option with the current js code.


Moved from SF. Original poster: alfonsoml

comment:2 Changed 17 years ago by Wojciech Olchawa

Keywords: Confirmed HasPatch added

comment:3 Changed 17 years ago by Wojciech Olchawa

#592 has been marked as DUP

#592 had the same request and also a request for relative/absolute URL's in the image dialog. Since we have more information about this issue here #592 has been closed.

Changed 16 years ago by David Zahler

Attachment: fck_link.patch added

Patch to add optional argument in SetUrl method. Argument allows setting protocol in link dialog when using custom link browser.

comment:4 Changed 13 years ago by Mark Stosberg

It's been 3 years since a patch was submitted for this issue. Could someone comment on whether the patch will accepted, rejected, or need refinement?

We frequently use CKEditor to compose and send HTML emails, so we *always* want absolute URLs to be used. However, CKEditor changes our absolute links back to relative, which breaks them for the email recipients. Ticket #592 addressed exactly this issue, but it was merged back in here, although the requests seem distinct:

Here: Explicitly support relative links in the select drop down box #592: Allow an option to explicitly use absolute links, and don't convert them back to relative.

comment:5 Changed 13 years ago by Mark Stosberg

It turns out that a Firefox bug could be partially or fully at fault here. Firefox was converting absolute URLs to relative in some cases. There's a bug report about that here:

"URL is not maintained for links or images after copy/paste in contenteditable" https://bugzilla.mozilla.org/show_bug.cgi?id=613517

This issue was addressed with a workaround in the JavaScript-based "Typo3" HTML editor. They used the approach of detecting of the browser was Firefox, and then converting URLs from relative back to absolute in some cases. You can see the Typo3 bug report about this issue here, which includes links to the "diff" showing how they fixed it:

http://forge.typo3.org/issues/28497

Thanks,

Mark

comment:6 Changed 10 years ago by Jakub Ś

Resolution: fixed
Status: confirmedclosed

I don't see this problem in CKEditro 4.x. I'm closing issue as Fixed.

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