-
+ <%= link_list(
+ (
+ [
+ t("pages.help.menu.debates"),
+ "#debates",
+ class: "button hollow expanded"
+ ] if feature?(:debates)
+ ),
+ (
+ [
+ t("pages.help.menu.proposals"),
+ "#proposals",
+ class: "button hollow expanded"
+ ] if feature?(:proposals)
+ ),
+ (
+ [
+ t("pages.help.menu.budgets"),
+ "#budgets",
+ class: "button hollow expanded"
+ ] if feature?(:budgets)
+ ),
+ (
+ [
+ t("pages.help.menu.polls"),
+ "#polls",
+ class: "button hollow expanded"
+ ] if feature?(:polls)
+ ),
+ (
+ [
+ t("pages.help.menu.processes"),
+ "#processes",
+ class: "button hollow expanded"
+ ]
+ ),
+ [
+ t("pages.help.menu.other"),
+ "#other",
+ class: "button hollow expanded"
+ ],
+ class: "menu-pages", "data-magellan": true
+ ) %>
diff --git a/app/views/valuation/_menu.html.erb b/app/views/valuation/_menu.html.erb
index d7d609cdd..a29fc1556 100644
--- a/app/views/valuation/_menu.html.erb
+++ b/app/views/valuation/_menu.html.erb
@@ -1,13 +1,5 @@
-
+<%= link_list(
+ [t("valuation.menu.title"), valuation_root_path],
+ [t("valuation.menu.budgets"), valuation_budgets_path, controller_name == "budget_investments", class: "budgets-link"],
+ id: "valuation_menu"
+) %>
diff --git a/spec/helpers/link_list_helper_spec.rb b/spec/helpers/link_list_helper_spec.rb
new file mode 100644
index 000000000..8fd04f5ac
--- /dev/null
+++ b/spec/helpers/link_list_helper_spec.rb
@@ -0,0 +1,72 @@
+require "rails_helper"
+
+describe LinkListHelper do
+ describe "#link_list" do
+ it "returns an empty string with an empty list" do
+ expect(helper.link_list).to eq ""
+ end
+
+ it "returns nothing with a list of nil elements" do
+ expect(helper.link_list(nil, nil)).to eq ""
+ end
+
+ it "generates a list of links" do
+ list = helper.link_list(["Home", "/"], ["Info", "/info"], class: "menu")
+
+ expect(list).to eq ''
+ expect(list).to be_html_safe
+ end
+
+ it "accepts options for links" do
+ render helper.link_list(["Home", "/", class: "root"], ["Info", "/info", id: "info"])
+
+ expect(page).to have_css "a", count: 2
+ expect(page).to have_css "a.root", count: 1, exact_text: "Home"
+ expect(page).to have_css "a#info", count: 1, exact_text: "Info"
+ end
+
+ it "ignores nil entries" do
+ render helper.link_list(["Home", "/", class: "root"], nil, ["Info", "/info", id: "info"])
+
+ expect(page).to have_css "a", count: 2
+ expect(page).to have_css "a.root", count: 1, exact_text: "Home"
+ expect(page).to have_css "a#info", count: 1, exact_text: "Info"
+ end
+
+ it "accepts an optional condition to check the active element" do
+ render helper.link_list(
+ ["Home", "/", false],
+ ["Info", "/info", true],
+ ["Help", "/help"]
+ )
+
+ expect(page).to have_css "li", count: 3
+ expect(page).to have_css "li[aria-current='true']", count: 1, exact_text: "Info"
+ end
+
+ it "allows passing both the active condition and link options" do
+ render helper.link_list(
+ ["Home", "/", false, class: "root"],
+ ["Info", "/info", true, id: "info"],
+ ["Help", "/help", rel: "help"]
+ )
+
+ expect(page).to have_css "li", count: 3
+ expect(page).to have_css "li[aria-current='true']", count: 1, exact_text: "Info"
+
+ expect(page).to have_css "a.root", count: 1, exact_text: "Home"
+ expect(page).to have_css "a#info", count: 1, exact_text: "Info"
+ expect(page).to have_css "a[rel='help']", count: 1, exact_text: "Help"
+ end
+ end
+
+ attr_reader :content
+
+ def render(content)
+ @content = content
+ end
+
+ def page
+ Capybara::Node::Simple.new(content)
+ end
+end