From f9de601841db1ce586b49c158de34682a745052c Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Javi=20Mart=C3=ADn?= <javim@elretirao.net>
Date: Sat, 5 Dec 2020 16:04:03 +0100
Subject: [PATCH] Fix double <nav> tag in valuation menu

We were defining a <nav> tag which was already defined in the admin
layout.
---
 app/views/valuation/_menu.html.erb | 22 ++++++++++------------
 1 file changed, 10 insertions(+), 12 deletions(-)

diff --git a/app/views/valuation/_menu.html.erb b/app/views/valuation/_menu.html.erb
index d7d609cdd..dddb16c35 100644
--- a/app/views/valuation/_menu.html.erb
+++ b/app/views/valuation/_menu.html.erb
@@ -1,13 +1,11 @@
-<nav class="admin-sidebar">
-  <ul id="valuation_menu">
-    <li>
-      <%= link_to t("valuation.menu.title"), valuation_root_path %>
-    </li>
+<ul id="valuation_menu">
+  <li>
+    <%= link_to t("valuation.menu.title"), valuation_root_path %>
+  </li>
 
-    <% if feature?(:budgets) %>
-      <li <%= "class=is-active" if controller_name == "budget_investments" %>>
-        <%= link_to t("valuation.menu.budgets"), valuation_budgets_path, class: "budgets-link" %>
-      </li>
-    <% end %>
-  </ul>
-</nav>
+  <% if feature?(:budgets) %>
+    <li <%= "class=is-active" if controller_name == "budget_investments" %>>
+      <%= link_to t("valuation.menu.budgets"), valuation_budgets_path, class: "budgets-link" %>
+    </li>
+  <% end %>
+</ul>