[BUGFIX] Improve styling of TS-Objectbrowsers edit view 64/41964/3
authorDaniel Goerz <ervaude@gmail.com>
Sun, 26 Jul 2015 08:52:45 +0000 (10:52 +0200)
committerMarkus Klein <markus.klein@typo3.org>
Mon, 27 Jul 2015 10:00:46 +0000 (12:00 +0200)
Bootsrap classes are applied to the form and the input fields
of the edit view in the objectbrowser to fit the rest of the
backend styling.
Additionally two unstyled a tags are transformed to buttons.

Change-Id: I845995389d85268b53d7647ff6f49b9591106113
Resolves: #68519
Releases: master
Reviewed-on: http://review.typo3.org/41964
Reviewed-by: Frank Nägler <frank.naegler@typo3.org>
Tested-by: Frank Nägler <frank.naegler@typo3.org>
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
typo3/sysext/tstemplate/Classes/Controller/TypoScriptTemplateInformationModuleFunctionController.php
typo3/sysext/tstemplate/Classes/Controller/TypoScriptTemplateModuleController.php
typo3/sysext/tstemplate/Classes/Controller/TypoScriptTemplateObjectBrowserModuleFunctionController.php
typo3/sysext/tstemplate/Resources/Private/Language/locallang_objbrowser.xlf
typo3/sysext/tstemplate/Resources/Private/Less/styles.less

index 13600d7..9118fc1 100644 (file)
@@ -274,8 +274,8 @@ class TypoScriptTemplateInformationModuleFunctionController extends AbstractFunc
 
                        // Edit all icon:
                        $editOnClick = BackendUtility::editOnClick('&createExtension=0&edit[sys_template][' . $tplRow['uid'] . ']=edit');
-                       $icon = IconUtility::getSpriteIcon('actions-document-open', array('title' => $lang->getLL('editTemplateRecord'))) . $lang->getLL('editTemplateRecord');
-                       $outCode .= '<br /><a href="#" onclick="' . htmlspecialchars($editOnClick) . '"><strong>' . $icon . '</strong></a>';
+                       $icon = IconUtility::getSpriteIcon('actions-document-open', array('title' => $lang->getLL('editTemplateRecord'))) . '&nbsp;' . $lang->getLL('editTemplateRecord');
+                       $outCode .= '<br /><a class="btn btn-default" href="#" onclick="' . htmlspecialchars($editOnClick) . '"><strong>' . $icon . '</strong></a>';
                        $theOutput .= $this->pObj->doc->section('', $outCode);
 
                                // hook after compiling the output
index 6dd8c8e..314989d 100755 (executable)
@@ -163,7 +163,7 @@ class TypoScriptTemplateModuleController extends BaseScriptClass {
                                'template' => 'all'
                        );
                        $aHref = BackendUtility::getModuleUrl('web_ts', $urlParameters);
-                       $this->doc->form = '<form action="' . htmlspecialchars($aHref) . '" method="post" enctype="multipart/form-data" name="editForm">';
+                       $this->doc->form = '<form action="' . htmlspecialchars($aHref) . '" method="post" enctype="multipart/form-data" name="editForm" class="form-inline">';
 
                        // JavaScript
                        $this->doc->JScode = '
