Revert "[TASK] Configurable width of the Element Browser" 16/29616/2
authorJigal van Hemert <jigal.van.hemert@typo3.org>
Wed, 23 Apr 2014 10:35:45 +0000 (12:35 +0200)
committerJigal van Hemert <jigal.van.hemert@typo3.org>
Wed, 23 Apr 2014 10:37:24 +0000 (12:37 +0200)
Reverted per request of the RM because:
- there is a merge freeze for regressions
- 6.2 is now a stable branch (no new features)

Besides:
- no documentation issue
- not enough votes

This reverts commit 8f520011793f558a5dad33f6a51ecabc5cef41c1

Change-Id: Ia1ffee069f825b678748808d756a2c04fd8edb18
Reviewed-on: https://review.typo3.org/29616
Reviewed-by: Jigal van Hemert
Tested-by: Jigal van Hemert
typo3/sysext/backend/Classes/Form/Element/InlineElement.php
typo3/sysext/backend/Classes/Form/FormEngine.php
typo3/sysext/backend/Resources/Public/JavaScript/FormEngine.js

index bd4e397..cd6cc39 100644 (file)
@@ -885,8 +885,6 @@ class InlineElement {
                $objectPrefix = $this->inlineNames['object'] . self::Structure_Separator . $foreign_table;
                $mode = 'db';
                $showUpload = FALSE;
-               $width = 0;
-               $height = 0;
                if (!empty($conf['appearance']['createNewRelationLinkTitle'])) {
                        $createNewRelationText = $GLOBALS['LANG']->sL($conf['appearance']['createNewRelationLinkTitle'], TRUE);
                } else {
@@ -905,15 +903,9 @@ class InlineElement {
                        if (isset($config['appearance']['elementBrowserAllowed'])) {
                                $allowed = $config['appearance']['elementBrowserAllowed'];
                        }
-                       if (isset($config['appearance']['elementBrowserWidth'])) {
-                               $width = $config['appearance']['elementBrowserWidth'];
-                       }
-                       if (isset($config['appearance']['elementBrowserHeight'])) {
-                               $height = $config['appearance']['elementBrowserHeight'];
-                       }
                }
                $browserParams = '|||' . $allowed . '|' . $objectPrefix . '|inline.checkUniqueElement||inline.importElement';
-               $onClick = 'setFormValueOpenBrowser(\'' . $mode . '\', \'' . $browserParams . '\',' . (int)$width . ',' . (int)$height . '); return false;';
+               $onClick = 'setFormValueOpenBrowser(\'' . $mode . '\', \'' . $browserParams . '\'); return false;';
 
                $item = '<a href="#" class="t3-button" onclick="' . htmlspecialchars($onClick) . '">';
                $item .= IconUtility::getSpriteIcon('actions-insert-record', array('title' => $createNewRelationText));
index e078d68..3072d06 100644 (file)
@@ -2598,20 +2598,8 @@ TBE_EDITOR.customEvalFunctions[\'' . $evalData . '\'] = function(value) {
                                $info .= '<span class="nobr">' . htmlspecialchars($this->getLL('l_allTables')) . '</span><br />';
                        } elseif ($tempFT) {
                                $onlySingleTableAllowed = count($tempFT) == 1;
-
-                               $width = 0;
-                               $height = 0;
-                               if (is_array($config['appearance'])) {
-                                       if (isset($config['appearance']['elementBrowserWidth'])) {
-                                               $width = $config['appearance']['elementBrowserWidth'];
-                                       }
-                                       if (isset($config['appearance']['elementBrowserHeight'])) {
-                                               $height = $config['appearance']['elementBrowserHeight'];
-                                       }
-                               }
-
                                foreach ($tempFT as $theT) {
-                                       $aOnClick = 'setFormValueOpenBrowser(\'db\', \'' . ($PA['itemFormElName'] . '|||' . $theT) . '\',' . (int)$width . ',' . (int)$height . '); return false;';
+                                       $aOnClick = 'setFormValueOpenBrowser(\'db\', \'' . ($PA['itemFormElName'] . '|||' . $theT) . '\'); return false;';
                                        $info .= '<span class="nobr">
                                                                        <a href="#" onclick="' . htmlspecialchars($aOnClick) . '">' . IconUtility::getSpriteIconForRecord($theT, array()) . htmlspecialchars($this->sL($GLOBALS['TCA'][$theT]['ctrl']['title'])) . '</a></span><br />';
                                }
@@ -3919,15 +3907,7 @@ TBE_EDITOR.customEvalFunctions[\'' . $evalData . '\'] = function(value) {
                                } else {
                                        $elementBrowserAllowed = $allowed;
                                }
-                               $width = 0;
-                               $height = 0;
-                               if (isset($config['appearance']['elementBrowserWidth'])) {
-                                       $width = $config['appearance']['elementBrowserWidth'];
-                               }
-                               if (isset($config['appearance']['elementBrowserHeight'])) {
-                                       $height = $config['appearance']['elementBrowserHeight'];
-                               }
-                               $aOnClick = 'setFormValueOpenBrowser(\'' . $elementBrowserType . '\',\'' . ($fName . '|||' . $elementBrowserAllowed . '|' . $aOnClickInline) . '\',' . $width . ',' . $height .'); return false;';
+                               $aOnClick = 'setFormValueOpenBrowser(\'' . $elementBrowserType . '\',\'' . ($fName . '|||' . $elementBrowserAllowed . '|' . $aOnClickInline) . '\'); return false;';
                                $icons['R'][] = '<a href="#" onclick="' . htmlspecialchars($aOnClick) . '">' . IconUtility::getSpriteIcon('actions-insert-record', array('title' => htmlspecialchars($this->getLL(('l_browse_' . ($mode == 'db' ? 'db' : 'file')))))) . '</a>';
                        }
                        if (!$params['dontShowMoveIcons']) {
index 79a49ea..55c5dd6 100644 (file)
@@ -38,14 +38,10 @@ define('TYPO3/CMS/Backend/FormEngine', ['jquery'], function ($) {
         *
         * @param mode can be "db" or "file"
         * @param params additional params for the browser window
-        * @param width width of the window
-        * @param height height of the window
         */
-       FormEngine.openPopupWindow = setFormValueOpenBrowser = function(mode, params, width, height) {
+       FormEngine.openPopupWindow = setFormValueOpenBrowser = function(mode, params) {
                var url = FormEngine.backPath + 'browser.php?mode=' + mode + '&bparams=' + params;
-               width = width ? width : 650;
-               height = height ? height : 650;
-               FormEngine.openedPopupWindow = window.open(url, 'Typo3WinBrowser', 'height=' + height + ',width=' + width + ',status=0,menubar=0,resizable=1,scrollbars=1');
+               FormEngine.openedPopupWindow = window.open(url, 'Typo3WinBrowser', 'height=650,width=' + (mode == 'db' ? 650 : 600) + ',status=0,menubar=0,resizable=1,scrollbars=1');
                FormEngine.openedPopupWindow.focus();
        };
 
@@ -574,4 +570,4 @@ define('TYPO3/CMS/Backend/FormEngine', ['jquery'], function ($) {
 
        // call the main initialize function and execute the hooks
        return initializeModule();
-});
\ No newline at end of file
+});