Skip to content

Commit c7d1cda

Browse files
committed
fix: fix scrolling issues with sidebar layout when nested nav sections are open; update sidebar layout to ensure viewport tools are still available on larger screens
1 parent 6655d4d commit c7d1cda

File tree

2 files changed

+10
-1
lines changed

2 files changed

+10
-1
lines changed

packages/uikit-workshop/src/sass/scss/04-components/_navigation.scss

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,10 @@
3434
overflow: visible;
3535
}
3636

37+
.pl-c-body--theme-sidebar & {
38+
overflow: hidden;
39+
}
40+
3741
.pl-c-body--theme-light & {
3842
box-shadow: 0 1px 1px darken($pl-color-gray-20, 15%);
3943
}
@@ -201,6 +205,11 @@
201205
height: auto;
202206
max-height: calc(100vh - #{$offset-top} - 1rem); /* 1 */
203207
}
208+
209+
.pl-c-body--theme-sidebar & {
210+
overflow: hidden;
211+
max-height: none;
212+
}
204213
}
205214

206215
.pl-c-nav__sublist--dropdown.pl-is-active {

packages/uikit-workshop/src/sass/scss/05-themes/_sidebar-theme.scss

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,6 @@
2121
border-bottom: 0; /* 4 */
2222
padding: 1rem;
2323
overflow: auto;
24-
@include hideScrollBar();
2524
justify-content: space-between;
2625
}
2726

@@ -140,6 +139,7 @@
140139
* they appear at the bottom
141140
*/
142141
.pl-c-controls {
142+
display: flex;
143143
justify-self: flex-end;
144144
margin-left: 0;
145145
flex-direction: column;

0 commit comments

Comments
 (0)