Merge remote-tracking branch 'upstream/pull/5401'

[rails.git] / app / views / issues / show.html.erb

diff --git a/app/views/issues/show.html.erb b/app/views/issues/show.html.erb
index 732fd2ef2a114c9ac50e1378772e196e99aa313e..74e7c0a4ce1f7793880c542da15d021a743c1ebc 100644 (file)
--- a/app/views/issues/show.html.erb
+++ b/app/views/issues/show.html.erb
@@ -1,64 +1,81 @@
 <% content_for :heading do %>
 <% content_for :heading do %>

-<h2><%= t ".title", :status => @issue.status.humanize, :issue_id => @issue.id %></h2>
+<h1><%= @title %></h1>

 <p><%= @issue.reportable.model_name.human %> : <%= link_to reportable_title(@issue.reportable), reportable_url(@issue.reportable) %></p>
 <p><%= @issue.reportable.model_name.human %> : <%= link_to reportable_title(@issue.reportable), reportable_url(@issue.reportable) %></p>

-<p class="deemphasize">
+<p class="text-body-secondary">

   <small>
     <%= @issue.assigned_role %>
   <small>
     <%= @issue.assigned_role %>

-    | <%= t ".reports", :count => @issue.reports.count %>
-    | <%= t ".report_created_at", :datetime => l(@issue.created_at.to_datetime, :format => :friendly) %>
-    <%= " | " + t(".last_resolved_at", :datetime => l(@issue.resolved_at.to_datetime, :format =>:friendly)) if @issue.resolved_at? %>
-    <%= " | " + t(".last_updated_at", :datetime => l(@issue.updated_at.to_datetime, :format => :friendly), :displayname => @issue.user_updated.display_name ) if @issue.user_updated %>
+    <% if @issue.reports.count > 0 %>
+      | <%= t ".reports", :count => @issue.reports.count %>
+    <% else %>
+      | <%= t ".no_reports" %>
+    <% end %>
+      | <%= t ".report_created_at_html",
+              :datetime => tag.time(l(@issue.created_at.to_datetime, :format => :friendly),
+                                    :datetime => @issue.created_at.xmlschema) %>
+    <% if @issue.resolved_at? %>
+      | <%= t ".last_resolved_at_html",
+              :datetime => tag.time(l(@issue.resolved_at.to_datetime, :format => :friendly),
+                                    :datetime => @issue.resolved_at.xmlschema) %>
+    <% end %>
+    <% if @issue.user_updated %>
+      | <%= t ".last_updated_at_html",
+              :datetime => tag.time(l(@issue.updated_at.to_datetime, :format => :friendly),
+                                    :datetime => @issue.updated_at.xmlschema),
+              :displayname => link_to(@issue.user_updated.display_name, @issue.user_updated) %>
+    <% end %>

   </small>
 </p>
   </small>
 </p>

-<p>
-  <%= link_to t('issues.resolve'), resolve_issue_url(@issue), :method => :post if @issue.may_resolve? %>
-  <% if @issue.may_ignore? %>
-  | <%= link_to t('issues.ignore'), ignore_issue_url(@issue), :method => :post %>
-  <% end %>
-</p>
-<p><%= link_to t('issues.reopen'), reopen_issue_url(@issue), :method => :post if @issue.may_reopen? %></p>
+<nav class="secondary-actions">
+  <ul class="clearfix">
+    <% if @issue.may_resolve? %>
+      <li><%= link_to t(".resolve"), resolve_issue_url(@issue), :method => :post %></li>
+    <% end %>
+    <% if @issue.may_ignore? %>
+      <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="report-related-block">
-
-  <div class="report-block">
+<div class="row">
+  <div class="col-md-8">

     <h3><%= t ".reports_of_this_issue" %></h3>
 
     <% if @read_reports.present? %>
     <h3><%= t ".reports_of_this_issue" %></h3>
 
     <% if @read_reports.present? %>

-    <div class="read-reports">
+    <div class="bg-body-tertiary text-body-secondary">

       <h4><%= t ".read_reports" %></h4>
       <h4><%= t ".read_reports" %></h4>

-      <%= render 'reports', reports: @read_reports %>
+      <%= render "reports", :reports => @read_reports %>

     </div>
     <% end %>
 
     <% if @unread_reports.any? %>
     </div>
     <% end %>
 
     <% if @unread_reports.any? %>

-    <div class="unread-reports">
+    <div>

       <h4><%= t ".new_reports" %></h4>
       <h4><%= t ".new_reports" %></h4>

-      <%= render 'reports', reports: @unread_reports %>
+      <%= render "reports", :reports => @unread_reports %>

     </div>
     <% end %>
     </div>
     <% end %>

-    <br/>

   </div>
 
   <% if @issue.reported_user %>
   </div>
 
   <% if @issue.reported_user %>

-    <div class="related-block">
+    <div class="col-md-4">

       <h3><%= t ".other_issues_against_this_user" %></h3>
       <h3><%= t ".other_issues_against_this_user" %></h3>

-      <div class="unread-reports">
-        <% if @related_issues.count > 1 %>
+      <% if @related_issues.count > 1 %>
+        <ul>

           <% @related_issues.each do |issue| %>
             <% if issue.id != @issue.id %>
           <% @related_issues.each do |issue| %>
             <% if issue.id != @issue.id %>

-              <%= link_to reportable_title(issue.reportable), issue %> <br/>
+              <li><%= link_to reportable_title(issue.reportable), issue %></li>

             <% end %>
           <% end %>
             <% end %>
           <% end %>

-        <% else %>
-          <p><%= t ".no_other_issues" %></p>
-        <% end %>
-      </div>
+        </ul>
+      <% else %>
+        <p><%= t ".no_other_issues" %></p>
+      <% end %>

     </div>
   <% end %>
 </div>
 
 <h3><%= t ".comments_on_this_issue" %></h3>
     </div>
   <% end %>
 </div>
 
 <h3><%= t ".comments_on_this_issue" %></h3>

-<div class="unread-reports">
-  <%= render 'comments', comments: @comments %>
-</div>
+<%= render "comments", :comments => @comments %>