From 1fd341187748456a8322beba1126b71a470d13ba Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Ren=C3=A9=20Panzar?= <rene.panzar@gmail.com>
Date: Fri, 2 Oct 2020 18:03:23 +0200
Subject: [PATCH] Migrate to latest Play version - addresses #92

Integrates the language switcher on the main and login pages
---
 conf/messages                                 |  1 +
 conf/messages.de                              |  1 +
 .../translatr-admin/src/assets/i18n/de.json   |  1 +
 .../translatr-admin/src/assets/i18n/en.json   |  1 +
 .../pages/main-page/main-page.component.html  |  1 -
 ui/apps/translatr/src/assets/i18n/de.json     |  1 +
 ui/apps/translatr/src/assets/i18n/en.json     |  1 +
 .../auth-bar-language-switcher.component.html |  1 -
 .../auth-bar-language-switcher.component.ts   |  5 +-
 .../modules/nav/navbar/navbar.component.scss  |  4 +
 .../login-page/login-page.component.html      | 51 +++++++----
 .../login-page/login-page.component.scss      | 91 ++++++++++---------
 12 files changed, 91 insertions(+), 68 deletions(-)

diff --git a/conf/messages b/conf/messages
index 50e83f45..1ec55538 100644
--- a/conf/messages
+++ b/conf/messages
@@ -76,6 +76,7 @@ auth.keycloak=Keycloak
 auth.login=Sign in
 auth.logout=Logout {{name}}
 auth.twitter=Twitter
+authclients.title=Sign-in Services
 button.cancel=Cancel
 button.close=Close
 button.create=Create
diff --git a/conf/messages.de b/conf/messages.de
index 4e4b1f2e..5a711ebd 100644
--- a/conf/messages.de
+++ b/conf/messages.de
@@ -76,6 +76,7 @@ auth.keycloak=Keycloak
 auth.login=Anmelden
 auth.logout={{name}} abmelden
 auth.twitter=Twitter
+authclients.title=Anmeldedienste
 button.cancel=Abbrechen
 button.close=Schließen
 button.create=Anlegen
diff --git a/ui/apps/translatr-admin/src/assets/i18n/de.json b/ui/apps/translatr-admin/src/assets/i18n/de.json
index 7931fdd5..81a2ec3f 100644
--- a/ui/apps/translatr-admin/src/assets/i18n/de.json
+++ b/ui/apps/translatr-admin/src/assets/i18n/de.json
@@ -77,6 +77,7 @@
   "auth.login" : "Anmelden",
   "auth.logout" : "{{name}} abmelden",
   "auth.twitter" : "Twitter",
+  "authclients.title" : "Anmeldedienste",
   "button.cancel" : "Abbrechen",
   "button.close" : "Schließen",
   "button.create" : "Anlegen",
diff --git a/ui/apps/translatr-admin/src/assets/i18n/en.json b/ui/apps/translatr-admin/src/assets/i18n/en.json
index 35efcf5e..0ec1e3e6 100644
--- a/ui/apps/translatr-admin/src/assets/i18n/en.json
+++ b/ui/apps/translatr-admin/src/assets/i18n/en.json
@@ -77,6 +77,7 @@
   "auth.login" : "Sign in",
   "auth.logout" : "Logout {{name}}",
   "auth.twitter" : "Twitter",
+  "authclients.title" : "Sign-in Services",
   "button.cancel" : "Cancel",
   "button.close" : "Close",
   "button.create" : "Create",
diff --git a/ui/apps/translatr/src/app/modules/pages/main-page/main-page.component.html b/ui/apps/translatr/src/app/modules/pages/main-page/main-page.component.html
index bcf45acd..1e22742e 100644
--- a/ui/apps/translatr/src/app/modules/pages/main-page/main-page.component.html
+++ b/ui/apps/translatr/src/app/modules/pages/main-page/main-page.component.html
@@ -1,6 +1,5 @@
 <app-navbar [elevated]="false" [overlay]="true" [showLogo]="false" [page]="'home.title' | transloco">
   <app-auth-bar-language-switcher
-    *featureFlag="Feature.LanguageSwitcher"
     [matTooltip]="'language.switcher' | transloco"
   ></app-auth-bar-language-switcher>
   <app-auth-bar-item [me]="me$ | async" [endpointUrl]="endpointUrl">
