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

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

diff --git a/app/views/issues/_comments.html.erb b/app/views/issues/_comments.html.erb
index d49ae867099097a1ea1de761cb8eadb952458444..e55bf42578fb5200ccf94040f02113b32fd7599c 100644 (file)
--- a/app/views/issues/_comments.html.erb
+++ b/app/views/issues/_comments.html.erb
@@ -1,25 +1,27 @@
-<div class="issue-comments">
+<div>
   <% comments.each do |comment| %>
-    <div class="comment">
-      <div style="float:left">
-        <%= link_to user_thumbnail(comment.user), :controller => :user, :action =>:view, :display_name => comment.user.display_name %>
+    <div class="row">
+      <div class="col-auto">
+        <%= link_to user_thumbnail(comment.user), user_path(comment.user) %>
+      </div>
+      <div class="col">
+        <p class="text-muted mb-0">
+          <%= t ".comment_from_html", :user_link => link_to(comment.user.display_name, user_path(comment.user)),
+                                      :comment_created_at => l(comment.created_at.to_datetime, :format => :friendly) %>
+        </p>
+        <p><%= comment.body %></p>
       </div>
-      <b> <%= link_to comment.user.display_name, :controller => :user, :action =>:view, :display_name => comment.user.display_name %> </b> <br/>
-      <%= comment.body %>
     </div>
-    <span class="deemphasize">
-      On <%= l comment.created_at.to_datetime, :format => :friendly %>
-    </span>
     <hr>
   <% end %>
 </div>
-<br/>
-<div class="comment">
-  <%= form_for @new_comment, url: issue_comments_path(@issue) do |f| %>
-  <%= richtext_area :issue_comment, :body, :cols => 10, :rows => 8, :required => true %>
-  <%= label_tag :reassign, t('issues.show.comments.reassign_param') %> <%= check_box_tag :reassign, true %>
-  <br/>
-  <br/>
-  <%= submit_tag 'Submit' %>
+
+<div>
+  <%= bootstrap_form_for @new_comment, :url => issue_comments_path(@issue) do |f| %>
+    <%= f.richtext_field :body, :cols => 80, :rows => 20, :hide_label => true %>
+    <%= f.form_group do %>
+      <%= f.check_box :reassign, { :label => t(".reassign_param"), :id => "reassign", :name => "reassign", :checked => false }, true, false %>
+    <% end %>
+    <%= f.primary %>
   <% end %>
 </div>