[TASK] Streamline button order/group 38/50138/2
authorDaniel Windloff <daniel.jc.windloff@googlemail.com>
Sat, 8 Oct 2016 16:41:28 +0000 (18:41 +0200)
committerChristian Kuhn <lolli@schwarzbu.ch>
Sun, 9 Oct 2016 12:47:08 +0000 (14:47 +0200)
In order to be consistent, change the button order and styling
(according to the list module).

Resolves: #78199
Releases: master
Change-Id: I44b03a7f6e4e49f397a62d00158083c568ce5cda
Reviewed-on: https://review.typo3.org/50138
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
typo3/sysext/beuser/Resources/Private/Partials/BackendUser/IndexListRow.html
typo3/sysext/beuser/Resources/Private/Partials/BackendUserGroup/IndexListRow.html
typo3/sysext/core/Classes/Database/QueryView.php
typo3/sysext/frontend/Classes/Controller/TranslationStatusController.php
typo3/sysext/scheduler/Classes/Controller/SchedulerModuleController.php

index 2bd89bd..348eec4 100644 (file)
@@ -29,9 +29,7 @@
                </f:if>
        </td>
        <td class="col-control">
-               <f:link.action action="addToCompareList" arguments="{uid: backendUser.uid}" class="btn btn-default"><core:icon identifier="actions-edit-add" size="small"/> <f:translate key="compare" /></f:link.action>
                <div class="btn-group" role="group">
-                       <a class="btn btn-default" href="#" onclick="top.launchView('be_users', '{backendUser.uid}'); return false;"><core:icon identifier="actions-document-info" /></a>
                        <a class="btn btn-default" href="{bu:editRecord(parameters: 'edit[be_users][{backendUser.uid}]=edit&returnUrl={returnUrl}')}"><core:icon identifier="actions-open" /></a>
                        <f:if condition="{backendUser.currentlyLoggedIn} == 1">
                                <f:then>
                                </f:else>
                        </f:if>
                        <bu:removeUser backendUser="{backendUser}" />
+               </div>
+               <div class="btn-group" role="group">
+                       <a class="btn btn-default" href="#" onclick="top.launchView('be_users', '{backendUser.uid}'); return false;"><core:icon identifier="actions-document-info" /></a>
+               </div>
+               <div class="btn-group" role="group">
+                       <f:link.action action="addToCompareList" arguments="{uid: backendUser.uid}" class="btn btn-default"><core:icon identifier="actions-edit-add" size="small"/> <f:translate key="compare" /></f:link.action>
                        <bu:SwitchUser backendUser="{backendUser}" />
                </div>
        </td>
index 6d27860..813ea0d 100644 (file)
@@ -24,7 +24,6 @@
        </td>
        <td class="col-control">
                <div class="btn-group" role="group">
-                       <a class="btn btn-default" href="#" onclick="top.launchView('be_groups', '{backendUserGroup.uid}'); return false;"><core:icon identifier="actions-document-info" /></a>
                        <a class="btn btn-default" href="{bu:editRecord(parameters: 'edit[be_groups][{backendUserGroup.uid}]=edit&returnUrl={returnUrl}')}"><core:icon identifier="actions-open" /></a>
                        <f:if condition="{backendUserGroup.hidden}">
                                <f:then>
@@ -35,5 +34,8 @@
                                </f:else>
                        </f:if>
                </div>
+               <div class="btn-group" role="group">
+                       <a class="btn btn-default" href="#" onclick="top.launchView('be_groups', '{backendUserGroup.uid}'); return false;"><core:icon identifier="actions-document-info" /></a
+               </div>
        </td>
 </tr>
index 3bfa8ad..9b70124 100644 (file)
@@ -679,8 +679,9 @@ class QueryView
                 $out .= '<td>' . $fVnew . '</td>';
             }
         }
-        $out .= '<td><div class="btn-group">';
+        $out .= '<td>';
         if (!$row['deleted']) {
+            $out .= '<div class="btn-group" role="group">';
             $url = BackendUtility::getModuleUrl('record_edit', [
                 'edit' => [
                     $table => [
@@ -690,12 +691,15 @@ class QueryView
                 'returnUrl' => GeneralUtility::getIndpEnv('REQUEST_URI')
                     . GeneralUtility::implodeArrayForUrl('SET', (array)GeneralUtility::_POST('SET'))
             ]);
+            $out .= '<a class="btn btn-default" href="' . htmlspecialchars($url) . '">'
+                . $this->iconFactory->getIcon('actions-open', Icon::SIZE_SMALL)->render() . '</a>';
+            $out .= '</div><div class="btn-group" role="group">';
             $out .= '<a class="btn btn-default" href="#" onClick="top.launchView(\'' . $table . '\',' . $row['uid']
                 . ');return false;">' . $this->iconFactory->getIcon('actions-document-info', Icon::SIZE_SMALL)->render()
                 . '</a>';
-            $out .= '<a class="btn btn-default" href="' . htmlspecialchars($url) . '">'
-                . $this->iconFactory->getIcon('actions-open', Icon::SIZE_SMALL)->render() . '</a>';
+            $out .= '</div>';
         } else {
+            $out .= '<div class="btn-group" role="group">';
             $out .= '<a class="btn btn-default" href="' . htmlspecialchars(BackendUtility::getModuleUrl('tce_db', [
                         'cmd' => [
                             $table => [
@@ -727,6 +731,7 @@ class QueryView
                     'redirect' => $redirectUrl
                 ])) . '" title="' . htmlspecialchars($this->languageService->getLL('undelete_and_edit')) . '">';
             $out .= $this->iconFactory->getIcon('actions-edit-restore-edit', Icon::SIZE_SMALL)->render() . '</a>';
+            $out .= '</div>';
         }
         $_params = [$table => $row];
         if (is_array($this->hookArray['additionalButtons'])) {
@@ -734,7 +739,7 @@ class QueryView
                 $out .= GeneralUtility::callUserFunction($_funcRef, $_params, $this);
             }
         }
-        $out .= '</div></td></tr>';
+        $out .= '</td></tr>';
         return $out;
     }
 
index f8a3b30..c2e1ec3 100644 (file)
@@ -166,11 +166,11 @@ class TranslationStatusController extends \TYPO3\CMS\Backend\Module\AbstractFunc
                 ],
                 'returnUrl' => GeneralUtility::getIndpEnv('REQUEST_URI')
             ]);
