Fix Category not respecting is_visible and is_accessible (#698)

Co-authored-by: Amin Alaee <mohammadamin.alaee@gmail.com>

sqladmin/_menu.py

@@ -42,7 +42,7 @@ def type_(self) -> str:
 class CategoryMenu(ItemMenu):
     def is_active(self, request: Request) -> bool:
         return any(
-            c.is_active(request) and c.is_accessible(request) for c in self.children
+            c.is_visible(request) and c.is_accessible(request) for c in self.children
         )
 
     @property

sqladmin/templates/_macros.html

@@ -1,5 +1,5 @@
 {% macro menu_category(menu, request) %}
-{% if menu.is_visible(request) and menu.is_accessible(request) %}
+{% if menu.is_active(request) %}
 <li class="nav-item dropdown">
   <a class="nav-link dropdown-toggle {% if menu.is_active(request) %}active{% endif %}" data-bs-toggle="dropdown"
     href="#">
@@ -12,13 +12,15 @@
     <div class="dropdown-menu-columns">
       <div class="dropdown-menu-column">
         {% for sub_menu in menu.children %}
+        {% if sub_menu.is_visible(request) and sub_menu.is_accessible(request) %}
         <a class="nav-link ps-lg-3 {% if sub_menu.is_active(request) %}active{% endif %}"
           href="{{ sub_menu.url(request) }}">
           <span class="nav-link-icon d-md-none d-lg-inline-block">
             {% if sub_menu.icon %}<i class="{{ sub_menu.icon }}"></i>{% endif %}
           </span>
           <span class="nav-link-title">{{ sub_menu.display_name }}</span>
         </a>
+        {% endif %}
         {% endfor %}
       </div>
     </div>
@@ -50,4 +52,4 @@
   {% endif %}
   {% endfor %}
 </div>
-{% endmacro %}
+{% endmacro %}