Opened 10 years ago

Last modified 10 years ago

#9797 closed enhancement

[patch] [cla] Add Target option to LinkDialog plugin for Dijit.Editor — at Version 3

Reported by: Nicholas VanderPyle Owned by:
Priority: high Milestone: 1.4
Component: Editor Version: 1.3.2
Keywords: dijit editor plugin link dialog target Cc: Jared Jurkiewicz
Blocked By: Blocking:

Description (last modified by bill)

All links in Dijit.Editor use the default link target behavior. It would be nice to have an option to change the target.

I've added a Target dropdown to the LinkDialog.js plugin and attached a patch.

It appears that the file is reused for InsertImage? plugin. The patch that I've attached will also show the Target dropdown when editing an image url, too.

The text in the target dropdown isn't internationalized or pulled from an NLS.

Change History (4)

Changed 10 years ago by Nicholas VanderPyle

Patch created from Subclipse in Aptana to add Target option to the LinkDialog?.

comment:1 Changed 10 years ago by bill

Summary: Add Target option to LinkDialog plugin for Dijit.Editor[patch] [no cla] Add Target option to LinkDialog plugin for Dijit.Editor

We can't look at patches unless you file a CLA, see http://dojofoundation.org/cla/, thanks. (I don't see your name under http://dojotoolkit.org/dojo-contributors)

comment:2 Changed 10 years ago by Nicholas VanderPyle

I received confirmation for my signed CLA before submitting the ticket. I just just updated my full name in the LDAP Entry of dojotoolkit.org and on the Preferences page here. Where should I go to change my name name under this account so that it reflects the one listed on the CLA site?

comment:3 Changed 10 years ago by bill

Description: modified (diff)
Summary: [patch] [no cla] Add Target option to LinkDialog plugin for Dijit.Editor[patch] [cla] Add Target option to LinkDialog plugin for Dijit.Editor

Oh OK great, your name should be added to the list automatically, just hasn't happened yet.

Note: See TracTickets for help on using tickets.