[TASK] Remove unused RTE option "keepButtonGroupTogether" 11/50911/2
authorBenni Mack <benni@typo3.org>
Fri, 9 Dec 2016 05:56:27 +0000 (06:56 +0100)
committerAndreas Fernandez <typo3@scripting-base.de>
Fri, 9 Dec 2016 08:47:52 +0000 (09:47 +0100)
The option "keepButtonGroupTogether" was used before 6.0
and then removed without updating the documentation.

The patch removes the last reference within the PHP file
which is handed over to JS (but never used there) and removes
all documentation on that as well.

Resolves: #78927
Releases: master
Change-Id: Ic03dc10da06c7949d02ae04a566808ec17bdbab3
Reviewed-on: https://review.typo3.org/50911
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Frans Saris <franssaris@gmail.com>
Tested-by: Frans Saris <franssaris@gmail.com>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Tested-by: Andreas Fernandez <typo3@scripting-base.de>
typo3/sysext/rtehtmlarea/Classes/Form/Element/RichTextElement.php
typo3/sysext/rtehtmlarea/Configuration/PageTSconfig/Demo/pageTSConfig.txt
typo3/sysext/rtehtmlarea/Configuration/PageTSconfig/Typical/pageTSConfig.txt
typo3/sysext/rtehtmlarea/Documentation/Configuration/PageTsconfig/interfaceConfiguration/Index.rst
typo3/sysext/rtehtmlarea/Documentation/DefaultConfigurations/Demo/Index.rst
typo3/sysext/rtehtmlarea/Documentation/DefaultConfigurations/Typical/Index.rst

index b9cbc09..0895138 100644 (file)
@@ -412,9 +412,6 @@ class RichTextElement extends AbstractFormElement
     {
         $backendUser = $this->getBackendUserAuthentication();
 
-        if ($this->client['browser'] === 'msie' || $this->client['browser'] === 'opera') {
-            $this->processedRteConfiguration['keepButtonGroupTogether'] = 0;
-        }
         $this->defaultToolbarOrder = 'bar, blockstylelabel, blockstyle, textstylelabel, textstyle, linebreak,
                        bar, formattext, bold,  strong, italic, emphasis, big, small, insertedtext, deletedtext, citation, code,'
                 . 'definition, keyboard, monospaced, quotation, sample, variable, bidioverride, strikethrough, subscript, superscript, underline, span,
@@ -683,7 +680,6 @@ class RichTextElement extends AbstractFormElement
         $jsArray[] = 'RTEarea[editornumber].disableObjectResizing = ' . (trim($this->processedRteConfiguration['disableObjectResizing']) ? 'true;' : 'false;');
         $jsArray[] = 'RTEarea[editornumber].removeTrailingBR = ' . (trim($this->processedRteConfiguration['removeTrailingBR']) ? 'true;' : 'false;');
         $jsArray[] = 'RTEarea[editornumber].useCSS = ' . (trim($this->processedRteConfiguration['useCSS']) ? 'true' : 'false') . ';';
-        $jsArray[] = 'RTEarea[editornumber].keepButtonGroupTogether = ' . (trim($this->processedRteConfiguration['keepButtonGroupTogether']) ? 'true;' : 'false;');
         $jsArray[] = 'RTEarea[editornumber].disablePCexamples = ' . (trim($this->processedRteConfiguration['disablePCexamples']) ? 'true;' : 'false;');
         $jsArray[] = 'RTEarea[editornumber].showTagFreeClasses = ' . (trim($this->processedRteConfiguration['showTagFreeClasses']) ? 'true;' : 'false;');
         $jsArray[] = 'RTEarea[editornumber].tceformsNested = ' . (!empty($this->data) ? json_encode($this->data['tabAndInlineStack']) : '[]') . ';';
index 74a0117..3e6acdb 100644 (file)
@@ -125,9 +125,6 @@ RTE.default {
                ## Toolbar options
        showButtons = *
 
-               ## More toolbar options
-       keepButtonGroupTogether = 1
-
                ## Enable status bar
        showStatusBar =  1
 
index 7fe85c9..51d62b1 100644 (file)
@@ -112,9 +112,6 @@ RTE.default {
        showButtons := addToList(columninsertbefore, columninsertafter, columndelete, columnsplit)
        showButtons := addToList(cellproperties, cellinsertbefore, cellinsertafter, celldelete, cellsplit, cellmerge)
 
-               ## More toolbar options
-       keepButtonGroupTogether = 1
-
                ## Enable status bar
        showStatusBar =  1
 
index 217014c..84f855b 100644 (file)
@@ -192,32 +192,6 @@ toolbarOrder
 
 
 
-.. _keepbuttongrouptogether:
-
-keepButtonGroupTogether
-~~~~~~~~~~~~~~~~~~~~~~~
-
-.. container:: table-row
-
-   Property
-         keepButtonGroupTogether
-
-   Data type
-         boolean
-
-   Description
-         Specifies that all buttons of a button group are displayed on the same
-         line of the tool bar. A button group is delimited by a linebreak or by
-         a bar.
-
-         Default: 0
-
-         Note: If enabled, the setting is honored only by Mozilla/Firefox and
-         Safari. It is ignored when the browser is Internet Explorer, Opera or
-         Mozilla 1.3.
-
-
-
 .. _defaultcontentlanguage:
 
 defaultContentLanguage
index 8e9c1ea..7c997c4 100644 (file)
@@ -148,12 +148,6 @@ The TCA configuration may add buttons to the toolbar
       showButtons := addToList(columninsertbefore, columninsertafter, columndelete, columnsplit)
       showButtons := addToList(cellproperties, cellinsertbefore, cellinsertafter, celldelete, cellsplit, cellmerge)
 
-More toolbar options
-
-::
-
-      keepButtonGroupTogether = 1
-
 Enable status bar
 
 ::