Skip to content

Commit c8ad8b9

Browse files
authored
fix(sidebar): avoid scrollbar on non body pushable
When a bottom sidebar was used inside non body pushable context, it was always added to the context divs height, making the vertical scrollbar appear. As the body will still provide a possible scrollbar it's save to remove the inner pushable vertical scrollbar. Reason for the behavior is the 100% translate3d() setting to hide the sidebar which adds its content to the parent div. For left/right the overflow-x:hidden is already set and for top it's a negative position of -100%, so the change of overflow-y is not affected in the other cases.
1 parent 0e6526d commit c8ad8b9

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed

src/definitions/modules/sidebar.less

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -112,6 +112,7 @@ body.pushable {
112112
/* Page Context */
113113
.pushable:not(body) {
114114
transform: translate3d(0, 0, 0);
115+
overflow-y: hidden;
115116
}
116117
.pushable:not(body) > .ui.sidebar,
117118
.pushable:not(body) > .fixed,

0 commit comments

Comments
 (0)