From dc3c89cc70246a31fedafecf9c8863e209b244b6 Mon Sep 17 00:00:00 2001
From: Magnus Hagander <magnus@hagander.net>
Date: Tue, 15 Jul 2014 11:09:08 +0200
Subject: [PATCH] Unify open and close status under one button

Instead we use subheaders to show the difference between statuses
that keep the patch open, and statuses that close it.
---
 .../commitfest/templates/patch_commands.inc   | 23 ++++++++-----------
 1 file changed, 10 insertions(+), 13 deletions(-)

diff --git a/pgcommitfest/commitfest/templates/patch_commands.inc b/pgcommitfest/commitfest/templates/patch_commands.inc
index aeb96e2..c78a1c0 100644
--- a/pgcommitfest/commitfest/templates/patch_commands.inc
+++ b/pgcommitfest/commitfest/templates/patch_commands.inc
@@ -11,19 +11,16 @@
 
 <div class="btn-group">
  <a class="btn btn-default dropdown-toggle" data-toggle="dropdown" href="#">Status <span class="caret"></span></a>
- <ul class="dropdown-menu">
-  <li><a href="status/review/">Needs review</a></li>
-  <li><a href="status/author/">Waiting on Author</a></li>
-  <li><a href="status/committer/">Ready for Committer</a></li>
- </ul>
-</div>
-
-<div class="btn-group">
- <a class="btn btn-default dropdown-toggle" data-toggle="dropdown" href="#">Close <span class="caret"></span></a>
- <ul class="dropdown-menu">
-  <li><a href="close/reject/" onclick="return verify_reject()">Rejected</a></li>
-  <li><a href="close/feedback/" onclick="return verify_returned()">Returned with feedback</a></li>
-  <li><a href="close/committed/" onclick="return verify_committed({{is_committer|yesno:"true,false"}}))">Committed</a></li>
+ <ul class="dropdown-menu" role="menu">
+  <li role="presentation" class="dropdown-header">Open statuses</li>
+  <li role="presentation"><a href="status/review/">Needs review</a></li>
+  <li role="presentation"><a href="status/author/">Waiting on Author</a></li>
+  <li role="presentation"><a href="status/committer/">Ready for Committer</a></li>
+  <li role="presentation" class="divider"></li>
+  <li role="presentation" class="dropdown-header">Closed statuses</li>
+  <li role="presentation"><a href="close/reject/" onclick="return verify_reject()">Rejected</a></li>
+  <li role="presentation"><a href="close/feedback/" onclick="return verify_returned()">Returned with feedback</a></li>
+  <li role="presentation"><a href="close/committed/" onclick="return verify_committed({{is_committer|yesno:"true,false"}}))">Committed</a></li>
  </ul>
 </div>
 
-- 
2.39.5