Optional open redirection page in new window

OFF - February 7, 2009 - 15:19
Project:External Link Page
Version:6.x-1.0
Component:Code
Category:feature request
Priority:normal
Assigned:zzolo
Status:closed
Description

I would like the reference to redirect page opened in a new window.

I mean this link (http://site.com/ext_link?url=http://www.other-site.com/) will have attribute target="_blank".

Well to make this possibility optional

#1

OFF - February 7, 2009 - 15:30
Assigned to:Anonymous» OFF

I have solved this problem, having edited a code

On line 220

I have replaced a line

return 'href="'. $url .'"';

with

return 'target="_blank" href="'. $url .'"';

I would like to see this change in the final version of the module, but I understand that the given variant will approach not to all.

If there is a possibility to make it on a choice, I would be very grateful!

#2

zzolo - February 19, 2009 - 17:30

@OFF

This is probably a good feature to have. There are two ways to implement this: 1) Have more options to theme the link and other parts, or 2) Add some administrative options. Or both.

It may be some time before I actually get around to enhancing this module, but it sounds like you found a solution.

--
zzolo

#3

zzolo - February 19, 2009 - 17:31
Assigned to:OFF» zzolo

#4

OFF - February 20, 2009 - 14:08

And what I must do now?

#5

zzolo - March 19, 2009 - 01:38

If you want, you can create a patch. http://drupal.org/patch/create

Or you can just wait until, I get around to doing some work on this module.

#6

OFF - March 23, 2009 - 12:44

my english very bad..

i just wait.. thanks

#7

zzolo - October 17, 2009 - 06:39
Status:active» fixed

This feature has been added as an admin option.

#8

System Message - October 31, 2009 - 06:40
Status:fixed» closed

Automatically closed -- issue fixed for 2 weeks with no activity.

 
 

Drupal is a registered trademark of Dries Buytaert.