Skip to content

Commit f7e123f

Browse files
committed
fix(layout): remove duplicate scrollbar UI by clipping wrapper overflow and hiding sidebar scrollbar
1 parent b07bcad commit f7e123f

2 files changed

Lines changed: 10 additions & 3 deletions

File tree

src/app/app.component.scss

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,10 +4,10 @@
44
@use '../assets/layout/responsive';
55

66
.layout-wrapper {
7-
min-height: 100vh;
7+
min-height: 100dvh;
88
position: relative;
99
overflow-x: hidden;
10-
overflow-y: visible;
10+
overflow-y: clip;
1111
min-width: 0;
1212
}
1313

@@ -56,7 +56,7 @@ app-sidebar {
5656
.layout-main-container {
5757
display: flex;
5858
flex-direction: column;
59-
min-height: 100vh;
59+
min-height: calc(100dvh - 4rem);
6060
justify-content: space-between;
6161
padding: 6rem 2rem 0 2rem;
6262
transition: margin-left .2s;
@@ -76,6 +76,7 @@ app-sidebar {
7676
}
7777

7878
.layout-main-container {
79+
min-height: calc(100dvh - 4rem);
7980
min-width: 0;
8081
width: 100%;
8182
max-width: 100%;

src/app/core/layout/component/app-menu/app-menu.component.scss

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,12 @@
1414
border-radius: var(--border-radius);
1515
box-shadow: 0 1px 3px 0 rgba(0, 0, 0, 0.1), 0 1px 2px -1px rgba(0, 0, 0, 0.1);
1616
padding: 0.5rem 1.5rem;
17+
scrollbar-width: none;
18+
-ms-overflow-style: none;
19+
20+
&::-webkit-scrollbar {
21+
display: none;
22+
}
1723
}
1824

1925
.layout-menu {

0 commit comments

Comments
 (0)