Merge pull request #859 from alexstotsky/fix-pie-chart-cut-off

(improvement) Concentration risk pie chart
bitfinexcom · Sep 13, 2024 · f33494a · f33494a
2 parents 1785629 + d0fcba7
commit f33494a
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/src/ui/Charts/PieChart.js b/src/ui/Charts/PieChart.js
@@ -63,13 +63,19 @@ class PieChart extends PureComponent {
     return COLORS[index % COLORS.length]
   }
 
+  getChartHeight = () => {
+    const { data } = this.props
+    // base height + 18px for each 4 items row
+    return 340 + data?.length / 4 * 18
+  }
+
   render() {
     const { data } = this.props
 
     return (
       <SimplePieChart
         width={282}
-        height={340}
+        height={this.getChartHeight()}
       >
         <Pie
           data={data}