diff --git a/app/assets/stylesheets/layout.scss b/app/assets/stylesheets/layout.scss index 425c17d87..c42d10a40 100644 --- a/app/assets/stylesheets/layout.scss +++ b/app/assets/stylesheets/layout.scss @@ -667,9 +667,12 @@ body > header, } } +.menu-button { + color: inherit; +} + .menu-icon { @include hamburger($color: currentcolor, $color-hover: currentcolor); - color: inherit; } .title-bar { diff --git a/app/views/layouts/_admin_header.html.erb b/app/views/layouts/_admin_header.html.erb index 1995fa7ff..73f93c602 100644 --- a/app/views/layouts/_admin_header.html.erb +++ b/app/views/layouts/_admin_header.html.erb @@ -11,8 +11,10 @@
- -
<%= t("application.menu") %>
+
diff --git a/app/views/layouts/_header.html.erb b/app/views/layouts/_header.html.erb index 6aa11150b..d11c57fe4 100644 --- a/app/views/layouts/_header.html.erb +++ b/app/views/layouts/_header.html.erb @@ -16,8 +16,10 @@
- - <%= t("application.menu") %> +

diff --git a/spec/system/admin_spec.rb b/spec/system/admin_spec.rb index 3f115c7dc..88e8000da 100644 --- a/spec/system/admin_spec.rb +++ b/spec/system/admin_spec.rb @@ -104,4 +104,22 @@ describe "Admin" do expect(page).to have_link "Participatory budgets" end end + + describe "Menu button", :admin do + scenario "is not present on large screens" do + visit admin_root_path + + expect(page).not_to have_button "Menu" + end + + scenario "toggles the menu on small screens", :small_window do + visit admin_root_path + + expect(page).not_to have_link "My account" + + click_button "Menu" + + expect(page).to have_link "My account" + end + end end diff --git a/spec/system/home_spec.rb b/spec/system/home_spec.rb index 0cacb822b..f1482e2d9 100644 --- a/spec/system/home_spec.rb +++ b/spec/system/home_spec.rb @@ -138,6 +138,24 @@ describe "Home" do end end + describe "Menu button" do + scenario "is not present on large screens" do + visit root_path + + expect(page).not_to have_button "Menu" + end + + scenario "toggles the menu on small screens", :small_window do + visit root_path + + expect(page).not_to have_link "Sign in" + + click_button "Menu" + + expect(page).to have_link "Sign in" + end + end + scenario "if there are cards, the 'featured' title will render" do create(:widget_card, title: "Card text",