]> git.ipfire.org Git - thirdparty/patchwork.git/commitdiff
templates: Move comments to above patch
authorJeremy Kerr <jk@ozlabs.org>
Tue, 29 Mar 2016 01:29:12 +0000 (09:29 +0800)
committerStephen Finucane <stephen.finucane@intel.com>
Tue, 29 Mar 2016 08:55:18 +0000 (09:55 +0100)
It's much more likely that existing comments will affect the user's
actions, rather than the patch content itself, so put the comments
first.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
Acked-by: Stephen Finucane <stephen.finucane@intel.com>
patchwork/templates/patchwork/patch.html

index a05f00dcafc6a14e261d7edc17b5e2ab682f3866..8b6d1fe39beeef7188075186b8d9f34dde02d8de 100644 (file)
@@ -202,6 +202,22 @@ function toggle_headers(link_id, headers_id)
 </pre>
 </div>
 
+{% for item in patch.comments.all %}
+{% if forloop.first %}
+<h2>Comments</h2>
+{% endif %}
+
+<div class="comment">
+<div class="meta">
+ <span>{{ item.submitter|personify:project }}</span>
+ <span class="pull-right">{{ item.date }}</span>
+</div>
+<pre class="content">
+{{ item|commentsyntax }}
+</pre>
+</div>
+{% endfor %}
+
 {% if patch.diff %}
 <h2>
  Patch
@@ -220,20 +236,4 @@ function toggle_headers(link_id, headers_id)
 </div>
 {% endif %}
 
-{% for item in patch.comments.all %}
-{% if forloop.first %}
-<h2>Comments</h2>
-{% endif %}
-
-<div class="comment">
-<div class="meta">
- <span>{{ item.submitter|personify:project }}</span>
- <span class="pull-right">{{ item.date }}</span>
-</div>
-<pre class="content">
-{{ item|commentsyntax }}
-</pre>
-</div>
-{% endfor %}
-
 {% endblock %}