Use window.open instead of adding target=_blank

Standart - October 9, 2009 - 13:36
Project:External links filter
Version:6.x-2.x-dev
Component:Code
Category:task
Priority:minor
Assigned:Unassigned
Status:closed
Description

We don't use target=_blank because it's not valid (X)HTML strict, right? So why utilizing JavaScript to add this piece of code to the DOM afterwards when we have window.open?

AttachmentSize
window.open-instead-of-target-_blank.patch976 bytes

#1

Xano - October 10, 2009 - 23:30
Status:active» fixed

Fixed and committed to 5.x-2.x-dev and 6.x-2.x-dev. Thanks a lot for the patch! :)

#2

System Message - October 24, 2009 - 23:30
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.