fix: pivot-table-fixed (#1360)

apache · Nov 26, 2021 · 4352600 · 4352600
1 parent 055d9f8
commit 4352600
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 2 deletions.
diff --git a/...mporary_superset_ui/superset-ui/plugins/legacy-plugin-chart-pivot-table/src/PivotTable.js b/...mporary_superset_ui/superset-ui/plugins/legacy-plugin-chart-pivot-table/src/PivotTable.js
@@ -110,6 +110,10 @@ function PivotTable(element, props) {
       });
   });
 
+  $container.find('table').each(function fullWidth() {
+    this.style = 'width: 100%';
+  });
+
   if (numGroups === 1) {
     // When there is only 1 group by column,
     // we use the DataTable plugin to make the header fixed.

diff --git a/...emporary_superset_ui/superset-ui/plugins/plugin-chart-pivot-table/src/PivotTableChart.tsx b/...emporary_superset_ui/superset-ui/plugins/plugin-chart-pivot-table/src/PivotTableChart.tsx
@@ -43,7 +43,7 @@ const Styles = styled.div<PivotTableStylesProps>`
   ${({ height, width, margin }) => `
       margin: ${margin}px;
       height: ${height - margin * 2}px;
-      width: ${width - margin * 2}px;
+      width: ${typeof width === 'string' ? parseInt(width, 10) : width - margin * 2}px;
  `}
 `;
 

diff --git a/...-frontend/temporary_superset_ui/superset-ui/plugins/plugin-chart-pivot-table/src/types.ts b/...-frontend/temporary_superset_ui/superset-ui/plugins/plugin-chart-pivot-table/src/types.ts
@@ -31,7 +31,7 @@ import { ColorFormatters } from '@superset-ui/chart-controls';
 
 export interface PivotTableStylesProps {
   height: number;
-  width: number;
+  width: number | string;
   margin: number;
 }