Ticket #7154 (review New Feature)

Opened 3 years ago

Last modified 2 years ago

Add support for a Display Text field to the Link dialog

Reported by: tmonahan Owned by: garry.yao
Priority: Normal Milestone:
Component: UI : Dialogs Version:
Keywords: IBM Cc: damo, satya, jamescun

Description

The current link dialog does not allow users to specify display text for the URL they are inserting. We would like to request support for a Display Text field on the link dialog so that users have this option. This would be consistent with the 'Insert Hyperlink' options in desktop editing programs such as Word, Lotus Symphony etc and has been requested by a lot of users.

Attachments

7154.patch (3.6 KB) - added by garry.yao 3 years ago.

Change History

comment:1 Changed 3 years ago by tmonahan

One further note for this requested feature is that the link dialog does not necessarily need to expose a display text field. It would be sufficient for us if an extension point was added to the existing dialog so that we can add a Display Text field ourselves.

For example, the data object in dialogs\link.js that is used to record entered values such as data.target and data.url, could also support data.text. The onOK function would need to be extended to consume this value so that the entered text value can be applied to the html for the link.

comment:2 Changed 3 years ago by wwalc

  • Status changed from new to confirmed

In Word when inserting a link there is a "Text to display" field available. If some text was initially selected, this field is filled automatically with selected text. If selected content is more complicated, "Text to display" is grayed out and <<Selection in Document>> is shown. When editing exisiting links it works in the same way.

I'd say that this is the way to go. Adding such a field will make the dialog more usable, so there is no need to keep it as a hidden feature.

comment:3 Changed 3 years ago by tmonahan

It's great that you can also see value in this feature. Do you know what milestone this will be targeted for? We would love to see it in the 3.6 release if possible.

Changed 3 years ago by garry.yao

comment:4 Changed 3 years ago by garry.yao

  • Owner set to garry.yao
  • Status changed from confirmed to review
  • Component changed from General to UI : Dialogs

comment:5 Changed 3 years ago by fredck

#8374 is DUP.

comment:6 Changed 3 years ago by luisete.serrano

I have applied the patch to the latest version and when you insert the link does not correctly (insert the url in the text) but when you edit and modify it if it does well.

comment:7 Changed 2 years ago by j.swiderski

Take also this ticket #8715 into account when reviewing this feature.

Also long time ago some other feature was implemented #4612. I'm just leaving this as a note so that one could make sure that this new feature is not causing any conflicts with this old one.

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