Ticket #4612 (closed New Feature: fixed)

Opened 2 years ago

Last modified 2 years ago

Link text is not updated when link protocol is changed

Reported by: JoeK Owned by: m.nguyen
Priority: Normal Milestone: CKEditor 3.2
Component: General Version: 3.0
Keywords: IBM Review+ Cc: damian.chojna@…

Description (last modified by fredck) (diff)

  1. Click on the Link button to open the Link dialog.
  2. Enter a URL, for example www.bbc.co.uk
  3. Choose  https:// for the protocol and click OK.
  4. Observe that the link is displayed as  https://www.bbc.co.uk
  5. Right click on the link and open the Link dialog.
  6. Update the protocol to  ftp:// and click OK
  7. Observe that the link text remains as  https://www.bbc.co.uk.

If you view the HTML source you can see that the link text has not updated:

<a href="ftp://www.bbc.co.uk">https://www.bbc.co.uk</a>

Attachments

4612.patch Download (2.8 KB) - added by m.nguyen 2 years ago.
4612_2.patch Download (1.1 KB) - added by m.nguyen 2 years ago.

Change History

comment:1 Changed 2 years ago by fredck

  • Description modified (diff)

comment:2 Changed 2 years ago by fredck

  • Type changed from Bug to New Feature
  • Milestone changed from CKEditor 3.1 to CKEditor 3.2

There is no bug here. It's missing feature. The fact is that the dialog doesn't look at the text when changing a link. It just assumes that the text present there is something written by the user; it could be an URL or any other kind of text.

We could consider a new feature that, in the case the link text matches the original href value, the text is also changed when updating the URL.

comment:3 Changed 2 years ago by m.nguyen

  • Keywords Review? added
  • Owner set to m.nguyen
  • Status changed from new to assigned

Changed 2 years ago by m.nguyen

comment:4 Changed 2 years ago by fredck

  • Keywords Review- added; Review? removed

The idea about having a new field for the link text is interesting, but it's out of the scope of this ticket. A dedicated ticket could be opened for that. We should instead provide a simpler solution for it.

Changed 2 years ago by m.nguyen

comment:5 Changed 2 years ago by m.nguyen

  • Keywords Review? added; Review- removed

comment:6 Changed 2 years ago by alfonsoml

  • Keywords Review+ added; Review? removed

comment:7 Changed 2 years ago by m.nguyen

  • Status changed from assigned to closed
  • Resolution set to fixed

Fixed with [4997].

Note: See TracTickets for help on using tickets.
© 2003 – 2011 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy