diff --git a/app/assets/javascripts/discourse/templates/header.js.handlebars b/app/assets/javascripts/discourse/templates/header.js.handlebars index 0e283fe4f..d1306aa82 100644 --- a/app/assets/javascripts/discourse/templates/header.js.handlebars +++ b/app/assets/javascripts/discourse/templates/header.js.handlebars @@ -37,14 +37,18 @@ <div class='current-username'> {{#if currentUser}} <span class='username'><a {{bind-attr href="currentUser.path"}}>{{currentUser.displayName}}</a></span> - {{else}} - <button {{action showLogin}} class='btn btn-primary btn-small'>{{i18n log_in}}</button> {{/if}} </div> {{/unless}} + {{#unless currentUser}} + <button {{action showLogin}} class='btn btn-primary btn-small login-button'> + <i class="fa fa-user"></i> + {{i18n log_in}} + </button> + {{/unless}} <ul class='icons clearfix' role='navigation'> - <li class='notifications'> - {{#if currentUser}} + {{#if currentUser}} + <li class='notifications'> <a class='icon' href="#" {{action showNotifications target="view"}} data-notifications="notifications-dropdown" id='user-notifications' title='{{i18n notifications.title}}'> <i class='fa fa-comment'></i> <span class='sr-only'>{{i18n notifications.title}}</span> @@ -55,10 +59,8 @@ {{#if currentUser.unread_private_messages}} <a href='#' class='badge-notification unread-private-messages'>{{currentUser.unread_private_messages}}</a> {{/if}} - {{else}} - <a class='icon' href="#" {{action showLogin}} title='{{i18n notifications.title}}' aria-hidden="true"><i class='fa fa-comment'></i></a> - {{/if}} - </li> + </li> + {{/if}} <li> {{#if Discourse.loginRequired}} <a id='search-button' class='icon expand' href='#' aria-hidden="true" {{action showLogin}}> @@ -98,8 +100,8 @@ <a href='/admin/flags/active' title='{{i18n notifications.total_flagged}}' class='badge-notification flagged-posts'>{{currentUser.site_flagged_posts_count}}</a> {{/if}} </li> - <li class='current-user dropdown'> - {{#if currentUser}} + {{#if currentUser}} + <li class='current-user dropdown'> <a class='icon' data-dropdown="user-dropdown" data-render="renderUserDropdown" @@ -108,10 +110,8 @@ id="current-user"> {{boundAvatar currentUser imageSize="medium"}} </a> - {{else}} - <div class="icon not-logged-in-avatar" aria-hidden="true" {{action showLogin}}><i class='fa fa-user' title='{{i18n not_logged_in_user}}'></i></div> - {{/if}} - </li> + </li> + {{/if}} </ul> {{render "search"}} diff --git a/app/assets/stylesheets/desktop/header.scss b/app/assets/stylesheets/desktop/header.scss index e842726cd..9947dab4f 100644 --- a/app/assets/stylesheets/desktop/header.scss +++ b/app/assets/stylesheets/desktop/header.scss @@ -47,9 +47,12 @@ display:block; margin-top: 10px; } - button { - margin-top: 9px; - } + } + button.login-button { + float: left; + margin-top: 7px; + padding: 6px 10px; + .fa { margin-right: 3px; } } .icons { float: left; diff --git a/app/assets/stylesheets/mobile/header.scss b/app/assets/stylesheets/mobile/header.scss index f111dfadb..c102af15e 100644 --- a/app/assets/stylesheets/mobile/header.scss +++ b/app/assets/stylesheets/mobile/header.scss @@ -51,14 +51,17 @@ font-size: 14px; line-height: 40px; } - button { - margin-top: 8px; - } + } + button.login-button { + float: left; + margin-top: 7px; + padding: 6px 10px; + .fa { margin-right: 3px; } } .icons { float: left; text-align: center; - margin: 0 0 0 15px; + margin: 0 0 0 5px; list-style: none; > li { float: left;