[PATCH 11/19] ui: Merge the TODO list into the user dropdown

Stephen Finucane stephen.finucane at intel.com
Thu Feb 4 08:31:46 AEDT 2016


From: Damien Lespiau <damien.lespiau at intel.com>

We still leave the badge on the top bar so we can see, at a glance, the
number of items on the TODO list. The actual menu item is folded into
the dropdown.

Signed-off-by: Damien Lespiau <damien.lespiau at intel.com>
Acked-by: Stephen Finucane <stephen.finucane at intel.com>
---
 templates/base.html | 18 ++++++++++--------
 1 file changed, 10 insertions(+), 8 deletions(-)

diff --git a/templates/base.html b/templates/base.html
index 6a44b69..f5a065d 100644
--- a/templates/base.html
+++ b/templates/base.html
@@ -68,16 +68,18 @@
 {% endif %}
      <ul class="nav navbar-nav navbar-right">
 {% if user.is_authenticated %}
-     <li>
-      <a href="{% url 'user-todos' %}">
-       Todo
-       <span class="badge">{{ user.profile.n_todo_patches }}</span></a>
-     </li>
      <li class="dropdown">
-       <a href="#" class="dropdown-toggle" data-toggle="dropdown"
-        ><strong>{{ user.username }}</strong> <span
-        class="caret" /></a>
+       <a href="#" class="dropdown-toggle" data-toggle="dropdown">
+         <span class="badge">{{ user.profile.n_todo_patches }}</span>
+         <strong>{{ user.username }}</strong> <span class="caret" />
+       </a>
        <ul class="dropdown-menu" role="menu">
+         <li>
+           <a href="{% url 'user-todos' %}">
+             Reviews pending
+             <span class="badge">{{ user.profile.n_todo_patches }}</span>
+           </a>
+         </li>
 {% if user.is_staff %}
          <li><a href="{% url 'admin:index' %}">Administration</a></li>
 {% endif %}
-- 
2.0.0



More information about the Patchwork mailing list