Merge pull request #2265 from ORCID/fix/9240-registry-header-accessib…

…ility-updates-desktop fix: Update text in sign in / register button and styling in orcid logo text color
ORCID · May 23, 2024 · 8c68830 · 8c68830
2 parents 0ec743a + dc1bd87
commit 8c68830
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 11 deletions.
diff --git a/src/app/layout/header/header.component.html b/src/app/layout/header/header.component.html
@@ -33,7 +33,7 @@
             ></app-language>
           </div>
           <div *ngIf="!mobileMenuState" class="col no-gutters">
-            <!-- <app-user-menu></app-user-menu> -->
+            <app-user-menu></app-user-menu>
           </div>
 
           <app-menu-icon
@@ -125,16 +125,16 @@
             signinRegisterButton
           "
         >
-          <!-- <button
+          <button
             mat-stroked-button
             class="sign-in-button"
             (click)="goto('signin')"
           >
-            <ng-container i18n="#upperCase@@header.signin">SIGN IN</ng-container
-            >/<ng-container i18n="#upperCase@@header.register"
-              >REGISTER</ng-container
+            <ng-container i18n="@@header.signin">Sign in</ng-container
+            >&nbsp;/&nbsp;<ng-container i18n="@@header.register"
+              >Register</ng-container
             >
-          </button> -->
+          </button>
         </li>
         <li
           class="col no-gutters"

diff --git a/src/app/layout/header/header.component.scss b/src/app/layout/header/header.component.scss
@@ -1,7 +1,6 @@
 header.container {
-  max-width: 100%;
   padding: 24px 32px;
-  margin: 0;
+  margin: auto;
 
   .logo-menu-container {
     .logo-container {

diff --git a/src/app/layout/header/header.component.scss-theme.scss b/src/app/layout/header/header.component.scss-theme.scss
@@ -11,7 +11,7 @@
 
   .container {
     .logo-container {
-      color: map-get($foreground, 'brand-logo-grey') !important;
+      color: mat.get-color-from-palette($background, placeholder-grey) !important;
     }
   }
   .top-bar {

diff --git a/src/app/layout/user-menu/user-menu.component.html b/src/app/layout/user-menu/user-menu.component.html
@@ -26,8 +26,8 @@
     [attr.aria-label]="labelSigninRegister"
     id="menu-signin-button"
   >
-    <ng-container i18n="#upperCase@@public-layout.sign_in">SIGN IN</ng-container
-    >/<ng-container i18n="#upperCase@@header.register">REGISTER</ng-container>
+    <ng-container i18n="@@header.signin">Sign in</ng-container
+    >&nbsp;/&nbsp;<ng-container i18n="@@header.register">Register</ng-container>
   </button>
 </section>
 <mat-menu #menu="matMenu" class="user-menu" id="cy-user-menu">