From b300fe1b7b218e834aee6b7b872a5ff96d34086a Mon Sep 17 00:00:00 2001
From: Patrick Mueller <patrick@mueller-patrick.tech>
Date: Thu, 8 Sep 2022 17:41:10 +0200
Subject: [PATCH] Formatting, adding comments

---
 .../components/header/header.component.html   |  2 +-
 .../components/header/header.component.scss   |  1 +
 src/app/components/header/header.component.ts | 20 ++++++++++---------
 3 files changed, 13 insertions(+), 10 deletions(-)

diff --git a/src/app/components/header/header.component.html b/src/app/components/header/header.component.html
index 6a0f7fe..b19b6eb 100644
--- a/src/app/components/header/header.component.html
+++ b/src/app/components/header/header.component.html
@@ -7,7 +7,7 @@
 			<div (click)="navigate('/stats')">Stats</div>
 		</div>
 		<div id="menu-mobile">
-			<div id="menu-mobile-dropdown-button" (click)="openMobileDropdown()">Open</div>
+			<div id="menu-mobile-dropdown-button" (click)="toggleMobileDropdown()">Open</div>
 			<div id="menu-mobile-dropdown-content">
 				<div (click)="navigate('/')">Home</div>
 				<div (click)="navigate('/gamenight')">Game night</div>
diff --git a/src/app/components/header/header.component.scss b/src/app/components/header/header.component.scss
index 8947f50..8d506d7 100644
--- a/src/app/components/header/header.component.scss
+++ b/src/app/components/header/header.component.scss
@@ -43,6 +43,7 @@
 		display: inherit;
 	}
 }
+
 @media(max-width: 767px) {
 	#menu-desktop {
 		display: none;
diff --git a/src/app/components/header/header.component.ts b/src/app/components/header/header.component.ts
index 1213eea..b71beb2 100644
--- a/src/app/components/header/header.component.ts
+++ b/src/app/components/header/header.component.ts
@@ -1,7 +1,7 @@
 import {Component, OnInit} from '@angular/core';
 import {User} from '../../models/user';
 import {StorageService} from '../../services/storage.service';
-import {Router} from "@angular/router";
+import {Router} from '@angular/router';
 
 @Component({
 	selector: 'app-header',
@@ -13,7 +13,7 @@ export class HeaderComponent implements OnInit {
 	showProfilePopOver: boolean = false;
 	user?: User;
 
-	constructor(private router:Router) {
+	constructor(private router: Router) {
 		this.user = StorageService.getUserInfo();
 	}
 
@@ -28,20 +28,22 @@ export class HeaderComponent implements OnInit {
 		this.showProfilePopOver = false;
 	}
 
+
 	getUserName(): string {
 		return this.user ? this.user.firstName : 'Logged out';
 	}
 
-	navigate(path:String) : void {
+	navigate(path: String): void {
 		this.router.navigate([path]);
 	}
 
-	openMobileDropdown() : void {
-		const dropdownContentStyle = document.getElementById("menu-mobile-dropdown-content")!.style;
-		if(dropdownContentStyle.display == "block")
-			dropdownContentStyle.display = "none";
-		else {
-			dropdownContentStyle.display = "block";
+	toggleMobileDropdown(): void {
+		const dropdownContentStyle = document.getElementById('menu-mobile-dropdown-content')!.style;
+		if (dropdownContentStyle.display == 'block') {
+			//FIXME: Currently doesn't work because content overlaps button
+			dropdownContentStyle.display = 'none';
+		} else {
+			dropdownContentStyle.display = 'block';
 		}
 	}
 }