From 562b4c66f292bb4a37804a9a9b53e3f85305cafe Mon Sep 17 00:00:00 2001
From: Daniel Axtens <dja@axtens.net>
Date: Mon, 27 Mar 2017 16:42:50 +1100
Subject: [PATCH] Put download buttons at the top as well

Suggested by a few people.

Signed-off-by: Daniel Axtens <dja@axtens.net>
Reviewed-by: Stephen Finucane <stephen@that.guru>
---
 .../templates/patchwork/download_buttons.html | 24 +++++++++++++++
 patchwork/templates/patchwork/submission.html | 29 +++----------------
 2 files changed, 28 insertions(+), 25 deletions(-)
 create mode 100644 patchwork/templates/patchwork/download_buttons.html

diff --git a/patchwork/templates/patchwork/download_buttons.html b/patchwork/templates/patchwork/download_buttons.html
new file mode 100644
index 00000000..df392b36
--- /dev/null
+++ b/patchwork/templates/patchwork/download_buttons.html
@@ -0,0 +1,24 @@
+<div class="btn-group pull-right">
+  <a href="{% url 'patch-raw' patch_id=submission.id %}"
+   class="btn btn-default" role="button" title="Download patch diff"
+   >diff</a>
+  <a href="{% url 'patch-mbox' patch_id=submission.id %}"
+   class="btn btn-default" role="button" title="Download patch mbox"
+   >mbox</a>
+  {% if submission.series.all|length == 1 %}
+  <a href="{% url 'series-mbox' series_id=submission.latest_series.id %}"
+   class="btn btn-default" role="button"
+   title="Download patch mbox with dependencies">series</a>
+  {% elif submission.series.all|length > 1 %}
+  <button type="button" class="btn btn-default dropdown-toggle"
+   data-toggle="dropdown">
+   series <span class="caret"></span>
+  </button>
+  <ul class="dropdown-menu" role="menu">
+  {% for series in submission.series.all %}
+   <li><a href="{% url 'series-mbox' series_id=series.id %}"
+    >{{ series }}</a></li>
+  {% endfor %}
+  </ul>
+  {% endif %}
+</div>
diff --git a/patchwork/templates/patchwork/submission.html b/patchwork/templates/patchwork/submission.html
index dd1b9e39..6ed20c34 100644
--- a/patchwork/templates/patchwork/submission.html
+++ b/patchwork/templates/patchwork/submission.html
@@ -27,7 +27,9 @@ function toggle_div(link_id, headers_id)
 }
 </script>
 
-<h1>{{ submission.name }}</h1>
+<h1>{{ submission.name }}
+{% include "patchwork/download_buttons.html" %}
+</h1>
 
 <table class="patchmeta">
  <tr>
@@ -281,30 +283,7 @@ function toggle_div(link_id, headers_id)
 {% if submission.diff %}
 <h2>
  Patch
- <div class="btn-group pull-right">
-  <a href="{% url 'patch-raw' patch_id=submission.id %}"
-   class="btn btn-default" role="button" title="Download patch diff"
-   >diff</a>
-  <a href="{% url 'patch-mbox' patch_id=submission.id %}"
-   class="btn btn-default" role="button" title="Download patch mbox"
-   >mbox</a>
-  {% if submission.series.all|length == 1 %}
-  <a href="{% url 'series-mbox' series_id=submission.latest_series.id %}"
-   class="btn btn-default" role="button"
-   title="Download patch mbox with dependencies">series</a>
-  {% elif submission.series.all|length > 1 %}
-  <button type="button" class="btn btn-default dropdown-toggle"
-   data-toggle="dropdown">
-   series <span class="caret"></span>
-  </button>
-  <ul class="dropdown-menu" role="menu">
-  {% for series in submission.series.all %}
-   <li><a href="{% url 'series-mbox' series_id=series.id %}"
-    >{{ series }}</a></li>
-  {% endfor %}
-  </ul>
-  {% endif %}
- </div>
+ {% include "patchwork/download_buttons.html" %}
 </h2>
 
 <div id="patch" class="patch">
-- 
2.47.3