[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