fix: minor bugfixes and UI adjustments

src/app/components/login/forgot-pw/forgot-pw.component.ts

@@ -35,9 +35,16 @@ export class ForgotPwComponent {
   ) {}
 
   onSubmit(ngForm: NgForm) {
-    this.buttonStateService.enableButton();
+    this.buttonStateService.disableButton();
     if (ngForm.submitted && ngForm.form.valid) {
+      try {
         this.authService.resetPassword(this.pwResetData.mail.toLowerCase());
+        this.buttonStateService.enableButton();
+      } catch (error: any) {
+        this.buttonStateService.enableButton();
+      }
+    } else {
+      this.buttonStateService.enableButton();
     }
   }
 
@@ -46,11 +53,6 @@ export class ForgotPwComponent {
   }
 
   isEmailValid(emailValue: string) {
-    const emailRegex = /^[\w.%+-]+@[\w.-]+\.[a-zA-Z]{2,}$/;
+    return /^[\w.%+-]+@[\w.-]+\.[a-zA-Z]{2,}$/.test(emailValue);
-    if (emailRegex.test(emailValue)) {
-      return true;
-    } else {
-      return false;
-    }
   }
 }

src/app/components/login/forgot-pw/pw-reset/pw-reset.component.ts

@@ -75,6 +75,8 @@ export class PwResetComponent {
         this.errorHttpMessage = error.message;
         this.buttonStateService.enableButton();
       }
+    } else {
+      this.buttonStateService.enableButton();
     }
   }
 }

src/app/components/login/header/header.component.html

@@ -1,4 +1,9 @@
-<img class="logo" src="./../../../assets/img/login/logo.svg" alt="" />
+<img
+  class="logo"
+  src="./../../../assets/img/login/logo.svg"
+  alt="logo"
+  routerLink="/login"
+/>
 @if (signUpBtn) {
 <div class="register">
   <p>{{ "loginHeader.notAUser" | translate }}</p>

src/app/components/login/header/header.component.scss

@@ -4,6 +4,7 @@
   top: 60px;
   height: 100px;
   width: 122px;
+  cursor: pointer;
 }
 
 .register {

src/app/components/login/header/header.component.ts

@@ -1,12 +1,12 @@
 import { Component, Input } from '@angular/core';
-import { RouterModule } from '@angular/router';
+import { RouterLink, RouterModule } from '@angular/router';
 import { TranslateModule } from '@ngx-translate/core';
 import { LanguageService } from '../../../services/language.service';
 
 @Component({
   selector: 'app-header',
   standalone: true,
-  imports: [RouterModule, TranslateModule],
+  imports: [RouterModule, RouterLink, TranslateModule],
   templateUrl: './header.component.html',
   styleUrl: './header.component.scss',
 })

src/app/components/login/login.component.html

@@ -39,8 +39,7 @@
           !isButtonDisabled()) {
           <p>{{ "login.errorMail1" | translate }}</p>
           } @if (mail.valid && mail.touched &&
-          !checkIfUserEmailIsValid(loginData.email.toLowerCase()) &&
+          !isEmailValid(loginData.email.toLowerCase()) && !isButtonDisabled()) {
-          !isButtonDisabled()) {
           <p>{{ "login.errorMail2" | translate }}</p>
           }
         </div>
@@ -109,7 +108,7 @@
             password.invalid ||
             !loginData.password ||
             isButtonDisabled() ||
-            !checkIfUserEmailIsValid(loginData.email)
+            !isEmailValid(loginData.email)
           "
         ></app-form-btn>
         <app-form-btn

src/app/components/login/login.component.ts

@@ -84,6 +84,8 @@ export class LoginComponent {
       } catch (error) {
         this.buttonStateService.enableButton();
       }
+    } else {
+      this.buttonStateService.enableButton();
     }
   }
 
@@ -94,12 +96,7 @@ export class LoginComponent {
     this.onSubmit({ submitted: true, form: { valid: true } } as NgForm);
   }
 
-  checkIfUserEmailIsValid(emailValue: string) {
+  isEmailValid(emailValue: string) {
-    const emailRegex = /^[\w.%+-]+@[\w.-]+\.[a-zA-Z]{2,}$/;
+    return /^[\w.%+-]+@[\w.-]+\.[a-zA-Z]{2,}$/.test(emailValue);
-    if (emailRegex.test(emailValue)) {
-      return true;
-    } else {
-      return false;
-    }
   }
 }

src/app/components/login/register/register.component.ts

@@ -80,11 +80,6 @@ export class RegisterComponent {
   }
 
   isEmailValid(emailValue: string) {
-    const emailRegex = /^[\w.%+-]+@[\w.-]+\.[a-zA-Z]{2,}$/;
+    return /^[\w.%+-]+@[\w.-]+\.[a-zA-Z]{2,}$/.test(emailValue);
-    if (emailRegex.test(emailValue)) {
-      return true;
-    } else {
-      return false;
-    }
   }
 }