Posted by Standart on October 9, 2009 at 1:36pm
3 followers
Jump to:
| Project: | External Links Filter |
| Version: | 6.x-2.x-dev |
| Component: | Code |
| Category: | task |
| Priority: | minor |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
Issue Summary
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?
| Attachment | Size |
|---|---|
| window.open-instead-of-target-_blank.patch | 976 bytes |
Comments
#1
Fixed and committed to 5.x-2.x-dev and 6.x-2.x-dev. Thanks a lot for the patch! :)
#2
Automatically closed -- issue fixed for 2 weeks with no activity.