index c3273c4..967ea0c 100644 (file)
@@ -330,21 +330,32 @@ class TypoScriptTemplateObjectBrowserModuleFunctionController extends AbstractFu
                        if ($existTemplate) {
                                // Value
                                $out = '';
-                               $out .= htmlspecialchars($this->pObj->sObj) . ' =<br />';
-                               $out .= '<input type="text" name="data[' . htmlspecialchars($this->pObj->sObj) . '][value]" value="' . htmlspecialchars($theSetupValue) . '"' . $documentTemplate->formWidth(40) . ' />';
-                               $out .= '<input class="btn btn-default" type="submit" name="update_value" value="' . $lang->getLL('updateButton') . '" />';
+                               $out .= '<div class="form-group">';
+                               $out .= '       <label>' . htmlspecialchars($this->pObj->sObj) . ' =' . '</label>';
+                               $out .= '       <input class="form-control" type="text" name="data[' . htmlspecialchars($this->pObj->sObj) . '][value]" value="' . htmlspecialchars($theSetupValue) . '"' . $documentTemplate->formWidth(40) . ' />';
+                               $out .= '       <input class="btn btn-default" type="submit" name="update_value" value="' . $lang->getLL('updateButton') . '" />';
+                               $out .= '</div>';
                                $theOutput .= $this->pObj->doc->section($lang->getLL('editProperty'), $out, 0, 0);
                                // Property
-                               $out = '<span class="text-nowrap">' . htmlspecialchars($this->pObj->sObj) . '.';
-                               $out .= '<input type="text" name="data[' . htmlspecialchars($this->pObj->sObj) . '][name]"' . $documentTemplate->formWidth(20) . ' /> = </span><br />';
-                               $out .= '<input type="text" name="data[' . htmlspecialchars($this->pObj->sObj) . '][propertyValue]"' . $documentTemplate->formWidth(40) . ' />';
-                               $out .= '<input class="btn btn-default" type="submit" name="add_property" value="' . $lang->getLL('addButton') . '" />';
+                               $out = '<div class="form-group">';
+                               $out .= '       <label>' . htmlspecialchars($this->pObj->sObj) . '.';
+                               $out .= '               <input class="form-control" type="text" name="data[' . htmlspecialchars($this->pObj->sObj) . '][name]"' . $documentTemplate->formWidth(20) . ' /> = ';
+                               $out .= '       </label>';
+                               $out .= '       <input class="form-control" type="text" name="data[' . htmlspecialchars($this->pObj->sObj) . '][propertyValue]"' . $documentTemplate->formWidth(40) . ' />';
+                               $out .= '       <input class="btn btn-default" type="submit" name="add_property" value="' . $lang->getLL('addButton') . '" />';
+                               $out .= '</div>';
                                $theOutput .= $this->pObj->doc->spacer(20);
                                $theOutput .= $this->pObj->doc->section($lang->getLL('addProperty'), $out, 0, 0);
                                // clear
-                               $out = htmlspecialchars($this->pObj->sObj) . ' <strong>' . $lang->csConvObj->conv_case($lang->charSet, $lang->getLL('clear'), 'toUpper') . '</strong> &nbsp;&nbsp;';
-                               $out .= '<input type="checkbox" name="data[' . htmlspecialchars($this->pObj->sObj) . '][clearValue]" value="1" />';
-                               $out .= '<input class="btn btn-default" type="submit" name="clear_object" value="' . $lang->getLL('clearButton') . '" />';
+                               $out = '<div class="form-group">';
+                               $out .= '       <div class="checkbox">';
+                               $out .= '               <label>';
+                               $out .= '                       ' . htmlspecialchars($this->pObj->sObj) . $lang->csConvObj->conv_case($lang->charSet, $lang->getLL('clear'), 'toUpper');
+                               $out .= '                       <input type="checkbox" name="data[' . htmlspecialchars($this->pObj->sObj) . '][clearValue]" value="1" />';
+                               $out .= '               </label>';
+                               $out .= '               <input class="btn btn-default" type="submit" name="clear_object" value="' . $lang->getLL('clearButton') . '" />';
+                               $out .= '       </div>';
+                               $out .= '</div>';
                                $theOutput .= $this->pObj->doc->spacer(20);
                                $theOutput .= $this->pObj->doc->section($lang->getLL('clearObject'), $out, 0, 0);
                                $theOutput .= $this->pObj->doc->spacer(10);
@@ -375,7 +386,7 @@ class TypoScriptTemplateObjectBrowserModuleFunctionController extends AbstractFu
                        }
                        // back
                        $out = $lang->getLL('back');
-                       $out = '<a href="' . htmlspecialchars($aHref) . '"><strong>' . $out . '</strong></a>';
+                       $out = '<a href="' . htmlspecialchars($aHref) . '" class="btn btn-default"><strong><i class="fa fa-chevron-left"></i>&nbsp;' . $out . '</strong></a>';
                        $theOutput .= $this->pObj->doc->divider(5);
                        $theOutput .= $this->pObj->doc->section('', $out);
                } else {
index 6b619f9..90963b3 100644 (file)
@@ -73,7 +73,7 @@
                                <source>&lt;b&gt;Add key&lt;/b&gt;&lt;/a&gt; "%s" to Object List</source>
                        </trans-unit>
                        <trans-unit id="back">
-                               <source>&lt; Back</source>
+                               <source>Back</source>
                        </trans-unit>
                        <trans-unit id="browse">
                                <source>Browse:</source>
index 3780e98..e2a8308 100644 (file)
@@ -37,4 +37,4 @@ Web > Template
                        }
                }
        }
-}
\ No newline at end of file
+}