Skip to content

Commit 578e3cf

Browse files
authored
fix(AnalyticalTable): fix scrollbars for Firefox and Safari (#4979)
Note: Firefox lacks capabilities for customizing scrollbars, that's why there might be discrepancies noticeable between browsers. Fixes #4379
1 parent 9c9a93d commit 578e3cf

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

packages/main/src/components/AnalyticalTable/AnayticalTable.jss.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -102,7 +102,7 @@ const styles = {
102102
position: 'relative',
103103
overflowX: 'hidden',
104104
overflowY: 'auto',
105-
scrollBarWidth: 'none !important',
105+
scrollbarWidth: 'none',
106106
borderBlockEnd: `1px solid ${ThemingParameters.sapList_TableFooterBorder}`,
107107
'-ms-overflow-style': 'none',
108108
'&::-webkit-scrollbar': {

packages/main/src/components/AnalyticalTable/scrollbars/VerticalScrollbar.tsx

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,8 @@ const styles = {
2626
borderBlockEnd: `${CustomThemingParameters.AnalyticalTableHeaderBorderWidth} solid ${ThemingParameters.sapList_HeaderBorderColor}`,
2727
backgroundColor: ThemingParameters.sapList_HeaderBackground,
2828
borderInlineStart: `1px solid ${ThemingParameters.sapList_BorderColor}`,
29-
marginInlineStart: '-1px'
29+
marginInlineStart: '-1px',
30+
width: `calc(${ThemingParameters.sapScrollBar_Dimension} + 2px)`
3031
},
3132
scrollbar: {
3233
overflowY: 'auto',

0 commit comments

Comments
 (0)