Fix-254: Autosize radial and pie charts.

michael-milette · May 7, 2023 · 8be8b3b · 8be8b3b
1 parent 84ce04a
commit 8be8b3b
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 3 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,8 +1,9 @@
 # Change Log
 All notable changes to this project will be documented in this file.
 
-## [2.3.5] 2023-01-31 (dev)
+## [2.3.5] 2023-05-07
 ### Updated
+- Partial fix for sizing issue of radial and pie charts in Moodle 4.1 and 4.2.
 - Copyright notice to include 2023.
 
 ## [2.3.4] 2022-12-11

diff --git a/filter.php b/filter.php
@@ -3363,7 +3363,7 @@ function ($matches) use($mycohorts) {
                         if ($CFG->branch >= 39) {
                             $chart->set_legend_options(['display' => false]);  // Hide chart legend.
                         }
-                        $html = $OUTPUT->render_chart($chart, false);
+                        $html = '<div class="fc-chart-pie">' . $OUTPUT->render_chart($chart, false) . '</div>';
                         break;
                     case 'pie': // Tag: {chart pie 99 Label to be displayed} - Display a pie chart.
                         $chart = new \core\chart_pie();
@@ -3377,7 +3377,7 @@ function ($matches) use($mycohorts) {
                         if ($CFG->branch >= 39) {
                             $chart->set_legend_options(['display' => false]);  // Hide chart legend.
                         }
-                        $html = $OUTPUT->render_chart($chart, false);
+                        $html = '<div class="fc-chart-pie">' . $OUTPUT->render_chart($chart, false) . '</div>';
                         break;
                     case 'progressbar': // Tag: {chart progressbar 99 Label to be displayed} - Display a horizontal progres bar.
                         $html = '

diff --git a/styles.css b/styles.css
@@ -0,0 +1,3 @@
+.fc-chart-pie .chart-area .chart-image {
+    width: auto;
+}