fixed issue with settings button

ma4nn · Mar 25, 2024 · b33d0c6 · b33d0c6
1 parent c2a5054
commit b33d0c6
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/README.md b/README.md
@@ -22,8 +22,7 @@ For more information see also [my blog post](https://dev-investor.de/finanz-apps
 ## Developing ⚒️
 
 ```
-make dist
-make test
+make dist && make test
 ```
 
 The compiled MoneyMoney extension can then be found in `dist/SankeyChart.lua` and installed with `make install`.

diff --git a/src/main.ts b/src/main.ts
@@ -118,7 +118,7 @@ ready(function() {
     if (Math.round(numberOfMonths) == 1) {
         document.querySelector("form input#isShowMonthlyValues").setAttribute('disabled', 'disabled');
     }
-    document.querySelector("#applySettingsButton").addEventListener('click', () => { setExcludedCategoriesFromSelect(); updateChartData() });
+    document.querySelector("#applySettingsButton").addEventListener('click', (event) => { event.preventDefault(); setExcludedCategoriesFromSelect(); updateChartData() });
 
     /** @see https://www.highcharts.com/docs/chart-and-series-types/sankey-diagram */
     chart = Highcharts.chart('chart-container', {