-            $info = '<a href="' . htmlspecialchars($editUrl)
+            $info = str_replace('###LANG_UID###', '0', $viewPageLink);
+            $info .= '<a href="' . htmlspecialchars($editUrl)
                 . '" class="btn btn-default" title="' . $lang->sL(
                     'LLL:EXT:frontend/Resources/Private/Language/locallang_webinfo.xlf:lang_renderl10n_editDefaultLanguagePage'
                 ) . '">' . $this->iconFactory->getIcon('actions-page-open', Icon::SIZE_SMALL)->render() . '</a>';
-            $info .= str_replace('###LANG_UID###', '0', $viewPageLink);
             $info .= '&nbsp;';
             $info .= GeneralUtility::hideIfDefaultLanguage($data['row']['l18n_cfg']) ? '<span title="' . htmlspecialchars($lang->sL('LLL:EXT:frontend/Resources/Private/Language/locallang_tca.xlf:pages.l18n_cfg.I.1')) . '">D</span>' : '&nbsp;';
             $info .= GeneralUtility::hideIfNotTranslated($data['row']['l18n_cfg']) ? '<span title="' . htmlspecialchars($lang->sL('LLL:EXT:frontend/Resources/Private/Language/locallang_tca.xlf:pages.l18n_cfg.I.2')) . '">N</span>' : '&nbsp;';
@@ -213,11 +213,11 @@ class TranslationStatusController extends \TYPO3\CMS\Backend\Module\AbstractFunc
                             ],
                             'returnUrl' => GeneralUtility::getIndpEnv('REQUEST_URI')
                         ]);
-                        $info = '<a href="' . htmlspecialchars($editUrl)
+                        $info = str_replace('###LANG_UID###', $langRow['uid'], $viewPageLink);
+                        $info .= '<a href="' . htmlspecialchars($editUrl)
                             . '" class="btn btn-default" title="' . $lang->sL(
                                 'LLL:EXT:frontend/Resources/Private/Language/locallang_webinfo.xlf:lang_renderl10n_editLanguageOverlayRecord'
                             ) . '">' . $this->iconFactory->getIcon('actions-open', Icon::SIZE_SMALL)->render() . '</a>';
-                        $info .= str_replace('###LANG_UID###', $langRow['uid'], $viewPageLink);
                         $tCells[] = '<td class="' . $status . '"><div class="btn-group">' . $info . '</div></td>';
                         $tCells[] = '<td class="' . $status . '" title="' . $lang->sL(
                                 'LLL:EXT:frontend/Resources/Private/Language/locallang_webinfo.xlf:lang_renderl10n_CEcount'
index 866ebfb..a7dcb72 100644 (file)
@@ -1142,7 +1142,7 @@ class SchedulerModuleController extends \TYPO3\CMS\Backend\Module\BaseScriptClas
                     // Define checkbox
                     $startExecutionElement = '<label class="btn btn-default btn-checkbox"><input type="checkbox" name="tx_scheduler[execute][]" value="' . $schedulerRecord['uid'] . '" id="task_' . $schedulerRecord['uid'] . '"><span class="t3-icon fa"></span></label>';
 
-                    $actions = $editAction . $toggleHiddenAction . $deleteAction;
+                    $actions = '<div class="btn btn-group" role="group">' . $editAction . $toggleHiddenAction . $deleteAction . '</div>';
 
                     // Check the disable status
                     // Row is shown dimmed if task is disabled, unless it is still running
@@ -1156,9 +1156,9 @@ class SchedulerModuleController extends \TYPO3\CMS\Backend\Module\BaseScriptClas
 
                     // Show no action links (edit, delete) if task is running
                     if ($isRunning) {
-                        $actions = $stopAction;
+                        $actions = '<div class="btn btn-group" role="group">' . $stopAction . '</div>';
                     } else {
-                        $actions .= $runAction;
+                        $actions .= '<div class="btn btn-group" role="group">' . $runAction . '</div>';
                     }
 
                     // Check if the last run failed
@@ -1196,7 +1196,7 @@ class SchedulerModuleController extends \TYPO3\CMS\Backend\Module\BaseScriptClas
                             . '<td>' . $multiple . '</td>'
                             . '<td>' . $lastExecution . '</td>'
                             . '<td>' . $nextDate . '</td>'
-                            . '<td nowrap="nowrap"><div class="btn-group" role="group">' . $actions . '</div></td>'
+                            . '<td nowrap="nowrap">' . $actions . '</td>'
                         . '</tr>';
                 } else {
                     // The task object is not valid