diff --git a/ui/apps/translatr/src/assets/i18n/de.json b/ui/apps/translatr/src/assets/i18n/de.json
index 7931fdd5..81a2ec3f 100644
--- a/ui/apps/translatr/src/assets/i18n/de.json
+++ b/ui/apps/translatr/src/assets/i18n/de.json
@@ -77,6 +77,7 @@
   "auth.login" : "Anmelden",
   "auth.logout" : "{{name}} abmelden",
   "auth.twitter" : "Twitter",
+  "authclients.title" : "Anmeldedienste",
   "button.cancel" : "Abbrechen",
   "button.close" : "Schließen",
   "button.create" : "Anlegen",
diff --git a/ui/apps/translatr/src/assets/i18n/en.json b/ui/apps/translatr/src/assets/i18n/en.json
index 35efcf5e..0ec1e3e6 100644
--- a/ui/apps/translatr/src/assets/i18n/en.json
+++ b/ui/apps/translatr/src/assets/i18n/en.json
@@ -77,6 +77,7 @@
   "auth.login" : "Sign in",
   "auth.logout" : "Logout {{name}}",
   "auth.twitter" : "Twitter",
+  "authclients.title" : "Sign-in Services",
   "button.cancel" : "Cancel",
   "button.close" : "Close",
   "button.create" : "Create",
diff --git a/ui/libs/translatr-components/src/lib/modules/nav/navbar/auth-bar-language-switcher/auth-bar-language-switcher.component.html b/ui/libs/translatr-components/src/lib/modules/nav/navbar/auth-bar-language-switcher/auth-bar-language-switcher.component.html
index 8d1104e2..7049cb14 100644
--- a/ui/libs/translatr-components/src/lib/modules/nav/navbar/auth-bar-language-switcher/auth-bar-language-switcher.component.html
+++ b/ui/libs/translatr-components/src/lib/modules/nav/navbar/auth-bar-language-switcher/auth-bar-language-switcher.component.html
@@ -1,5 +1,4 @@
 <button
-  *ngIf="active"
   [mat-menu-trigger-for]="languageSelector"
   mat-icon-button
 >
