Nobody likes multiple windows but since there is no way to navigate back to you drupal site (other than the back button) from Flickr it would really be preferable to have all links to Flickr go to new windows. If not always then at least provide an option or tell me where I can add the 'target="_blank" for customization...

Comments

drewish’s picture

drewish commented

i'd be into adding an option for this...

gan_drupal’s picture

gan_drupal commented

Hi,

return l($img, $photo_url, array('target' => '_blank'), NULL, NULL, TRUE, TRUE);

replace the above line with

return l($img, $photo_url, array('target' => '_blank'), array('title' => $title), NULL, NULL, TRUE, TRUE);

under the function "theme_flickr_photoset($ps, $owner, $size, $attribs = NULL)".

probably this could be the last line of your flickr.module.

Pear Head’s picture

Pear Head commented

I don't know about v6, but for v5 that doesn't work. This is what I used:

return l($img, $photo_url, array('title' => $title, 'target' => '_blank'), NULL, NULL, TRUE, TRUE);

I also had to change it under the theme_flickr_photo function.

gan_drupal’s picture

gan_drupal commented

Hey i'm also using v5, it works fine in both the functions [theme_flickr_photo and theme_flickr_photosets] !!

andrewlevine’s picture

andrewlevine commented

If someone creates a patch for this we can get it committed.

Rowanw’s picture

Rowanw commented

To avoid adding this same option to every module in Drupal, you should use something like the External Links module.

ksenzee’s picture

ksenzee
she/her
commented

The External Links module does the trick. Another great option is Lightbox2. It has built-in Flickr module functionality. Just install version 2 of the module, and your Flickr links will automagically turn into beautiful Lightbox images, keeping users on your site and happy.

Anonymous’s picture

Anonymous (not verified) commented
Status: Active » Closed (fixed)

Nothing more need be said :-)

Although i also said this ..
http://drupal.org/node/140825