[TASK] Improve username information in top toolbar
authorFelix Kopp <felix-source@phorax.com>
Mon, 12 Dec 2011 03:45:28 +0000 (04:45 +0100)
committerChristian Kuhn <lolli@schwarzbu.ch>
Sat, 28 Jan 2012 12:42:04 +0000 (13:42 +0100)
Hides the username text if a full name is available.
If the superuser mode is enabled the username is preserved.

Change-Id: Id8c95391697f760b123cf18c039f573d99716898
Resolves: #32450
Releases: 4.7
Reviewed-on: http://review.typo3.org/7209
Reviewed-by: Markus Klein
Tested-by: Markus Klein
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
typo3/backend.php
typo3/sysext/t3skin/stylesheets/visual/toolbar_username.css

index 213e2b0..680dc98 100644 (file)
@@ -366,29 +366,31 @@ class TYPO3backend {
         * @return      string          html code snippet displaying the currently logged in user
         */
        protected function getLoggedInUserLabel() {
+               $css = 'toolbar-item no-separator';
                $icon = t3lib_iconWorks::getSpriteIcon('status-user-' . ($GLOBALS['BE_USER']->isAdmin() ? 'admin' : 'backend'));
+               $realName = $GLOBALS['BE_USER']->user['realName'];
+               $username = $GLOBALS['BE_USER']->user['username'];
 
-               $label = $GLOBALS['BE_USER']->user['realName'] ?
-                       $GLOBALS['BE_USER']->user['realName'] . ' (' . $GLOBALS['BE_USER']->user['username'] . ')' :
-                       $GLOBALS['BE_USER']->user['username'];
+               $label = $realName ? $realName : $username;
+               $title = $username;
 
                        // Link to user setup if it's loaded and user has access
                $link = '';
-               if (t3lib_extMgm::isLoaded('setup') && $GLOBALS['BE_USER']->check('modules','user_setup')) {
-                       $link = '<a href="#" onclick="top.goToModule(\'user_setup\');this.blur();return false;">';
+               if (t3lib_extMgm::isLoaded('setup') && $GLOBALS['BE_USER']->check('modules', 'user_setup')) {
+                       $link = '<a href="#" onclick="top.goToModule(\'user_setup\'); this.blur(); return false;">';
                }
 
-               $username = '">'.$link.$icon.'<span>'.htmlspecialchars($label).'</span>'.($link?'</a>':'');
-
                        // superuser mode
                if ($GLOBALS['BE_USER']->user['ses_backuserid']) {
-                       $username   = ' su-user">'.$icon.
-                       '<span title="' . $GLOBALS['LANG']->getLL('switchtouser') . '">' .
-                       $GLOBALS['LANG']->getLL('switchtousershort') . ' </span>' .
-                       '<span>' . htmlspecialchars($label) . '</span>';
+                       $css .= ' su-user';
+                       $title = $GLOBALS['LANG']->getLL('switchtouser') . ': ' . $username;
+                       $label = $GLOBALS['LANG']->getLL('switchtousershort') . ' ' .
+                               ($realName ? $realName . ' (' . $username . ')' : $username);
                }
 
-               return '<div id="username" class="toolbar-item no-separator'.$username.'</div>';
+               return '<div id="username" class="' . $css . '">' . $link . $icon .
+                               '<span title="' . htmlspecialchars($title) . '">' . htmlspecialchars($label) . '</span>' .
+                               ($link ? '</a>' : '') . '</div>';
        }
 
        /**
index bfc4ac1..60892f8 100644 (file)
@@ -8,15 +8,14 @@ Username inside the toolbar
        font-weight: bold;
 }
 
-#username.su-user {
-       color: #eb6e5a;
-       font-weight: bold;
-}
-
 #username a {
        color: #ffffff;
        font-weight: bold;
        font-size: 11px;
        line-height: 16px;
        text-decoration: none;
+}
+
+#username.su-user a {
+       color: #eb6e5a;
 }
\ No newline at end of file