[TASK] Remove TBE_STYLES mainColors from PHP 68/33168/2
authorBenjamin Mack <benni@typo3.org>
Sun, 5 Oct 2014 08:20:07 +0000 (10:20 +0200)
committerBenjamin Mack <benni@typo3.org>
Sun, 5 Oct 2014 20:03:03 +0000 (22:03 +0200)
TYPO3 uses $TBE_STYLES to define certain
variables like background colors in PHP that
can be overridden with extensions.

In order to streamline the rendering of the
backend that should completely done via
LESS / CSS, the corresponding PHP parts are
removed, as they should have no effect
anyway - this is already done via CSS.

Releases: master
Resolves: #62039
Change-Id: I81533c1d2912fc25b96f106974e5cd625b827c90
Reviewed-on: http://review.typo3.org/33168
Reviewed-by: Felix Kopp <felix-source@phorax.com>
Tested-by: Felix Kopp <felix-source@phorax.com>
typo3/sysext/backend/Classes/Controller/Wizard/RteController.php
typo3/sysext/backend/Classes/Form/FormEngine.php
typo3/sysext/backend/Classes/Template/DocumentTemplate.php
typo3/sysext/core/Classes/Database/QueryGenerator.php
typo3/sysext/core/Documentation/Changelog/master/Breaking-62039-RemovedTBE_StylesMainColors.rst [new file with mode: 0644]
typo3/sysext/t3skin/Classes/Hook/StyleGenerationHook.php
typo3/sysext/t3skin/ext_tables.php

