Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
It is very annoying if you need to remember to always right-click on that link to open it in a new tab or window, so you don't go out of Hostmaster site. This should be forced by default.
Comments
Comment #1
omega8cc CreditAttribution: omega8cc commentedSimple patch for review: http://drupalcode.org/sandbox/omega8cc/1074912.git/commit/4035462463464f...
Comment #2
ergonlogicAren't we better off adding
target="_blank"
?Comment #3
omega8cc CreditAttribution: omega8cc commentedYeah, we could, even if the target method is no longer a valid XHTML. However, why to use invalid method if the Aegir interface requires javascript enabled anyway?
Comment #4
ergonlogicIt's back in HTML5: http://www.w3.org/TR/html5/browsers.html#valid-browsing-context-name-or-...
Comment #5
anarcat CreditAttribution: anarcat commentedThis should be themed, not forced by default. Besides, we should look at how this is done elsewhere in Drupal.
Comment #6
ergonlogicFixed in dc74e22.
I added a theme function, as per #5, and used target='_blank', since it's more commonly used in Drupal core.
Comment #7
anarcat CreditAttribution: anarcat commenteddamn you, i hate popups. oh well. ;)