[TASK] Deprecate page.insertClassesFromRTE 14/51914/4
authorBenni Mack <benni@typo3.org>
Wed, 1 Mar 2017 11:30:18 +0000 (12:30 +0100)
committerChristian Kuhn <lolli@schwarzbu.ch>
Wed, 1 Mar 2017 12:44:06 +0000 (13:44 +0100)
The TypoScript option which fetches TSconfig to
load RTE specific classes and writes inline CSS
has been marked as deprecated, as the cross-dependency
for RTE/pageTSconfig/TypoScript is superseeded by
modern techniques of having a separate frontend from
the backend styling - using contents.css for that
case, and LESS/SASS/preprocessing for frontend
rendering.

Resolves: #80076
Releases: master
Change-Id: I865174823a358252bd3bb47ab4af274d19795472
Reviewed-on: https://review.typo3.org/51914
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
typo3/sysext/core/Documentation/Changelog/master/Deprecation-80076-TypoScriptOptionPageinsertClassesFromRTE.rst [new file with mode: 0644]
typo3/sysext/frontend/Classes/Page/PageGenerator.php

diff --git a/typo3/sysext/core/Documentation/Changelog/master/Deprecation-80076-TypoScriptOptionPageinsertClassesFromRTE.rst b/typo3/sysext/core/Documentation/Changelog/master/Deprecation-80076-TypoScriptOptionPageinsertClassesFromRTE.rst
new file mode 100644 (file)
index 0000000..67beabf
--- /dev/null
@@ -0,0 +1,37 @@
+.. include:: ../../Includes.txt
+
+=================================================================
+Deprecation: #80076 - TypoScript option page.insertClassesFromRTE
+=================================================================
+
+See :issue:`80076`
+
+Description
+===========
+
+The TypoScript setting `page.insertClassesFromRTE` has been marked as deprecated.
+
+The option enabled loading of CSS classes defined by backend PageTs config `RTE.classes` as inline CSS
+into a frontend page. However it did not take merged RTE options and userTS/pageTS overrides
+into account.
+
+
+Impact
+======
+
+Setting `page.insertClassesFromRTE` in TypoScript will trigger a deprecation log entry.
+
+
+Affected Installations
+======================
+
+Any installation having the option activated in TypoScript.
+
+
+Migration
+=========
+
+In order to separate the functionality, all CSS classes which have been used in the RTE should be
+defined separately for the frontend rendering in a custom CSS/LESS/SASS file.
+
+.. index:: TypoScript
\ No newline at end of file
index 414f26d..22ebcfa 100644 (file)
@@ -490,6 +490,13 @@ class PageGenerator
         // Stylesheets
         $style = '';
         if ($tsfe->pSetup['insertClassesFromRTE']) {
+            $tsfe->logDeprecatedTyposcript(
+                'page.insertClassesFromRTE',
+                'Loading CSS classes from the RTE directly is discouraged in TYPO3 v8, as CSS classes should be '
+                . 'defined in CSS/LESS/SASS files instead, ensuring to load only what is necessary for a page, and '
+                . 'speeding up page rendering ("above the fold"). Additionally CSS should be defined in CSS files or '
+                . 'TypoScript and not via magic of pageTSconfig, overlaid by userTSconfig.'
+            );
             $pageTSConfig = $tsfe->getPagesTSconfig();
             $RTEclasses = $pageTSConfig['RTE.']['classes.'];
             if (is_array($RTEclasses)) {