From 471b3bc55709008991abc27ce84d22a77651e40d Mon Sep 17 00:00:00 2001
From: Arne Vlaeminck <vlaeminck.ar@gmail.com>
Date: Sun, 30 Jul 2023 21:39:23 +0200
Subject: [PATCH] Design review

---
 src/app/app.component.html               | 16 ++++++++--------
 src/app/collapse/collapse.component.html |  2 +-
 src/app/collapse/collapse.component.scss | 14 +++++++++++++-
 src/app/header/header.component.html     |  2 +-
 src/index.html                           |  1 +
 src/styles.scss                          |  3 ++-
 6 files changed, 26 insertions(+), 12 deletions(-)

diff --git a/src/app/app.component.html b/src/app/app.component.html
index 62ffbbd..b7b8e48 100644
--- a/src/app/app.component.html
+++ b/src/app/app.component.html
@@ -20,8 +20,8 @@
     <app-collapse [icon]="'fa-book-open'" [title]="'Boekhouding/Accountancy'">
       <ul>
         <li>Het voeren van de boekhouding van A tot Z (met inbegrip van alle BTW verplichtingen)</li>
-        <li>ul bezorgen u geregeld (of op verzoek) een tussentijds resultaat zodat onder meer</li>
-        <li>voorafbetalingen van belasting optimaal kunnen opgevolgd worden</li>
+        <li>Bezorgen u geregeld (of op verzoek) een tussentijds resultaat zodat onder meer</li>
+        <li>Voorafbetalingen van belasting optimaal kunnen opgevolgd worden</li>
         <li>Ondersteuning van uw interne boekhoudafdeling</li>
         <li>Opmaken van de interne jaarrekening en de statutaire jaarrekening (NBB)</li>
         <li>Opstellen van de verslagen aan en van de algemene vergadering</li>
@@ -128,11 +128,11 @@
         ter zake bij onder meer :
       </p>
       <ul>
-        <li>huurovereenkomsten</li>
-        <li>onderhandse leningsovereenkomsten</li>
-        <li>overeenkomsten voor het kopen, verkopen, overdracht van aandelen</li>
-        <li>overeenkomsten voor het kopen en verkopen van handelsfonds</li>
-        <li>overeenkomsten van recht van opstal</li>
+        <li>Huurovereenkomsten</li>
+        <li>Onderhandse leningsovereenkomsten</li>
+        <li>Overeenkomsten voor het kopen, verkopen, overdracht van aandelen</li>
+        <li>Overeenkomsten voor het kopen en verkopen van handelsfonds</li>
+        <li>Overeenkomsten van recht van opstal</li>
     </ul>
     </app-collapse>
   </div>
@@ -229,7 +229,7 @@
     <a href="tel:+32475707887"><span class="fa-solid fa-mobile-screen"></span>0475/70.78.87 (Robert Sinia)</a>
     <a href="mailto:robert@sinia.be,frank@sinia.be"><span class="fa-solid fa-envelope"></span>robert@sinia.be / frank@sinia.be</a>
     <a href="https://www.linkedin.com/company/sinia-accountants-en-belastingconsulenten-bvba/about/" target="_blank"><span class="fa-brands fa-linkedin"></span>Linkedin</a>
-    <a href="/"><span class="fa-solid fa-home"></span>Homepage</a>
+    <a href="#"><span class="fa-solid fa-home"></span>Homepage</a>
   </div>
   <div class="google-maps">
     <iframe src="https://www.google.com/maps/embed?pb=!1m14!1m8!1m3!1d26011.531706816328!2d3.7327019887977553!3d51.06778304400319!3m2!1i1024!2i768!4f13.1!3m3!1m2!1s0x47c376cf113b3e6f%3A0x3f11537833278a46!2sSinia%20Accountants%20En%20Belastingconsulenten!5e0!3m2!1sen!2sbe!4v1690233562850!5m2!1sen!2sbe" width="600" height="450" style="border:0;" allowfullscreen="" loading="lazy" referrerpolicy="no-referrer-when-downgrade" title="google maps"></iframe>
diff --git a/src/app/collapse/collapse.component.html b/src/app/collapse/collapse.component.html
index 6ac9e29..ab15746 100644
--- a/src/app/collapse/collapse.component.html
+++ b/src/app/collapse/collapse.component.html
@@ -2,7 +2,7 @@
   <div class="heading" (click)="collapsed = !collapsed">
     <span class="fa-solid {{icon}} icon"></span>
     <h3>{{title}}</h3>
-    <span class="fa-solid" [ngClass]="{'fa-chevron-up': !collapsed, 'fa-chevron-down': collapsed}"></span>
+    <span class="fa-solid fa-chevron-up" [ngClass]="{'up': !collapsed, 'down': collapsed}"></span>
   </div>
   <div class="collapse" [ngClass]="{'collapsed': collapsed}">
     <div class="collapse-container">
diff --git a/src/app/collapse/collapse.component.scss b/src/app/collapse/collapse.component.scss
index 3c3d085..c04783b 100644
--- a/src/app/collapse/collapse.component.scss
+++ b/src/app/collapse/collapse.component.scss
@@ -38,8 +38,20 @@ span {
   font-size: 2rem;
 }
 
+.fa-chevron-up {
+
+  &.up {
+    transition: all 0.5s ease;
+  }
+
+  &.down {
+    transform: rotate(-180deg);
+    transition: all 0.5s ease;
+  }
+}
+
 .collapse {
-  max-height: 50rem;
+  max-height: 80rem;
   transition: max-height 0.5s ease-in;
   overflow: hidden;
 }
diff --git a/src/app/header/header.component.html b/src/app/header/header.component.html
index 3486199..fa8be93 100644
--- a/src/app/header/header.component.html
+++ b/src/app/header/header.component.html
@@ -1,5 +1,5 @@
 <div class="logo">
-  <a href ="/" class="logo" ><img src="assets/logo.jpg" alt="logo"></a>
+  <a href ="#" class="logo" ><img src="assets/logo.jpg" alt="logo"></a>
   <div class="subtext">
     <h1>Sinia</h1>
     <p>Accountants en Belastingconsultenten BV</p>
diff --git a/src/index.html b/src/index.html
index 3c527f8..b93e55f 100644
--- a/src/index.html
+++ b/src/index.html
@@ -19,6 +19,7 @@
     body *{
       font-family: 'Inter';
       font-size: 2rem;
+      line-height: 1.5;
     }
 
     body {
diff --git a/src/styles.scss b/src/styles.scss
index d8bc9ff..ab4f93a 100644
--- a/src/styles.scss
+++ b/src/styles.scss
@@ -21,5 +21,6 @@ li {
 }
 
 ul {
-  margin-top: 0.25rem;
+  margin-top: 1rem;
+
 }