[!!!][TASK] Remove the TS option page.bodyTagMargins 48/54548/2
authorGeorg Ringer <georg.ringer@gmail.com>
Fri, 3 Nov 2017 17:39:37 +0000 (18:39 +0100)
committerFrans Saris <franssaris@gmail.com>
Sat, 4 Nov 2017 23:40:24 +0000 (00:40 +0100)
The option 'page.bodyTagMargins' is rarly used and can be removed.
Any style information should be placed in a CSS file.

Resolves: #82914
Releases: master
Change-Id: Ib34ae835d0e50653b70f7ecf9573f42fbcca83a6
Reviewed-on: https://review.typo3.org/54548
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Morton Jonuschat <m.jonuschat@mojocode.de>
Tested-by: Morton Jonuschat <m.jonuschat@mojocode.de>
Reviewed-by: Frans Saris <franssaris@gmail.com>
Tested-by: Frans Saris <franssaris@gmail.com>
typo3/sysext/core/Documentation/Changelog/master/Breaking-82914-RemoveTypoScriptOptionPagebodyTagMargins.rst [new file with mode: 0644]
typo3/sysext/frontend/Classes/Page/PageGenerator.php
typo3/sysext/t3editor/Resources/Private/tsref.xml
typo3/sysext/t3editor/Resources/Public/JavaScript/Mode/typoscript/typoscript.js

diff --git a/typo3/sysext/core/Documentation/Changelog/master/Breaking-82914-RemoveTypoScriptOptionPagebodyTagMargins.rst b/typo3/sysext/core/Documentation/Changelog/master/Breaking-82914-RemoveTypoScriptOptionPagebodyTagMargins.rst
new file mode 100644 (file)
index 0000000..363481c
--- /dev/null
@@ -0,0 +1,32 @@
+.. include:: ../../Includes.txt
+
+===============================================================
+Breaking: #82914 - Remove TypoScript option page.bodyTagMargins
+===============================================================
+
+See :issue:`82914`
+
+Description
+===========
+
+The TypoScript option `page.bodyTagMargins` has been removed.
+
+
+Impact
+======
+
+Setting this option will have no effect anymore.
+
+
+Affected Installations
+======================
+
+Any TYPO3 installation using this option.
+
+
+Migration
+=========
+
+Move the configuration into your used CSS files.
+
+.. index:: NotScanned
index 09b5202..3b7d807 100644 (file)
@@ -348,16 +348,8 @@ class PageGenerator
             }
         }
 
-        // Stylesheets
-        $style = '';
-        // Setting body tag margins in CSS:
-        if (isset($tsfe->pSetup['bodyTagMargins']) && $tsfe->pSetup['bodyTagMargins.']['useCSS']) {
-            $margins = (int)$tsfe->pSetup['bodyTagMargins'];
-            $style .= '
-       BODY {margin: ' . $margins . 'px ' . $margins . 'px ' . $margins . 'px ' . $margins . 'px;}';
-        }
         // CSS_inlineStyle from TS
-        $style .= trim($tsfe->pSetup['CSS_inlineStyle']);
+        $style = trim($tsfe->pSetup['CSS_inlineStyle']);
         $style .= $tsfe->cObj->cObjGet($tsfe->pSetup['cssInline.'], 'cssInline.');
         if (trim($style)) {
             self::addCssToPageRenderer($style, true, 'additionalTSFEInlineStyle');
@@ -733,13 +725,6 @@ class PageGenerator
         } else {
             $defBT = $tsfe->pSetup['bodyTagCObject'] ? $tsfe->cObj->cObjGetSingle($tsfe->pSetup['bodyTagCObject'], $tsfe->pSetup['bodyTagCObject.'], 'bodyTagCObject') : '<body>';
             $bodyTag = $tsfe->pSetup['bodyTag'] ? $tsfe->pSetup['bodyTag'] : $defBT;
-            if (isset($tsfe->pSetup['bodyTagMargins'])) {
-                $margins = (int)$tsfe->pSetup['bodyTagMargins'];
-                if ($tsfe->pSetup['bodyTagMargins.']['useCSS']) {
-                } else {
-                    $bodyTag = preg_replace('/>$/', '', trim($bodyTag)) . ' leftmargin="' . $margins . '" topmargin="' . $margins . '" marginwidth="' . $margins . '" marginheight="' . $margins . '">';
-                }
-            }
             if (trim($tsfe->pSetup['bodyTagAdd'])) {
                 $bodyTag = preg_replace('/>$/', '', trim($bodyTag)) . ' ' . trim($tsfe->pSetup['bodyTagAdd']) . '>';
             }
index bbf1a28..313016c 100644 (file)
@@ -4851,18 +4851,6 @@ page.bodyTag = <body bgcolor="{$bgCol}">]]></description>
                        <default><![CDATA[
 ]]></default>
                </property>
-               <property name="bodyTagMargins" type="int">
-                       <description><![CDATA[margins in the bodytag.
-
-Property:
-.useCSS = 1 (boolean) - will set a "BODY {margin: ...}" line in the in-document style declaration - for XHTML compliance.
-
-Example:
-value 4
-adds leftmargin="4" topmargin="4" marginwidth="4" marginheight="4"  to the bodyTag.]]></description>
-                       <default><![CDATA[
-]]></default>
-               </property>
                <property name="cssInline" type="CARRAY">
                        <description><![CDATA[Use cObjects for creating inline CSS
 
index 8d585a7..ed63c4b 100644 (file)
                                'bodyTag': kw('bodyTag'),
                                'bodyTagAdd': kw('bodyTagAdd'),
                                'bodyTagCObject': kw('bodyTagCObject'),
-                               'bodyTagMargins': kw('bodyTagMargins'),
                                'bodytext': kw('bodytext'),
                                'border': kw('border'),
                                'borderCol': kw('borderCol'),
                        }
                };
        });
-});
\ No newline at end of file
+});