index bddc278..61a2658 100644 (file)
@@ -132,8 +132,6 @@ class RteController {
                        $tceforms->initDefaultBEMode();
                        // SPECIAL: Disables all wizards - we are NOT going to need them.
                        $tceforms->disableWizards = 1;
-                       // SPECIAL: Setting background color of the RTE to ordinary background
-                       $tceforms->colorScheme[0] = $this->doc->bgColor;
                        // Initialize style for RTE object:
                        // Getting reference to the RTE object used to render the field!
                        $RTEobj = BackendUtility::RTEgetObj();
index 47857cd..8dd0dcf 100644 (file)
@@ -736,18 +736,6 @@ class FormEngine {
                        $this->commentMessages[] = 'RTE NOT ENABLED IN SYSTEM due to:' . LF . $this->RTEenabled_notReasons;
                }
                // Default color+class scheme
-               $docTemplate = $this->getControllerDocumentTemplate();
-               $this->defColorScheme = array(
-                       $docTemplate->bgColor,
-                       // Background for the field AND palette
-                       GeneralUtility::modifyHTMLColorAll($docTemplate->bgColor, -20),
-                       // Background for the field header
-                       GeneralUtility::modifyHTMLColorAll($docTemplate->bgColor, -10),
-                       // Background for the palette field header
-                       'black',
-                       // Field header font color
-                       '#666666'
-               );
                $this->defColorScheme = array();
                // Override / Setting defaults from TBE_STYLES array
                $this->resetSchemes();
index d1d7dda..76dc10c 100644 (file)
@@ -150,42 +150,42 @@ function jumpToUrl(URL) {
        // If you want to change this, please do so through a skin using the global var $GLOBALS['TBE_STYLES']
        // Light background color
        /**
-        * @todo Define visibility
+        * @deprecated since TYPO3 CMS 7, will be removed in CMS 8
         */
        public $bgColor = '#F7F3EF';
 
        // Steel-blue
        /**
-        * @todo Define visibility
+        * @deprecated since TYPO3 CMS 7, will be removed in CMS 8
         */
        public $bgColor2 = '#9BA1A8';
 
        // dok.color
        /**
-        * @todo Define visibility
+        * @deprecated since TYPO3 CMS 7, will be removed in CMS 8
         */
        public $bgColor3 = '#F6F2E6';
 
        // light tablerow background, brownish
        /**
-        * @todo Define visibility
+        * @deprecated since TYPO3 CMS 7, will be removed in CMS 8
         */
        public $bgColor4 = '#D9D5C9';
 
        // light tablerow background, greenish
        /**
-        * @todo Define visibility
+        * @deprecated since TYPO3 CMS 7, will be removed in CMS 8
         */
        public $bgColor5 = '#ABBBB4';
 
        // light tablerow background, yellowish, for section headers. Light.
        /**
-        * @todo Define visibility
+        * @deprecated since TYPO3 CMS 7, will be removed in CMS 8
         */
        public $bgColor6 = '#E7DBA8';
 
        /**
-        * @todo Define visibility
+        * @deprecated since TYPO3 CMS 7, will be removed in CMS 8
         */
        public $hoverColor = '#254D7B';
 
@@ -351,31 +351,6 @@ function jumpToUrl(URL) {
                        // Have to unset - otherwise the second instantiation will do it again!
                        unset($GLOBALS['TBE_STYLES']['scriptIDindex'][$this->scriptID]);
                }
-               // Color scheme:
-               if ($GLOBALS['TBE_STYLES']['mainColors']['bgColor']) {
-                       $this->bgColor = $GLOBALS['TBE_STYLES']['mainColors']['bgColor'];
-               }
-               if ($GLOBALS['TBE_STYLES']['mainColors']['bgColor1']) {
-                       $this->bgColor1 = $GLOBALS['TBE_STYLES']['mainColors']['bgColor1'];
-               }
-               if ($GLOBALS['TBE_STYLES']['mainColors']['bgColor2']) {
-                       $this->bgColor2 = $GLOBALS['TBE_STYLES']['mainColors']['bgColor2'];
-               }
-               if ($GLOBALS['TBE_STYLES']['mainColors']['bgColor3']) {
-                       $this->bgColor3 = $GLOBALS['TBE_STYLES']['mainColors']['bgColor3'];
-               }
-               if ($GLOBALS['TBE_STYLES']['mainColors']['bgColor4']) {
-                       $this->bgColor4 = $GLOBALS['TBE_STYLES']['mainColors']['bgColor4'];
-               }
-               if ($GLOBALS['TBE_STYLES']['mainColors']['bgColor5']) {
-                       $this->bgColor5 = $GLOBALS['TBE_STYLES']['mainColors']['bgColor5'];
-               }
-               if ($GLOBALS['TBE_STYLES']['mainColors']['bgColor6']) {
-                       $this->bgColor6 = $GLOBALS['TBE_STYLES']['mainColors']['bgColor6'];
-               }
-               if ($GLOBALS['TBE_STYLES']['mainColors']['hoverColor']) {
-                       $this->hoverColor = $GLOBALS['TBE_STYLES']['mainColors']['hoverColor'];
-               }
                // Main Stylesheets:
                if ($GLOBALS['TBE_STYLES']['stylesheet']) {
                        $this->styleSheetFile = $GLOBALS['TBE_STYLES']['stylesheet'];
index 430039e..561f658 100644 (file)
@@ -950,9 +950,8 @@ class QueryGenerator {
                        $indent = '<td style="vertical-align:top;"><img height="1" width="50"></td>';
                }
                $lf = $l * 30;
-               $bgColor = \TYPO3\CMS\Core\Utility\GeneralUtility::modifyHTMLColor($GLOBALS['TBE_TEMPLATE']->bgColor2, $lf, $lf, $lf);
                foreach ($codeArr as $k => $v) {
-                       $line .= '<tr>' . $indent . '<td bgcolor="' . $bgColor . '"' . $this->noWrap . '>' . $v['html'] . '</td></tr>';
+                       $line .= '<tr>' . $indent . '<td class="bgColor2" ' . $this->noWrap . '>' . $v['html'] . '</td></tr>';
                        if ($this->enableQueryParts) {
                                $line .= '<tr>' . $indent . '<td>' . $this->formatQ($v['query']) . '</td></tr>';
                        }
diff --git a/typo3/sysext/core/Documentation/Changelog/master/Breaking-62039-RemovedTBE_StylesMainColors.rst b/typo3/sysext/core/Documentation/Changelog/master/Breaking-62039-RemovedTBE_StylesMainColors.rst
new file mode 100644 (file)
index 0000000..d033edd
--- /dev/null
@@ -0,0 +1,28 @@
+=================================================
+Breaking: #62039 - Removed TBE_STYLES[mainColors]
+=================================================
+
+Description
+===========
+
+The values within $TBE_STYLES[mainColors] are redundant and can be completely defined via CSS nowadays. The
+corresponding PHP leftovers are removed from the core and have no effect anymore.
+
+
+Impact
+======
+
+Setting the variables within $TBE_STYLES[mainColors] and using the $doc->bgColor* and $doc->hoverColor properties
+of DocumentTemplate have no effect anymore.
+
+
+Affected installations
+======================
+
+Any installation using an extension that is overriding skin info via $TBE_STYLES[mainColors].
+
+
+Migration
+=========
+
+Use CSS directly to modify the appearance of the Backend.
index 25e7278..49d60d3 100644 (file)
@@ -18,6 +18,7 @@ namespace TYPO3\CMS\T3skin\Hook;
  * Hook for adding styles to backend page-generation in DocumentTemplate
  *
  * @author Stefan Neufeind <info [at] speedpartner.de>
+ * @deprecated since TYPO3 CMS 7, will be removed in CMS 8
  */
 class StyleGenerationHook {
 
index 4807434..d868d92 100644 (file)
@@ -14,22 +14,6 @@ if (TYPO3_MODE === 'BE' || TYPO3_MODE === 'FE' && isset($GLOBALS['BE_USER'])) {
        $TBE_STYLES['skins'][$_EXTKEY]['stylesheetDirectories']['sprites'] = 'EXT:t3skin/stylesheets/sprites/';
 
        /** Setting up backend styles and colors */
-       $TBE_STYLES['mainColors'] = array(
-               // Always use #xxxxxx color definitions!
-               'bgColor' => '#FFFFFF',
-               // Light background color
-               'bgColor2' => '#FEFEFE',
-               // Steel-blue
-               'bgColor3' => '#F1F3F5',
-               // dok.color
-               'bgColor4' => '#E6E9EB',
-               // light tablerow background, brownish
-               'bgColor5' => '#F8F9FB',
-               // light tablerow background, greenish
-               'bgColor6' => '#E6E9EB',
-               // light tablerow background, yellowish, for section headers. Light.
-               'hoverColor' => '#FF0000'
-       );
        $TBE_STYLES['colorschemes'][0] = '-|class-main1,-|class-main2,-|class-main3,-|class-main4,-|class-main5';
        $TBE_STYLES['colorschemes'][1] = '-|class-main11,-|class-main12,-|class-main13,-|class-main14,-|class-main15';
        $TBE_STYLES['colorschemes'][2] = '-|class-main21,-|class-main22,-|class-main23,-|class-main24,-|class-main25';
@@ -80,10 +64,6 @@ if (TYPO3_MODE === 'BE' || TYPO3_MODE === 'FE' && isset($GLOBALS['BE_USER'])) {
        // Default navigation frame width
        $TBE_STYLES['dims']['navFrameWidth'] = 280;
 
-       // Setting roll-over background color for click menus:
-       // Notice, this line uses the the 'scriptIDindex' feature to override another value in this array (namely $TBE_STYLES['mainColors']['bgColor5']), for a specific script "typo3/alt_clickmenu.php"
-       $TBE_STYLES['scriptIDindex']['typo3/alt_clickmenu.php']['mainColors']['bgColor5'] = '#dedede';
-
        // Setting up auto detection of alternative icons:
        $TBE_STYLES['skinImgAutoCfg'] = array(
                'absDir' => \TYPO3\CMS\Core\Utility\ExtensionManagementUtility::extPath($_EXTKEY) . 'icons/',