diff --git a/app/assets/stylesheets/budgets/ballot/investment.scss b/app/assets/stylesheets/budgets/ballot/investment.scss index 8333d1aa7..638a10475 100644 --- a/app/assets/stylesheets/budgets/ballot/investment.scss +++ b/app/assets/stylesheets/budgets/ballot/investment.scss @@ -29,6 +29,7 @@ .remove-budget-investment { @include has-fa-icon(times, solid); + cursor: pointer; font-size: $close-icon-size; position: absolute; right: $close-icon-margin; diff --git a/app/components/budgets/ballot/investment_component.html.erb b/app/components/budgets/ballot/investment_component.html.erb index 58e62f4ea..42aa8c9f2 100644 --- a/app/components/budgets/ballot/investment_component.html.erb +++ b/app/components/budgets/ballot/investment_component.html.erb @@ -3,11 +3,11 @@ <%= investment_price %> <% if budget.balloting? %> - <%= link_to delete_path, - title: t("budgets.ballots.show.remove"), - class: "remove-budget-investment", - method: :delete, - remote: true do %> + <%= button_to delete_path, + title: t("budgets.ballots.show.remove"), + class: "remove-budget-investment", + method: :delete, + remote: true do %> <%= t("budgets.ballots.show.remove") %> <% end %> <% end %> diff --git a/spec/system/budgets/ballots_spec.rb b/spec/system/budgets/ballots_spec.rb index 36bf600d0..2be0cdd0f 100644 --- a/spec/system/budgets/ballots_spec.rb +++ b/spec/system/budgets/ballots_spec.rb @@ -379,7 +379,7 @@ describe "Ballots" do expect(page).to have_content("You have voted one investment") within("#budget_investment_#{investment.id}") do - click_link "Remove vote" + click_button "Remove vote" end expect(page).to have_current_path(budget_ballot_path(budget)) @@ -406,7 +406,7 @@ describe "Ballots" do end within("#sidebar #budget_investment_#{investment1.id}_sidebar") do - click_link "Remove vote" + click_button "Remove vote" end expect(page).to have_css("#total_amount", text: "€20,000") @@ -436,7 +436,7 @@ describe "Ballots" do expect(page).to have_content("You have voted one investment") within(".ballot-list li", text: "Sully monument") do - click_link "Remove vote" + click_button "Remove vote" end expect(page).to have_content("You have voted 0 investments") @@ -623,7 +623,7 @@ describe "Ballots" do end within("#budget_investment_#{bi1.id}_sidebar") do - click_link "Remove vote" + click_button "Remove vote" end expect(page).not_to have_css "#budget_investment_#{bi1.id}_sidebar"