Remove redundant links in admin tables

There were places where we had two links pointing to the same place; one
link would be the name/title of a record, and one link would be under
the "actions" column.

This is confusing, since users would probably expect these links to
point to different places (which is what happens in other tables in the
admin section) and might try to click one of them and then the other
one and be surprised when they found out both of them go to the same
page.
This commit is contained in:
Javi Martín
2020-07-06 20:45:48 +02:00
parent 7cb0a4135b
commit 48db31cd6b
12 changed files with 12 additions and 20 deletions

View File

@@ -1,8 +1,6 @@
<tr id="<%= dom_id(poll) %>" class="poll">
<td>
<strong>
<%= link_to poll.name, admin_poll_path(poll) %>
</strong>
<strong><%= poll.name %></strong>
</td>
<td class="text-center">
<%= l poll.starts_at.to_date %>

View File

@@ -18,9 +18,7 @@
<% @poll.questions.each do |question| %>
<tr id="<%= dom_id(question) %>">
<td>
<strong>
<%= link_to question.title, admin_question_path(question) %>
</strong>
<strong><%= question.title %></strong>
<% if question.proposal.present? %>
<small>
<%= link_to t("admin.polls.show.see_proposal"),

View File

@@ -10,9 +10,7 @@
<tbody>
<% @polls.each do |poll| %>
<tr id="<%= dom_id(poll) %>" class="poll">
<td>
<%= link_to poll.name, manage_admin_poll_booth_assignments_path(poll) %>
</td>
<td><%= poll.name %></td>
<td>
<%= l poll.starts_at.to_date %> - <%= l poll.ends_at.to_date %>
</td>