Skip to content

Commit 4ae5af6

Browse files
committed
Copilot suggestion refactoring - removed the !important keywords
1 parent a59a9d6 commit 4ae5af6

2 files changed

Lines changed: 9 additions & 11 deletions

File tree

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

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -56,6 +56,15 @@
5656
padding-bottom: var(--ds-admin-sidebar-item-padding);
5757
}
5858

59+
// Remove vertical padding from the wrapper and apply it to the anchor
60+
// In case an anchor also has .sidebar-item, reset it to avoid double padding
61+
.sidebar-sub-level-item-list {
62+
.sidebar-item, .ds-menu-item.sidebar-item {
63+
padding-top: 0;
64+
padding-bottom: 0;
65+
}
66+
}
67+
5968
// These classes handle the collapsing behavior
6069
.sidebar-section-wrapper {
6170

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

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -12,21 +12,10 @@
1212
padding-left: var(--ds-admin-sidebar-section-indent);
1313
}
1414

15-
/* Remove vertical padding from the wrapper and apply it to the anchor */
16-
.sidebar-item {
17-
padding-top: 0 !important;
18-
padding-bottom: 0 !important;
19-
}
2015
.sidebar-item .ds-menu-item {
2116
padding-top: var(--ds-admin-sidebar-item-padding);
2217
padding-bottom: var(--ds-admin-sidebar-item-padding);
2318
}
24-
25-
/* In case an anchor also has .sidebar-item, reset it to avoid double padding */
26-
.ds-menu-item.sidebar-item {
27-
padding-top: 0 !important;
28-
padding-bottom: 0 !important;
29-
}
3019
}
3120
}
3221

0 commit comments

Comments
 (0)