Remove no longer necessary flag/unflag HTML IDs

They were added in commit 015fe704 because we used them in the specs,
but we don't use them anymore and they make the code hard to read.

app/views/budgets/investments/_flag_actions.html.erb

@@ -1,25 +1,23 @@
 <span class="flag-content">
   <% if show_flag_action? investment %>
-    <a id="flag-expand-investment-<%= investment.id %>" data-toggle="flag-drop-investment-<%= investment.id %>" title="<%= t("shared.flag") %>">
+    <a data-toggle="flag-drop-investment-<%= investment.id %>" title="<%= t("shared.flag") %>">
       <span class="icon-flag flag-disable"></span>
     </a>
     <span class="dropdown-pane" id="flag-drop-investment-<%= investment.id %>" data-dropdown data-auto-focus="true">
       <%= link_to t("shared.flag"), flag_budget_investment_path(investment.budget, investment.id),
                                     method: :put,
-                                    remote: true,
-                                    id: "flag-investment-#{investment.id}" %>
+                                    remote: true %>
     </span>
   <% end %>
 
   <% if show_unflag_action? investment %>
-    <a id="unflag-expand-investment-<%= investment.id %>" data-toggle="unflag-drop-investment-<%= investment.id %>" title="<%= t("shared.unflag") %>">
+    <a data-toggle="unflag-drop-investment-<%= investment.id %>" title="<%= t("shared.unflag") %>">
       <span class="icon-flag flag-active"></span>
     </a>
     <span class="dropdown-pane" id="unflag-drop-investment-<%= investment.id %>" data-dropdown data-auto-focus="true">
       <%= link_to t("shared.unflag"), unflag_budget_investment_path(investment.budget, investment.id),
                                       method: :put,
-                                      remote: true,
-                                      id: "unflag-investment-#{investment.id}" %>
+                                      remote: true %>
     </span>
   <% end %>
 </span>

app/views/comments/_flag_actions.html.erb

@@ -1,9 +1,7 @@
 <span class="flag-content">
   <% if show_flag_action? comment %>
     <span class="divider">&nbsp;|&nbsp;</span>
-    <a id="flag-expand-comment-<%= comment.id %>"
-    data-toggle="flag-drop-comment-<%= comment.id %>"
-    title="<%= t("shared.flag") %>">
+    <a data-toggle="flag-drop-comment-<%= comment.id %>" title="<%= t("shared.flag") %>">
       <span class="icon-flag flag-disable"></span>
     </a>
     <span id="flag-drop-comment-<%= comment.id %>"
@@ -11,20 +9,18 @@
          data-dropdown
          data-auto-focus="true">
       <%= link_to t("shared.flag"), flag_comment_path(comment), method: :put,
-                  remote: true, id: "flag-comment-#{comment.id}" %>
+                  remote: true %>
     </span>
   <% end %>
 
   <% if show_unflag_action? comment %>
     <span class="divider">&nbsp;|&nbsp;</span>
-    <a id="unflag-expand-comment-<%= comment.id %>"
-       data-toggle="unflag-drop-comment-<%= comment.id %>"
-       title="<%= t("shared.unflag") %>">
+    <a data-toggle="unflag-drop-comment-<%= comment.id %>" title="<%= t("shared.unflag") %>">
       <span class="icon-flag flag-active"></span>
     </a>
     <span class="dropdown-pane" id="unflag-drop-comment-<%= comment.id %>" data-dropdown data-auto-focus="true">
       <%= link_to t("shared.unflag"), unflag_comment_path(comment), method: :put,
-                  remote: true, id: "unflag-comment-#{comment.id}" %>
+                  remote: true %>
     </span>
   <% end %>
 </span>

app/views/debates/_flag_actions.html.erb

@@ -1,19 +1,19 @@
 <span class="flag-content">
   <% if show_flag_action? debate %>
-    <a id="flag-expand-debate-<%= debate.id %>" data-toggle="flag-drop-debate-<%= debate.id %>" title="<%= t("shared.flag") %>">
+    <a data-toggle="flag-drop-debate-<%= debate.id %>" title="<%= t("shared.flag") %>">
       <span class="icon-flag flag-disable"></span>
     </a>
     <span class="dropdown-pane" id="flag-drop-debate-<%= debate.id %>" data-dropdown data-auto-focus="true">
-      <%= link_to t("shared.flag"), flag_debate_path(debate), method: :put, remote: true, id: "flag-debate-#{debate.id}" %>
+      <%= link_to t("shared.flag"), flag_debate_path(debate), method: :put, remote: true %>
     </span>
   <% end %>
 
   <% if show_unflag_action? debate %>
