Skip to content

Commit f695215

Browse files
authored
feat(AnalyticalTable): add horizontal scrollbar in fiori design (#583)
1 parent 5413184 commit f695215

File tree

3 files changed

+24
-15
lines changed

3 files changed

+24
-15
lines changed

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

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

packages/main/src/components/AnalyticalTable/__snapshots__/AnalyticalTable.test.tsx.snap

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ exports[`AnalyticalTable Alternate Row Color 1`] = `
1616
<div
1717
aria-colcount="4"
1818
aria-rowcount="5"
19-
class="AnalyticalTable-table-0"
19+
class="AnalyticalTable-table-0 sapScrollBar"
2020
data-per-page="15"
2121
role="grid"
2222
>
@@ -523,7 +523,7 @@ exports[`AnalyticalTable Loading - Loader 1`] = `
523523
<div
524524
aria-colcount="4"
525525
aria-rowcount="5"
526-
class="AnalyticalTable-table-0"
526+
class="AnalyticalTable-table-0 sapScrollBar"
527527
data-per-page="15"
528528
role="grid"
529529
>
@@ -1030,7 +1030,7 @@ exports[`AnalyticalTable Loading - Placeholder 1`] = `
10301030
<div
10311031
aria-colcount="4"
10321032
aria-rowcount="5"
1033-
class="AnalyticalTable-table-0"
1033+
class="AnalyticalTable-table-0 sapScrollBar"
10341034
data-per-page="15"
10351035
role="grid"
10361036
>
@@ -1529,7 +1529,7 @@ exports[`AnalyticalTable Tree Table 1`] = `
15291529
<div
15301530
aria-colcount="5"
15311531
aria-rowcount="5"
1532-
class="AnalyticalTable-table-0"
1532+
class="AnalyticalTable-table-0 sapScrollBar"
15331533
data-per-page="15"
15341534
role="grid"
15351535
>
@@ -2179,7 +2179,7 @@ exports[`AnalyticalTable custom row height 1`] = `
21792179
<div
21802180
aria-colcount="4"
21812181
aria-rowcount="5"
2182-
class="AnalyticalTable-table-0"
2182+
class="AnalyticalTable-table-0 sapScrollBar"
21832183
data-per-page="15"
21842184
role="grid"
21852185
>
@@ -2686,7 +2686,7 @@ exports[`AnalyticalTable render without data 1`] = `
26862686
<div
26872687
aria-colcount="4"
26882688
aria-rowcount="5"
2689-
class="AnalyticalTable-table-0"
2689+
class="AnalyticalTable-table-0 sapScrollBar"
26902690
data-per-page="15"
26912691
role="grid"
26922692
>
@@ -2958,7 +2958,7 @@ exports[`AnalyticalTable test Asc desc 1`] = `
29582958
<div
29592959
aria-colcount="4"
29602960
aria-rowcount="5"
2961-
class="AnalyticalTable-table-0"
2961+
class="AnalyticalTable-table-0 sapScrollBar"
29622962
data-per-page="15"
29632963
role="grid"
29642964
>
@@ -3465,7 +3465,7 @@ exports[`AnalyticalTable test Asc desc 2`] = `
34653465
<div
34663466
aria-colcount="4"
34673467
aria-rowcount="5"
3468-
class="AnalyticalTable-table-0"
3468+
class="AnalyticalTable-table-0 sapScrollBar"
34693469
data-per-page="15"
34703470
role="grid"
34713471
>
@@ -3972,7 +3972,7 @@ exports[`AnalyticalTable test Asc desc 3`] = `
39723972
<div
39733973
aria-colcount="4"
39743974
aria-rowcount="5"
3975-
class="AnalyticalTable-table-0"
3975+
class="AnalyticalTable-table-0 sapScrollBar"
39763976
data-per-page="15"
39773977
role="grid"
39783978
>
@@ -4479,7 +4479,7 @@ exports[`AnalyticalTable test drag and drop of a draggable column 1`] = `
44794479
<div
44804480
aria-colcount="4"
44814481
aria-rowcount="5"
4482-
class="AnalyticalTable-table-0"
4482+
class="AnalyticalTable-table-0 sapScrollBar"
44834483
data-per-page="15"
44844484
role="grid"
44854485
>
@@ -4986,7 +4986,7 @@ exports[`AnalyticalTable with highlight row 1`] = `
49864986
<div
49874987
aria-colcount="6"
49884988
aria-rowcount="5"
4989-
class="AnalyticalTable-table-0"
4989+
class="AnalyticalTable-table-0 sapScrollBar"
49904990
data-per-page="15"
49914991
role="grid"
49924992
>
@@ -5621,7 +5621,7 @@ exports[`AnalyticalTable without selection Column 1`] = `
56215621
<div
56225622
aria-colcount="4"
56235623
aria-rowcount="5"
5624-
class="AnalyticalTable-table-0"
5624+
class="AnalyticalTable-table-0 sapScrollBar"
56255625
data-per-page="15"
56265626
role="grid"
56275627
>

packages/main/src/components/AnalyticalTable/hooks/useStyling.ts

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,17 +1,20 @@
1+
import { getRTL } from '@ui5/webcomponents-base/dist/config/RTL';
2+
import { StyleClassHelper } from '@ui5/webcomponents-react-base/lib/StyleClassHelper';
13
import { ThemingParameters } from '@ui5/webcomponents-react-base/lib/ThemingParameters';
4+
import { GlobalStyleClasses } from '@ui5/webcomponents-react/lib/GlobalStyleClasses';
25
import { TableSelectionBehavior } from '@ui5/webcomponents-react/lib/TableSelectionBehavior';
36
import { TableSelectionMode } from '@ui5/webcomponents-react/lib/TableSelectionMode';
47
import { TextAlign } from '@ui5/webcomponents-react/lib/TextAlign';
58
import { VerticalAlign } from '@ui5/webcomponents-react/lib/VerticalAlign';
6-
import { getRTL } from '@ui5/webcomponents-base/dist/config/RTL';
79
import { CSSProperties } from 'react';
810

911
const getTableProps = (tableProps, { instance }) => {
1012
const { classes } = instance.webComponentsReactProperties;
13+
const classNames = StyleClassHelper.of(classes.table, GlobalStyleClasses.sapScrollBar).className;
1114
return [
1215
tableProps,
1316
{
14-
className: classes.table
17+
className: classNames
1518
}
1619
];
1720
};

0 commit comments

Comments
 (0)