Overlay overwrites existing target attribute

I agree.

Committed to CVS HEAD. Thanks.

I also want to use target="_blank" for a link when user is editing content. I don't want to use the same tab/window because they will lose their changes!

I got the same need for using target="_blank" link in the overlay.

In my case i want to create the following link in the help text of a field:
<a href="/node/add" target="_blank">Create new node</a> because I don't want the user to lose the current node edit form while creating another node. (often used when trying to reference a node that does not yet exist)

I have the same need as Yuri. I want to add a button with Form API to a node edit form, wrapping the button with the prefix and suffix attributes with a tag with target="_blank", to open a node addition form in a new window / tab.

I've also experienced this issue.

If the link is external, the target won't be overridden if it already has been set. I can't see why this couldn't be implemented for internal links as well.

Ok, seriously, my apologies.

Third time's the charm!

I didn't test it, but the patch looks good and makes sense. Just needs a quick reroll for Drupal 8...

I've done some limited testing on it and it works :-)

And here's the same patch for Drupal 8.

This looks ready to go, for both Drupal 8 and Drupal 7.

A quick way to test it is to use the Field UI to add some help text to the body field on the node form (and have the text be something like <a href="/node" target="_blank">Home</a>)... I tried that and confirmed that before the patch this link won't open in a new window, but with the patch it will.

D'oh, just realized that the test scenario I came up with is almost the exact same thing Yuri said in #7... Well, it definitely works :)

Thanks! Committed/pushed to 8.x. Moving to 7.x for backport.

D7 backport.

Fixing tag.

What needs happen to get this patch committed to core??

You can help by doing a patch review!

http://drupal.org/patch/review

While testing this check if the overlay stays open after clearing the cache....

#21 works for me, look forward to see this patch committed to 7.x

Hi. Patch in #21 does NOT work for me.

I am trying to add a link through Views actually, writing this code in the Header part of the view:

if (user_access('create page content')) 
  print '<strong><a target="_blank" href="node/add/page">Create new item</a></strong>&nbsp;&nbsp;';

Overlay open in the same direction.

BTW, is was better if I could add destination parameter while opening in the SAME tab, but I couldn't find any documentation of how to do in a simple PHP snippet inside a view. The naive code might be something like this:

if (user_access('create page content')) 
  print '<strong><a href="node/add/page&destination=' . current_path() . '">Create new item</a></strong>&nbsp;&nbsp;';

Any help - either for making it open in a new tab or to redirect back to the original URL - will be appreciated!

Thanks.

I've tested the patch in #21 with an internal link with target="_blank" in a field's help text, as described in #18 (which happens to be exactly my use case).

With the patch applied the link does open in a new window. Codewise the patch is the same as what was committed for D8, so I think this is reviewed and tested.

Re #27: I think you need to write the link like this:

print l('Link text', 'node/add/page', array(
  'attributes' => array(
    'title' => "Title text."),
   'query' => array(
     'destination' => current_path())));

api.drupal.org/api/drupal/includes!common.inc/function/l/7

+1 For this !

Why has it not been committed yet ? The backport patch is almost a year old :-).

Committed to 7.x - thanks! http://drupalcode.org/project/drupal.git/commit/c9d2148

It seems that the patch only works for non-admin links, due to this condition:

if (this.isAdminLink(href))

My requirement is that the user is adding a node, and in the description of one of the fields I want to let them add another type of node, but I don't want them to lose their progress in the current form. Therefore I give it a target of _blank, but because it's an admin link, the target is ignored and it gets opened in the overlay.

I don't know if anything has changed since the patch in 7.22, because it seems to me that #7 and #18 are exactly the same requirement I have and the patch that was committed (#21) does NOT resolve it.

I managed to solve this by adding the overlay-exclude class:

<a class="overlay-exclude" target="_blank" href="node/add/thing">Add a thing</a>

This means all of the overlay-related code is skipped, and the link will simply be opened as normal, using whatever target you give it. Using the overlay-exclude class actually obviates the need for the fix in patch #21.

Rob230, thank you so much for sharing the info about overlay-exclude, it solved my problem in a simple elegant way.