cst1/openstreetmap-website
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Use a secondary-actions nav for issue actions

Browse source 
This fits with actions on other pages, avoids using pipe characters,
and resolves an issue with empty paragraphs.
This commit is contained in:
Andy Allan 2022-05-11 16:08:48 +01:00
parent 9fe69b0b7f
commit 68d6b7279e
1 changed files with 13 additions and 7 deletions
Download patch file Download diff file Expand all files Collapse all files

20
app/views/issues/show.html.erb
View file

@ -10,13 +10,19 @@
<%= " | #{t('.last_updated_at', :datetime => l(@issue.updated_at.to_datetime, :format => :friendly), :displayname => @issue.user_updated.display_name)}" if @issue.user_updated %> <%= " | #{t('.last_updated_at', :datetime => l(@issue.updated_at.to_datetime, :format => :friendly), :displayname => @issue.user_updated.display_name)}" if @issue.user_updated %>
</small> </small>
</p> </p>
<p> <nav class="secondary-actions">
<%= link_to t(".resolve"), resolve_issue_url(@issue), :method => :post if @issue.may_resolve? %> <ul class="clearfix">
<% if @issue.may_ignore? %> <% if @issue.may_resolve? %>
| <%= link_to t(".ignore"), ignore_issue_url(@issue), :method => :post %> <li><%= link_to t(".resolve"), resolve_issue_url(@issue), :method => :post %></li>
<% end %> <% end %>
</p> <% if @issue.may_ignore? %>
<p><%= link_to t(".reopen"), reopen_issue_url(@issue), :method => :post if @issue.may_reopen? %></p> <li><%= link_to t(".ignore"), ignore_issue_url(@issue), :method => :post %></li>
<% end %>
<% if @issue.may_reopen? %>
<li><%= link_to t(".reopen"), reopen_issue_url(@issue), :method => :post %></li>
<% end %>
</ul>
</nav>
<% end %> <% end %>
<div class="row"> <div class="row">