diff --git a/ui/libs/translatr-components/src/lib/modules/nav/navbar/auth-bar-language-switcher/auth-bar-language-switcher.component.ts b/ui/libs/translatr-components/src/lib/modules/nav/navbar/auth-bar-language-switcher/auth-bar-language-switcher.component.ts
index 152d4344..53e426d0 100644
--- a/ui/libs/translatr-components/src/lib/modules/nav/navbar/auth-bar-language-switcher/auth-bar-language-switcher.component.ts
+++ b/ui/libs/translatr-components/src/lib/modules/nav/navbar/auth-bar-language-switcher/auth-bar-language-switcher.component.ts
@@ -9,7 +9,6 @@ import { LanguageSwicher } from '../language-swicher';
 })
 export class AuthBarLanguageSwitcherComponent {
   readonly availableLanguages = this.translocoService.getAvailableLangs();
-  readonly active = this.languageSwicher !== null;
   activeLang = this.translocoService.getActiveLang();
 
   constructor(
@@ -21,7 +20,9 @@ export class AuthBarLanguageSwitcherComponent {
     if (this.activeLang !== language) {
       this.translocoService.setActiveLang(language);
       this.activeLang = this.translocoService.getActiveLang();
-      this.languageSwicher.updatePreferredLanguage(language);
+      if (this.languageSwicher !== null) {
+        this.languageSwicher.updatePreferredLanguage(language);
+      }
     }
   }
 }
diff --git a/ui/libs/translatr-components/src/lib/modules/nav/navbar/navbar.component.scss b/ui/libs/translatr-components/src/lib/modules/nav/navbar/navbar.component.scss
index ae7b32ee..c761d6aa 100644
--- a/ui/libs/translatr-components/src/lib/modules/nav/navbar/navbar.component.scss
+++ b/ui/libs/translatr-components/src/lib/modules/nav/navbar/navbar.component.scss
@@ -19,6 +19,10 @@
 
 .mat-toolbar {
   z-index: 2;
+
+  .mat-toolbar-row {
+    color: white;
+  }
 }
 
 .elevated .mat-toolbar-row {
diff --git a/ui/libs/translatr-components/src/lib/modules/pages/login-page/login-page.component.html b/ui/libs/translatr-components/src/lib/modules/pages/login-page/login-page.component.html
index 280bede4..493112b4 100644
--- a/ui/libs/translatr-components/src/lib/modules/pages/login-page/login-page.component.html
+++ b/ui/libs/translatr-components/src/lib/modules/pages/login-page/login-page.component.html
@@ -1,24 +1,37 @@
 <ng-container *transloco="let t">
-  <app-navbar [page]="t('login.title')"></app-navbar>
+  <app-navbar [page]="t('login.title')" [overlay]="true" [elevated]="false">
+    <app-auth-bar-language-switcher></app-auth-bar-language-switcher>
+  </app-navbar>
 
-  <div class="wrapper">
-    <mat-card class="login">
-      <div class="info">
-        <mat-icon>account_circle</mat-icon>
-        <span transloco="login.text">Sign in with any of the given providers</span>
+  <div class="main-container">
+    <header class="header header-graphic">
+      <div class="container title">
+        <div class="content">
+          <mat-icon class="image">account_circle</mat-icon>
+          <div class="text">
+            <h1 class="page" transloco="login.title">Login Page</h1>
+            <p class="description" transloco="login.text">Sign in with any of the given providers</p>
+          </div>
+        </div>
       </div>
-      <div class="options">
-        <a
-          *ngFor="let provider of providers$ | async"
-          [ngClass]="'client-' + provider.key"
-          href="{{endpointUrl}}{{provider.url}}{{redirectUri$ | async}}"
-          mat-button
-          class="client"
-        >
-          <fa-icon [icon]="icons[provider.key]"></fa-icon>
-          {{names[provider.key]}}
-        </a>
-      </div>
-    </mat-card>
+    </header>
+
+    <div class="wrapper">
+      <mat-card class="login">
+        <mat-card-title transloco="authclients.title">Identity Providers</mat-card-title>
+        <mat-card-content class="options">
+          <a
+            *ngFor="let provider of providers$ | async"
+            [ngClass]="'client-' + provider.key"
+            href="{{endpointUrl}}{{provider.url}}{{redirectUri$ | async}}"
+            mat-button
+            class="client"
+          >
+            <fa-icon [icon]="icons[provider.key]"></fa-icon>
+            {{names[provider.key]}}
+          </a>
+        </mat-card-content>
+      </mat-card>
+    </div>
   </div>
 </ng-container>
diff --git a/ui/libs/translatr-components/src/lib/modules/pages/login-page/login-page.component.scss b/ui/libs/translatr-components/src/lib/modules/pages/login-page/login-page.component.scss
index 381d302b..1da11224 100644
--- a/ui/libs/translatr-components/src/lib/modules/pages/login-page/login-page.component.scss
+++ b/ui/libs/translatr-components/src/lib/modules/pages/login-page/login-page.component.scss
@@ -7,6 +7,35 @@
   flex-direction: column;
 }
 
+.header {
+  background-color: #1976d2;
+
+  &.header-graphic {
+    background-image: url(assets/headers/stripes.svg);
+  }
+
+  .content {
+    display: flex;
+    flex-direction: row;
+    align-items: center;
+    padding: 16px 8px;
+
+    .text {
+      margin-left: 8px;
+
+      p {
+        margin: 0;
+      }
+    }
+
+    .image {
+      width: 40px;
+      height: 40px;
+      font-size: 40px;
+    }
+  }
+}
+
 .wrapper {
   justify-content: center;
   margin: 8px;
@@ -15,59 +44,31 @@
     display: flex;
     flex: 1 100%;
     align-items: center;
+
+    margin-top: 32px;
   }
+}
 
-  .login {
-    display: flex;
-    flex-direction: column;
+.mat-card {
+  display: flex;
+  flex-direction: column;
 
-    .options,
-    .info {
-      flex: 1 50%;
-      display: flex;
-      justify-content: center;
-      align-items: center;
-      flex-direction: column;
+  .options {
+    .mat-button {
+      display: block;
+      width: 100%;
       margin: 8px 0;
-    }
+      color: #fff;
 
-    .info {
-      color: rgba(0, 0, 0, 0.6);
-      text-align: center;
+      @include mat-elevation-transition;
 
-      .mat-icon {
-        font-size: 128px;
-        color: rgba(0, 0, 0, 0.3);
-        display: block;
-        height: 128px;
-        width: 128px;
-        margin: 0 auto;
+      &:hover {
+        @include mat-elevation(4);
       }
     }
+  }
 
-    .options {
-      .mat-button {
-        display: block;
-        width: 100%;
-        margin: 4px 8px;
-        color: #fff;
-
-        @include mat-elevation-transition;
-
-        &:hover {
-          @include mat-elevation(4);
-        }
-      }
-    }
-
-    @include medium {
-      flex-direction: row;
-      min-width: 500px;
-
-      .options,
-      .info {
-        margin: 16px;
-      }
-    }
+  @include medium {
+    min-width: 240px;
   }
 }