Merge pull request #1691 from anujsharma8d/fix/remove-extra-sidebar-closebtn

fix: remove extra sidebar close button

Author: steam-bell-92

web-app/css/styles.css

@@ -731,24 +731,6 @@ body.sidebar-active .sidebar-dock {
 }
 
 
-.sidebar-mobile-close {
-  display: none;
-  position: absolute;
-  top: 16px;
-  right: 16px;
-  width: 36px;
-  height: 36px;
-  border: none;
-  border-radius: 50%;
-  background: var(--bg-glass);
-  color: var(--text-primary);
-  font-size: 16px;
-  align-items: center;
-  justify-content: center;
-  cursor: pointer;
-  z-index: 1001;
-}
-
 .sidebar-backdrop {
   display: none;
   position: fixed;
@@ -4518,9 +4500,6 @@ button:focus-visible,
 }
 
 @media (max-width: 768px) {
-  .sidebar-mobile-close {
-    display: flex;
-  }
 
   body.sidebar-active .sidebar-backdrop {
     display: block;

web-app/index.html

@@ -54,9 +54,6 @@
   <div aria-hidden="true" class="cursor-glow" id="cursorGlow"></div>
 
   <aside class="sidebar-dock" id="mainSidebar">
-    <button class="sidebar-mobile-close" id="sidebarMobileClose" aria-label="Close sidebar" type="button">
-      <i class="fas fa-times" aria-hidden="true"></i>
-    </button>
     <div class="sidebar-search">
       <div class="search-box" style="position: relative">
         <i class="fas fa-search" aria-hidden="true"></i>

web-app/js/main.js

@@ -410,7 +410,6 @@ document.addEventListener("DOMContentLoaded", function () {
     });
   }
 
-  var sidebarMobileClose = document.getElementById("sidebarMobileClose");
   var sidebarBackdrop = document.getElementById("sidebarBackdrop");
 
   function closeMobileSidebar() {
@@ -437,9 +436,6 @@ document.addEventListener("DOMContentLoaded", function () {
     closeMobileSidebar();
   }
 
-  if (sidebarMobileClose) {
-    sidebarMobileClose.addEventListener("click", closeMobileSidebar);
-  }
   if (sidebarBackdrop) {
     sidebarBackdrop.addEventListener("click", closeMobileSidebar);
   }