Show pie chart only if there is more than 1 account

simonmichael · Jan 25, 2021 · c9d25a8 · c9d25a8
1 parent 6ef0f22
commit c9d25a8
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/hledger-web/Hledger/Web/Handler/RegisterR.hs b/hledger-web/Hledger/Web/Handler/RegisterR.hs
@@ -126,6 +126,7 @@ registerPieChartHtml (items, _) = $(hamletFile "templates/piechart.hamlet")
       sortOn snd $
       flip concatMap items $ \(accname, _, _, Mixed as) ->
         flip map as $ \a -> (accname, aquantity a)
+    showChart = if ((length labelDataTuples) > 1) then "true" else "false" :: String
 
 dayToJsTimestamp :: Day -> Integer
 dayToJsTimestamp d =

diff --git a/hledger-web/templates/piechart.hamlet b/hledger-web/templates/piechart.hamlet
@@ -1,11 +1,12 @@
 <label #register-pie-chart-label style=""><br>
-<div #register-pie-chart style="height:150px; margin-bottom:1em; display:block;">
+<div #register-pie-chart style="height:150px; margin-bottom:1em; display:none;">
 <div #piehover>
 <script type=text/javascript>
  \$(document).ready(function() {
    var $chartdiv = $('#register-pie-chart');
-   if ($chartdiv.is(':visible')) {
+   if (#{showChart}) {
      \$('#register-pie-chart-label').text('#{charttitle}');
+     \$('#register-pie-chart').show();
      var data =
        [
          $forall (label, dat) <- labelDataTuples