From 7b59f404e26e1702adf8f91c70f6e11df5504162 Mon Sep 17 00:00:00 2001
From: Josaphat Imani <josaphat.imani97@gmail.com>
Date: Mon, 18 Nov 2024 15:19:52 +0200
Subject: [PATCH] Reduce left menu height to make it appealing

---
 modules/core/output_modules.php               | 2 +-
 modules/core/site.css                         | 5 ++++-
 modules/imap/site.css                         | 6 ++----
 tests/phpunit/modules/core/output_modules.php | 2 +-
 4 files changed, 8 insertions(+), 7 deletions(-)

diff --git a/modules/core/output_modules.php b/modules/core/output_modules.php
index 27bbd1be92..cbdb3ea13c 100644
--- a/modules/core/output_modules.php
+++ b/modules/core/output_modules.php
@@ -15,7 +15,7 @@ class Hm_Output_search_from_folder_list extends Hm_Output_Module {
      * Add a search form to the top of the folder list
      */
     protected function output() {
-        $res = '<li class="menu_search mb-4"><form method="get">';
+        $res = '<li class="menu_search mb-2"><form method="get">';
         $res .= '<div class="input-group">';
         if (!$this->get('hide_folder_icons')) {
             $res .= '<a href="?page=search" class="input-group-text" id="basic-addon1">' . 
diff --git a/modules/core/site.css b/modules/core/site.css
index 705073b918..176ea7b161 100644
--- a/modules/core/site.css
+++ b/modules/core/site.css
@@ -382,6 +382,7 @@ button {
 }
 .inner_list {
   margin-bottom: 10px;
+  margin-top: 5px;
 }
 .folders a,
 .unread_link,.all_tags {
@@ -415,9 +416,11 @@ button {
   padding: 0.5rem;
   border-radius: var(--bs-border-radius);
   transition: all linear 0.3s;
-  margin-bottom: 10px;
   position: relative;
 }
+.folders li a {
+  display: block;
+}
 .inner_list li {
   margin: 2px;
   white-space: nowrap;
diff --git a/modules/imap/site.css b/modules/imap/site.css
index 7dad732212..ddc00f994b 100644
--- a/modules/imap/site.css
+++ b/modules/imap/site.css
@@ -535,15 +535,13 @@
 .fade-in {
   animation-name: fadeIn;
 }
-li[class^="imap_"] .inner_list li[class^="imap_"] {
-    /* display: flex;
-    align-items: center; */
+li[class^="imap_"] .inner_list li[class^="imap_"], li.manage_folders_li {
     position: relative;
     display: flex;
     align-items: center;
     border-radius: 4px;
-    margin: 2px 0;
     transition: background-color .3s;
+    padding: 5px 7px;
 }
 .inner_list li[class^="imap_"] #main-link {
     flex-grow: 1;
diff --git a/tests/phpunit/modules/core/output_modules.php b/tests/phpunit/modules/core/output_modules.php
index cb7260dcf6..4d2317be89 100644
--- a/tests/phpunit/modules/core/output_modules.php
+++ b/tests/phpunit/modules/core/output_modules.php
@@ -18,7 +18,7 @@ public function test_search_from_folder_list() {
         $test->run();
         $test->rtype = 'AJAX';
         $res = $test->run();
-        $this->assertEquals('<li class="menu_search mb-4"><form method="get"><div class="input-group"><a href="?page=search" class="input-group-text" id="basic-addon1"><i class="bi bi-search"></i></a><input type="hidden" name="page" value="search" /><input type="search" class="search_terms form-control form-control-sm" aria-describedby="basic-addon1" name="search_terms" placeholder="Search" /></div></form></li>', $res->output_data['formatted_folder_list']);
+        $this->assertEquals('<li class="menu_search mb-2"><form method="get"><div class="input-group"><a href="?page=search" class="input-group-text" id="basic-addon1"><i class="bi bi-search"></i></a><input type="hidden" name="page" value="search" /><input type="search" class="search_terms form-control form-control-sm" aria-describedby="basic-addon1" name="search_terms" placeholder="Search" /></div></form></li>', $res->output_data['formatted_folder_list']);
     }
     /**
      * @preserveGlobalState disabled