[TASK] Don't define extTarget = _top as default 97/37897/3
authorBenjamin Mack <benni@typo3.org>
Tue, 17 Mar 2015 08:56:29 +0000 (09:56 +0100)
committerGeorg Ringer <georg.ringer@gmail.com>
Tue, 17 Mar 2015 11:29:47 +0000 (12:29 +0100)
In the past, External links not generated via RTE/parseFunc,
e.g. just plain typolink, had the target="_top" attribute, which
in sites without frames is the same as not setting the target.

As frames are extraordinary these days, the default should be
to not set the extTarget at all by default, but let CSS Styled
Content set the extTarget to blank.

Resolves: #65783
Releases: master
Change-Id: Ifba46110ad4d099691530421c34b54e9da502f69
Reviewed-on: http://review.typo3.org/37897
Reviewed-by: Thomas Schlumberger <thomas@b13.de>
Tested-by: Thomas Schlumberger <thomas@b13.de>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
typo3/sysext/css_styled_content/static/setup.txt
typo3/sysext/frontend/ext_localconf.php

index 3d11d4d..561481d 100644 (file)
@@ -46,8 +46,8 @@ styles.content.editPanelPage {
 
 
 
-
-
+# default link target for external URLs
+config.extTarget = {$styles.content.links.extTarget}
 
 
 
index 3495d09..ee5fa7a 100644 (file)
@@ -10,13 +10,6 @@ if (TYPO3_MODE === 'FE' && !isset($_REQUEST['eID'])) {
        );
 }
 
-\TYPO3\CMS\Core\Utility\ExtensionManagementUtility::addTypoScript(
-       'frontend',
-       'setup',
-       'config.extTarget = _top'
-);
-
-
 if (TYPO3_MODE === 'FE') {
 
        // Register eID provider for showpic