diff mbox

[38/51] patch: Pull the patch between the commit message and comments

Message ID 1440440620-25937-39-git-send-email-damien.lespiau@intel.com
State Changes Requested
Headers show

Commit Message

Damien Lespiau Aug. 24, 2015, 6:23 p.m. UTC
This follows a logical flow, commit message, patch then comments.

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
---
 patchwork/templates/patchwork/patch.html | 19 +++++++++----------
 1 file changed, 9 insertions(+), 10 deletions(-)

Comments

Stephen Finucane Sept. 10, 2015, 4:19 p.m. UTC | #1
> This follows a logical flow, commit message, patch then comments.

> 

> Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>


Consider tabs: one for patch and one for commit message/comments. This should minimize the amount of scrolling/parsing messages, albeit at the expense of a small amount of context.
diff mbox

Patch

diff --git a/patchwork/templates/patchwork/patch.html b/patchwork/templates/patchwork/patch.html
index 5a45016..eb09990 100644
--- a/patchwork/templates/patchwork/patch.html
+++ b/patchwork/templates/patchwork/patch.html
@@ -187,6 +187,15 @@  function toggle_headers(link_id, headers_id)
 </div>
 {% endfor %}
 
+{% if patch.content %}
+<h2>Patch</h2>
+<div class="patch">
+<pre class="content">
+{{ patch|patchsyntax }}
+</pre>
+</div>
+{% endif %}
+
 {% for item in patch.answers %}
 {% if forloop.first %}
 <h2>Comments</h2>
@@ -200,14 +209,4 @@  function toggle_headers(link_id, headers_id)
 </div>
 {% endfor %}
 
-{% if patch.content %}
-<h2>Patch</h2>
-<div class="patch">
-<pre class="content">
-{{ patch|patchsyntax }}
-</pre>
-</div>
-{% endif %}
-
-
 {% endblock %}