Skip to content

Commit 3ceb0de

Browse files
fix(sapScrollBar): use correct height for horizontal scrollbar (#587)
1 parent 330d639 commit 3ceb0de

File tree

2 files changed

+13
-9
lines changed

2 files changed

+13
-9
lines changed

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

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -12,13 +12,7 @@ const styles = {
1212
minHeight: '3rem',
1313
fontFamily: ThemingParameters.sapFontFamily,
1414
fontSize: ThemingParameters.sapFontSize,
15-
fontWeight: 'normal',
16-
'&::-webkit-scrollbar': {
17-
height: ThemingParameters.sapScrollBar_Dimension
18-
},
19-
'&::-webkit-scrollbar-thumb': {
20-
height: ThemingParameters.sapScrollBar_Dimension
21-
}
15+
fontWeight: 'normal'
2216
},
2317
tableHeaderRow: {
2418
height: CssSizeVariables.sapWcrAnalyticalTableRowHeight,

packages/main/src/components/ThemeProvider/GlobalStyleClasses.jss.ts

Lines changed: 12 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,14 +6,24 @@ export const GlobalStyleClassesStyles = {
66
[`.${GlobalStyleClasses.sapScrollBar}`]: {
77
'&::-webkit-scrollbar': {
88
backgroundColor: ThemingParameters.sapScrollBar_TrackColor,
9-
width: ThemingParameters.sapScrollBar_Dimension
9+
'&:horizontal': {
10+
height: ThemingParameters.sapScrollBar_Dimension
11+
},
12+
'&:vertical': {
13+
width: ThemingParameters.sapScrollBar_Dimension
14+
}
1015
},
1116
'&::-webkit-scrollbar-thumb': {
1217
backgroundColor: ThemingParameters.sapScrollBar_FaceColor,
13-
width: ThemingParameters.sapScrollBar_Dimension,
1418
borderRadius: '0.25rem',
1519
'&:hover': {
1620
backgroundColor: ThemingParameters.sapScrollBar_Hover_FaceColor
21+
},
22+
'&:horizontal': {
23+
height: ThemingParameters.sapScrollBar_Dimension
24+
},
25+
'&:vertical': {
26+
width: ThemingParameters.sapScrollBar_Dimension
1727
}
1828
},
1929
'&::-webkit-scrollbar-corner': {

0 commit comments

Comments
 (0)