Opened 7 years ago

Closed 7 years ago

#4612 closed New Feature (fixed)

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)

  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 (2)

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

Download all attachments as: .zip

Change History (9)

comment:1 Changed 7 years ago by fredck

  • Description modified (diff)

comment:2 Changed 7 years ago by fredck

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

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 7 years ago by m.nguyen

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

Changed 7 years ago by m.nguyen

comment:4 Changed 7 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 7 years ago by m.nguyen

comment:5 Changed 7 years ago by m.nguyen

  • Keywords Review? added; Review- removed

comment:6 Changed 7 years ago by alfonsoml

  • Keywords Review+ added; Review? removed

comment:7 Changed 7 years ago by m.nguyen

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

Fixed with [4997].

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