From 90ca78a0bd32dd10176c0b75bda77de1e225e7dc Mon Sep 17 00:00:00 2001
From: decabeza <alberto@decabeza.es>
Date: Wed, 10 Apr 2019 16:17:03 +0200
Subject: [PATCH] Show actions section only if there is any action

---
 app/views/dashboard/_menu.html.erb    | 20 +++++++++++---------
 app/views/dashboard/progress.html.erb |  6 +++++-
 2 files changed, 16 insertions(+), 10 deletions(-)

diff --git a/app/views/dashboard/_menu.html.erb b/app/views/dashboard/_menu.html.erb
index c6d633e79..f38e492ba 100644
--- a/app/views/dashboard/_menu.html.erb
+++ b/app/views/dashboard/_menu.html.erb
@@ -13,15 +13,17 @@
     <% end %>
   </li>
 
-  <li class="section-title <%= 'is-active' if recommended_actions_menu_active? %>">
-    <span class="icon-checkmark-circle"></span>
-    <%= link_to recommended_actions_proposal_dashboard_path(proposal.to_param) do %>
-      <strong><%= t("dashboard.menu.recommended_actions") %></strong>
-    <% end %>
-    <% if new_resources_since_last_login?(proposed_actions, @new_actions_since_last_login) %>
-      <span class="label"><%= t("dashboard.progress.new_action" ) %></span>
-    <% end %>
-  </li>
+  <% if proposed_actions.any? %>
+    <li class="section-title <%= 'is-active' if recommended_actions_menu_active? %>">
+      <span class="icon-checkmark-circle"></span>
+      <%= link_to recommended_actions_proposal_dashboard_path(proposal.to_param) do %>
+        <strong><%= t("dashboard.menu.recommended_actions") %></strong>
+      <% end %>
+      <% if new_resources_since_last_login?(proposed_actions, @new_actions_since_last_login) %>
+        <span class="label"><%= t("dashboard.progress.new_action" ) %></span>
+      <% end %>
+    </li>
+  <% end %>
 
   <% if resources_menu_visible?(proposal, resources) %>
     <li class="section-title <%= 'is-active' if resources_menu_active? %>">
diff --git a/app/views/dashboard/progress.html.erb b/app/views/dashboard/progress.html.erb
index 45b4547b9..95c3f0297 100644
--- a/app/views/dashboard/progress.html.erb
+++ b/app/views/dashboard/progress.html.erb
@@ -33,5 +33,9 @@
 <% end %>
 
 <%= render 'next_goal' %>
-<%= render 'summary_recommended_actions' %>
+
+<% if @pending_actions.any? || @done_actions.any? %>
+  <%= render 'summary_recommended_actions' %>
+<% end %>
+
 <%= render 'resources' %>