Active
Project:
Drupal core
Version:
main
Component:
install system
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
1 Apr 2009 at 09:35 UTC
Updated:
23 Apr 2025 at 05:19 UTC
Jump to comment: Most recent
Comments
Comment #1
dries commentedThe target-attribute will break XHTML 1.0 Strict validation. I don't think that is a critical problem, but there are Javascript alternatives that do validate. Any thoughts? I think this patch is a step in the right direction.
Comment #2
yoroy commentedtagging
Comment #3
Bojhan commentedI think we need a universal solution to this.
Comment #5
Bojhan commentedBojhan: XHTML (which i hate) became modularized and target is a module that needs to be loaded (in the doctype in the HTML header) so that's quite possible to fix
chx: oke great,
Bojhan: http://dionidium.com/tests/extend-xhtml/xhtml-strict
Bojhan: [ ] needs to be added to doctype.
chx: and thats only required on the pages which need this functionality?
Bojhan: right but it does not hurt on others
So lets do this, and apply it where it is REALLY needed. So we can for once fix this damm thing.
Comment #6
dcor commentedI'm not sure if this is related or that we need a new issue for it ---
When following a link off the admin pages.. (i.e. in appearance where it says "alternative themes are available") where "themes" is a link to the main drupal site...
If I click on the link (or shift click to open in a new window) - it takes me to the themes page on drupal - my problem is I can't get this to open in another tab/window (using firefox here).
I think these links (in general - that are all over the place) should automatically open a new tab or window... as otherwise, you're moving away from your site and you have to go back and start all over again to find where you left off.
Hope this makes sense - I'm installing and running D7 properly so as to make up some themes and figure I may as well report on what I find!
Cheers.
Comment #7
tr commentedComment #16
quietone commentedA policy issue has been opened to discuss this problem, #2702881: [policy, no patch] Formalize how external links are handled in core. And since that has not been resolved, changing this to postponed.
Comment #21
quietone commented