fix(AnalyticalTable): remove padding and scrollbar from select-all he…

…ader cell (#536) closes #532
SAP · May 22, 2020 · 01a257e · 01a257e
1 parent c3bb684
commit 01a257e
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 2 deletions.
diff --git a/packages/main/src/components/AnalyticalTable/AnayticalTable.jss.ts b/packages/main/src/components/AnalyticalTable/AnayticalTable.jss.ts
@@ -27,6 +27,7 @@ const styles = {
     borderTop: `1px solid ${ThemingParameters.sapList_BorderColor}`,
     borderBottom: `1px solid ${ThemingParameters.sapList_BorderColor}`,
     borderRight: `1px solid ${ThemingParameters.sapList_BorderColor}`,
+    padding: `0 0.5rem`,
     textAlign: 'start',
     boxSizing: 'border-box',
     '&:first-child': {

diff --git a/packages/main/src/components/AnalyticalTable/ColumnHeader/index.tsx b/packages/main/src/components/AnalyticalTable/ColumnHeader/index.tsx
@@ -60,7 +60,7 @@ const styles = {
     position: 'relative',
     width: '100%',
     overflowX: 'hidden',
-    padding: `0 0.5rem`,
+    overflowY: 'hidden',
     boxSizing: 'border-box',
     '&[data-h-align="End"]': {
       '& $text': {

diff --git a/packages/main/src/components/AnalyticalTable/index.tsx b/packages/main/src/components/AnalyticalTable/index.tsx
@@ -368,7 +368,7 @@ const AnalyticalTable: FC<TableProps> = forwardRef((props: TableProps, ref: Ref<
           return (
             // eslint-disable-next-line react/jsx-key
             <header {...headerProps} role="rowgroup">
-              {headerGroup.headers.map((column, index) => (
+              {headerGroup.headers.map((column) => (
                 // eslint-disable-next-line react/jsx-key
                 <ColumnHeader
                   {...column.getHeaderProps()}