diff mbox

[13/51] base: Don't display the breadcrumb bar on the project list page

Message ID 1440440620-25937-14-git-send-email-damien.lespiau@intel.com
State Changes Requested
Headers show

Commit Message

Damien Lespiau Aug. 24, 2015, 6:23 p.m. UTC 
We already have the information on the main bar, no need to have a
breadcum bar here.

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
---
 templates/base.html | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

Comments

Stephen Finucane Sept. 9, 2015, 2:44 p.m. UTC | #1 
Consistency is key here: I'd keep this even if it's pretty much useless. If nothing else, the spacing change between the project list and project view page (and subsequent scrolling) is going to annoy some people (me, for example).
diff mbox

Patch

diff --git a/templates/base.html b/templates/base.html
index 844640a..3b2c83e 100644
--- a/templates/base.html
+++ b/templates/base.html
@@ -60,9 +60,9 @@ 
     </div>
    </div>
   </nav>
+{% if project %}
   <div id="nav">
    <div id="navleft">
-   {% if project %}
     <strong>Project</strong>: {{ project.linkname }}
      :
      <a href="{% url 'patchwork.views.patch.list' project_id=project.linkname %}"
@@ -71,14 +71,11 @@ 
      :
      <a href="{% url 'patchwork.views.projects' %}"
      >other projects</a>
-     {% endif %}
-    {% else %}
-     <a href="{% url 'patchwork.views.projects' %}"
-     >project list</a>
     {% endif %}
    </div>
    <div style="clear: both"></div>
   </div>
+{% endif %}
 {% if messages %}
   <div id="messages">
   {% for message in messages %}