-    <a id="unflag-expand-debate-<%= debate.id %>" data-toggle="unflag-drop-debate-<%= debate.id %>" title="<%= t("shared.unflag") %>">
+    <a data-toggle="unflag-drop-debate-<%= debate.id %>" title="<%= t("shared.unflag") %>">
       <span class="icon-flag flag-active"></span>
     </a>
     <span class="dropdown-pane" id="unflag-drop-debate-<%= debate.id %>" data-dropdown data-auto-focus="true">
-      <%= link_to t("shared.unflag"), unflag_debate_path(debate), method: :put, remote: true, id: "unflag-debate-#{debate.id}" %>
+      <%= link_to t("shared.unflag"), unflag_debate_path(debate), method: :put, remote: true %>
     </span>
   <% end %>
 </span>

app/views/legislation/proposals/_flag_actions.html.erb

@@ -1,19 +1,19 @@
 <span class="flag-content">
   <% if show_flag_action? proposal %>
-    <a id="flag-expand-proposal-<%= proposal.id %>" data-toggle="flag-drop-proposal-<%= proposal.id %>" title="<%= t("shared.flag") %>">
+    <a data-toggle="flag-drop-proposal-<%= proposal.id %>" title="<%= t("shared.flag") %>">
       <span class="icon-flag flag-disable"></span>
     </a>
     <span class="dropdown-pane" id="flag-drop-proposal-<%= proposal.id %>" data-dropdown data-auto-focus="true">
-      <%= link_to t("shared.flag"), flag_legislation_process_proposal_path(proposal.process, proposal), method: :put, remote: true, id: "flag-proposal-#{proposal.id}" %>
+      <%= link_to t("shared.flag"), flag_legislation_process_proposal_path(proposal.process, proposal), method: :put, remote: true %>
     </span>
   <% end %>
 
   <% if show_unflag_action? proposal %>
-    <a id="unflag-expand-proposal-<%= proposal.id %>" data-toggle="unflag-drop-proposal-<%= proposal.id %>" title="<%= t("shared.unflag") %>">
+    <a data-toggle="unflag-drop-proposal-<%= proposal.id %>" title="<%= t("shared.unflag") %>">
       <span class="icon-flag flag-active"></span>
     </a>
     <span class="dropdown-pane" id="unflag-drop-proposal-<%= proposal.id %>" data-dropdown data-auto-focus="true">
-      <%= link_to t("shared.unflag"), unflag_legislation_process_proposal_path(proposal.process, proposal), method: :put, remote: true, id: "unflag-proposal-#{proposal.id}" %>
+      <%= link_to t("shared.unflag"), unflag_legislation_process_proposal_path(proposal.process, proposal), method: :put, remote: true %>
     </span>
   <% end %>
 </span>

app/views/proposals/_flag_actions.html.erb

@@ -1,19 +1,19 @@
 <span class="flag-content">
   <% if show_flag_action? proposal %>
-    <a id="flag-expand-proposal-<%= proposal.id %>" data-toggle="flag-drop-proposal-<%= proposal.id %>" title="<%= t("shared.flag") %>">
+    <a data-toggle="flag-drop-proposal-<%= proposal.id %>" title="<%= t("shared.flag") %>">
       <span class="icon-flag flag-disable"></span>
     </a>
     <span class="dropdown-pane" id="flag-drop-proposal-<%= proposal.id %>" data-dropdown data-auto-focus="true">
-      <%= link_to t("shared.flag"), flag_proposal_path(proposal), method: :put, remote: true, id: "flag-proposal-#{proposal.id}" %>
+      <%= link_to t("shared.flag"), flag_proposal_path(proposal), method: :put, remote: true %>
     </span>
   <% end %>
 
   <% if show_unflag_action? proposal %>
-    <a id="unflag-expand-proposal-<%= proposal.id %>" data-toggle="unflag-drop-proposal-<%= proposal.id %>" title="<%= t("shared.unflag") %>">
+    <a data-toggle="unflag-drop-proposal-<%= proposal.id %>" title="<%= t("shared.unflag") %>">
       <span class="icon-flag flag-active"></span>
     </a>
     <span class="dropdown-pane" id="unflag-drop-proposal-<%= proposal.id %>" data-dropdown data-auto-focus="true">
-      <%= link_to t("shared.unflag"), unflag_proposal_path(proposal), method: :put, remote: true, id: "unflag-proposal-#{proposal.id}" %>
+      <%= link_to t("shared.unflag"), unflag_proposal_path(proposal), method: :put, remote: true %>
     </span>
   <% end %>
 </span>