Show pie chart only if there is more than 1 account

simonmichael · Jan 24, 2021 · 4dbec0d · 4dbec0d
1 parent 0497c4a
commit 4dbec0d
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 amounts) ->
         flip map amounts $ \amount -> (accname, aquantity amount)
+    show = 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 (#{show}) {
      \$('#register-pie-chart-label').text('#{charttitle}');
+     \$('#register-pie-chart').show();
      var data =
        [
          $forall (label, dat) <- labelDataTuples