[BUGFIX] Fix RTE width 74/31274/2
authorMarkus Klein <klein.t3@mfc-linz.at>
Fri, 20 Jun 2014 16:58:45 +0000 (18:58 +0200)
committerWouter Wolters <typo3@wouterwolters.nl>
Tue, 1 Jul 2014 20:03:27 +0000 (22:03 +0200)
edit_wideDocument option has been removed with #57295 but it was forgotten
to adjust RTE accordingly.

Wide screens are default now, so we remove the check for this option
also in RTE. Furthermore the settings for small screens are superfluous
too.

Documentation is adjusted as well.

Resolves: #59692
Releases: 6.3, 6.2
Change-Id: I9bb5fc2a892f3de446137856e9fa18317f40b5c1
Reviewed-on: https://review.typo3.org/31274
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
typo3/sysext/rtehtmlarea/Classes/Controller/FrontendRteController.php
typo3/sysext/rtehtmlarea/Classes/RteHtmlAreaBase.php
typo3/sysext/rtehtmlarea/Documentation/Configuration/UserTsconfig/Index.rst

index d90db1d..2c08853 100644 (file)
@@ -207,7 +207,7 @@ class FrontendRteController extends \TYPO3\CMS\Rtehtmlarea\RteHtmlAreaBase {
                 * SET STYLES
                 * =======================================
                 */
-               $width = 460 + ($this->TCEform->docLarge ? 150 : 0);
+               $width = 610;
                if (isset($this->thisConfig['RTEWidthOverride'])) {
                        if (strstr($this->thisConfig['RTEWidthOverride'], '%')) {
                                if ($this->client['browser'] != 'msie') {
index cea8dad..1fe7a89 100644 (file)
@@ -14,6 +14,7 @@ namespace TYPO3\CMS\Rtehtmlarea;
  * The TYPO3 project - inspiring people to share!
  */
 
+use TYPO3\CMS\Backend\Form\FormEngine;
 use TYPO3\CMS\Backend\Utility\BackendUtility;
 use TYPO3\CMS\Core\Utility\GeneralUtility;
 
@@ -320,21 +321,21 @@ class RteHtmlAreaBase extends \TYPO3\CMS\Backend\Rte\AbstractRte {
        /**
         * Draws the RTE as an iframe
         *
-        * @param       object          Reference to parent object, which is an instance of the TCEforms.
-        * @param       string          The table name
-        * @param       string          The field name
-        * @param       array           The current row from which field is being rendered
-        * @param       array           Array of standard content for rendering form fields from TCEforms. See TCEforms for details on this. Includes for instance the value and the form field name, java script actions and more.
-        * @param       array           "special" configuration - what is found at position 4 in the types configuration of a field from record, parsed into an array.
-        * @param       array           Configuration for RTEs; A mix between TSconfig and otherwise. Contains configuration for display, which buttons are enabled, additional transformation information etc.
-        * @param       string          Record "type" field value.
-        * @param       string          Relative path for images/links in RTE; this is used when the RTE edits content from static files where the path of such media has to be transformed forth and back!
-        * @param       integer         PID value of record (true parent page id)
-        * @return      string          HTML code for RTE!
+        * @param FormEngine $parentObject Reference to parent object, which is an instance of the TCEforms.
+        * @param string $table The table name
+        * @param string $field The field name
+        * @param array $row The current row from which field is being rendered
+        * @param array $PA Array of standard content for rendering form fields from TCEforms. See TCEforms for details on this. Includes for instance the value and the form field name, java script actions and more.
+        * @param array $specConf "special" configuration - what is found at position 4 in the types configuration of a field from record, parsed into an array.
+        * @param array $thisConfig Configuration for RTEs; A mix between TSconfig and otherwise. Contains configuration for display, which buttons are enabled, additional transformation information etc.
+        * @param string $RTEtypeVal Record "type" field value.
+        * @param string $RTErelPath Relative path for images/links in RTE; this is used when the RTE edits content from static files where the path of such media has to be transformed forth and back!
+        * @param integer $thePidValue PID value of record (true parent page id)
+        * @return string HTML code for RTE!
         * @todo Define visibility
         */
-       public function drawRTE(&$parentObject, $table, $field, $row, $PA, $specConf, $thisConfig, $RTEtypeVal, $RTErelPath, $thePidValue) {
-               global $LANG, $TYPO3_DB, $TYPO3_CONF_VARS;
+       public function drawRTE($parentObject, $table, $field, $row, $PA, $specConf, $thisConfig, $RTEtypeVal, $RTErelPath, $thePidValue) {
+               global $LANG, $TYPO3_DB;
                $this->TCEform = $parentObject;
                $inline = $this->TCEform->inline;
                $LANG->includeLLFile('EXT:' . $this->ID . '/locallang.xml');
@@ -441,9 +442,8 @@ class RteHtmlAreaBase extends \TYPO3\CMS\Backend\Rte\AbstractRte {
                                $RTEPaddingRight = '0';
                                $editorWrapWidth = '100%';
                        } else {
-                               $RTEWidth = isset($GLOBALS['BE_USER']->userTS['options.']['RTESmallWidth']) ? $GLOBALS['BE_USER']->userTS['options.']['RTESmallWidth'] : '530';
-                               $RTEHeight = isset($GLOBALS['BE_USER']->userTS['options.']['RTESmallHeight']) ? $GLOBALS['BE_USER']->userTS['options.']['RTESmallHeight'] : '380';
-                               $RTEWidth = $RTEWidth + ($this->TCEform->docLarge ? (isset($GLOBALS['BE_USER']->userTS['options.']['RTELargeWidthIncrement']) ? $GLOBALS['BE_USER']->userTS['options.']['RTELargeWidthIncrement'] : '150') : 0);
+                               $options = $GLOBALS['BE_USER']->userTS['options.'];
+                               $RTEWidth = 530 + (isset($options['RTELargeWidthIncrement']) ? (int)$options['RTELargeWidthIncrement'] : 150);
                                $RTEWidth -= $inline->getStructureDepth() > 0 ? ($inline->getStructureDepth() + 1) * $inline->getLevelMargin() : 0;
                                $RTEWidthOverride = is_object($GLOBALS['BE_USER']) && isset($GLOBALS['BE_USER']->uc['rteWidth']) && trim($GLOBALS['BE_USER']->uc['rteWidth']) ? trim($GLOBALS['BE_USER']->uc['rteWidth']) : trim($this->thisConfig['RTEWidthOverride']);
                                if ($RTEWidthOverride) {
@@ -456,7 +456,7 @@ class RteHtmlAreaBase extends \TYPO3\CMS\Backend\Rte\AbstractRte {
                                        }
                                }
                                $RTEWidth = strstr($RTEWidth, '%') ? $RTEWidth : $RTEWidth . 'px';
-                               $RTEHeight = $RTEHeight + ($this->TCEform->docLarge ? (isset($GLOBALS['BE_USER']->userTS['options.']['RTELargeHeightIncrement']) ? $GLOBALS['BE_USER']->userTS['options.']['RTELargeHeightIncrement'] : 0) : 0);
+                               $RTEHeight = 380 + (isset($options['RTELargeHeightIncrement']) ? (int)$options['RTELargeHeightIncrement'] : 0);
                                $RTEHeightOverride = is_object($GLOBALS['BE_USER']) && isset($GLOBALS['BE_USER']->uc['rteHeight']) && (int)$GLOBALS['BE_USER']->uc['rteHeight'] ? (int)$GLOBALS['BE_USER']->uc['rteHeight'] : (int)$this->thisConfig['RTEHeightOverride'];
                                $RTEHeight = $RTEHeightOverride > 0 ? $RTEHeightOverride : $RTEHeight;
                                $RTEPaddingRight = '2px';
index f4cbc32..4e4ab17 100644 (file)
@@ -409,29 +409,6 @@ options.noThumbsInRTEimageSelect
 
 
 
-.. _options-rtesmallwidth:
-
-options.RTESmallWidth
-^^^^^^^^^^^^^^^^^^^^^
-
-.. container:: table-row
-
-   Property
-         options.RTESmallWidth
-
-         options.RTESmallHeight
-
-   Data type
-         pixels
-
-   Description
-         Width and height of the editor area when edit\_wideDocument is NOT
-         set.
-
-         Default: RTESmallWidth = 530, RTESmallHeight = 380
-
-
-
 .. _options-rtelargewidthincrement:
 
 options.RTELargeWidthIncrement
@@ -448,8 +425,7 @@ options.RTELargeWidthIncrement
          pixels
 
    Description
-         Increments applied to the width and height of the editor area when
-         edit\_wideDocument IS set.
+         Increments applied to the width and height of the editor area
 
          Default: RTELargeWidthIncrement= 150, RTELargeHeilghtIncrement = 0