From 6d1fc8e14be0e2d166c97f5a97748d892358145a Mon Sep 17 00:00:00 2001
From: Ryan Lerch <rlerch@redhat.com>
Date: Feb 04 2016 11:29:37 +0000
Subject: fix for #682: made the PR actions a buttonbar


Made the PR actions at the top of the PR into a buttonbar
to make it look similar to the other buttons in for eding
comments. (but left them as coloured buttons)

I had to move the form call outside of the button-group
class as it was causing issues with the styling.

---

diff --git a/pagure/templates/pull_request.html b/pagure/templates/pull_request.html
index 25e9a4b..6d5d515 100644
--- a/pagure/templates/pull_request.html
+++ b/pagure/templates/pull_request.html
@@ -27,25 +27,33 @@
   <div class="{% if pull_request %}col-md-8{% elif form and (repo_admin or remote_git) %}col-md-12{%endif%}">
 {% if pull_request %}
 <h2><span class="label label-default">PR#{{requestid}}</span> {{ pull_request.title | noJS(ignore="img") | safe
-  }}{% if authenticated and (g.fas_user.username == pull_request.user.username
-    or repo_admin) and pull_request.status == 'Open'
-  %}
+  }}
+  {% if authenticated and (g.fas_user.username == pull_request.user.username
+    or repo_admin) and pull_request.status == 'Open'%}
      <span class="pull-xs-right">
-    <a class="btn btn-sm btn-primary"
-       href="{{ url_for('request_pull_edit', username=username,
-            repo=repo.name, requestid=requestid) }}"
-      title="Update title">edit</a>
-
-    {% if pull_request.status == 'Open' and authenticated and
+     {% if pull_request.status == 'Open' and authenticated and
+        (repo_admin or g.fas_user.username == pull_request.user.username) %}
+        <form style="display:inline;" action="{{ url_for('cancel_request_pull', username=username,
+            repo=repo.name, requestid=requestid) }}" method="POST">
+     {% endif %}
+          <div class="btn-group" role="group" aria-label="Basic example">
+            <a class="btn btn-sm btn-primary"
+               href="{{ url_for('request_pull_edit', username=username,
+               repo=repo.name, requestid=requestid) }}"
+               title="Update title"><span class="oi" data-glyph="pencil"></span></a>
+     {% if pull_request.status == 'Open' and authenticated and
         (repo_admin or g.fas_user.username == pull_request.user.username) %}
-      <span><form style="display:inline;" action="{{ url_for('cancel_request_pull', username=username,
-          repo=repo.name, requestid=requestid) }}" method="POST">
         {{ mergeform.csrf_token }}
-        <button type="submit" value="Close" id="cancel_pr" class="btn btn-danger btn-sm">close</button>
-      </form></span>
+            <button type="submit" value="Close" id="cancel_pr"
+                    class="btn btn-danger btn-sm"><span class="oi" data-glyph="trash"></span></button>
     {% endif %}
-      </span>
-      {% endif %}
+          </div>
+    {% if pull_request.status == 'Open' and authenticated and
+        (repo_admin or g.fas_user.username == pull_request.user.username) %}
+      </form>
+    {% endif %}
+    </span>
+  {% endif %}
 </h2>
 
   <h5 class="text-muted">