Added style for expandable-admin-sidebar-section to sidebar-sub-level-item-list's childs

Author: jr-rk

src/app/admin/admin-sidebar/expandable-admin-sidebar-section/expandable-admin-sidebar-section.component.html

@@ -34,7 +34,7 @@
     <div class="sidebar-collapsible-element-outer-wrapper">
       <div class="sidebar-collapsible-element-inner-wrapper">
         <div class="sidebar-sub-level-item-list" role="menu" [id]="adminMenuSectionId(section.id)" [attr.aria-label]="('menu.section.' + section.id) | translate">
-          <div *ngFor="let subSection of (subSections$ | async)">
+          <div class="sidebar-item" *ngFor="let subSection of (subSections$ | async)">
             {{ (sectionMap$ | async).get(subSection.id).component | json}}
             <ng-container
               *ngComponentOutlet="(sectionMap$ | async).get(subSection.id).component; injector: (sectionMap$ | async).get(subSection.id).injector;"></ng-container>

src/app/admin/admin-sidebar/expandable-admin-sidebar-section/expandable-admin-sidebar-section.component.scss

@@ -3,8 +3,30 @@
     list-style: none;
     margin: 0;
     padding: 0;
-    ::ng-deep .ds-menu-item {
-      padding-left: var(--ds-admin-sidebar-section-indent);
+    /* Make the entire sub-level row clickable by applying padding to the anchor */
+    ::ng-deep {
+      /* Ensure anchors are full-width blocks */
+      .ds-menu-item {
+        display: block;
+        width: 100%;
+        padding-left: var(--ds-admin-sidebar-section-indent);
+      }
+
+      /* Remove vertical padding from the wrapper and apply it to the anchor */
+      .sidebar-item {
+        padding-top: 0 !important;
+        padding-bottom: 0 !important;
+      }
+      .sidebar-item .ds-menu-item {
+        padding-top: var(--ds-admin-sidebar-item-padding);
+        padding-bottom: var(--ds-admin-sidebar-item-padding);
+      }
+
+      /* In case an anchor also has .sidebar-item, reset it to avoid double padding */
+      .ds-menu-item.sidebar-item {
+        padding-top: 0 !important;
+        padding-bottom: 0 !important;
+      }
     }
   }
 

src/app/shared/menu/menu-item/onclick-menu-item.component.html

@@ -1,5 +1,5 @@
 <a *ngIf="!item.disabled"
-   class="ds-menu-item sidebar-item" role="menuitem"
+   class="ds-menu-item" role="menuitem"
    routerLinkActive="active"
    href="javascript:void(0);"
    [title]="item.text | translate"

src/app/shared/menu/menu-item/onclick-menu-item.component.scss

@@ -1,5 +1,3 @@
-.ds-menu-item {
+a {
     cursor: pointer;
-    display: block;
-    width: 100%;
 }