removed breadcrumbs logic to its own template

removed breadcrumbs block from templates
added breadcrumbs that were missing from settings page

comic/templates/base.html

@@ -41,16 +41,18 @@
         <div id="navbar" class="collapse navbar-collapse">
           <ul class="nav navbar-nav">
             {% for page, link in menu.menu_items.items %}
-              <li {% if menu.current_page = page %} class="active"{% endif %}><a href="{{ link }}">{{ page }}</a></li>
+              <li {% if menu.current_page == page %} class="active"{% endif %}><a href="{{ link }}">{{ page }}</a></li>
             {% endfor %}
           </ul>
         </div><!--/.nav-collapse -->
       </div>
     </nav>
     <ol class="breadcrumb">
-        {% block breadcrumb %}
+        {% if breadcrumbs %}
+          {% include "comic/breadcrumbs.html" %}
+        {% else %}
             <li><a href="/">Home</a></li>
-        {% endblock %}
+        {% endif %}
     </ol>
 
         {% block content %}{% endblock %}

comic/templates/comic/breadcrumbs.html

@@ -0,0 +1,7 @@
+{% for crumb in breadcrumbs %}
+    {% if not forloop.last %}
+        <li><a href="{{ crumb.url }}">{{ crumb.name }}</a></li>
+    {% else %}
+        <li class="active">{{ crumb.name }}</li>
+    {% endif %}
+{% endfor %}

comic/templates/comic/comic_list.html

@@ -1,15 +1,6 @@
 {% extends "base.html" %}
 {% block title %}{{ title }}{% endblock %}
 
-{% block breadcrumb %}
-    {% for crumb in breadcrumbs %}
-        {% if not forloop.last %}
-            <li><a href="{{ crumb.url }}">{{ crumb.name }}</a></li>
-        {% else %}
-            <li class="active">{{ crumb.name }}</li>
-        {% endif %}
-    {% endfor %}
-{% endblock %}
 {% block content %}
 <div class="container">
     <h2 class="center">Comics</h2>

comic/templates/comic/read_comic.html

@@ -1,14 +1,6 @@
 {% extends "base.html" %}
 {% block title %}{{ title }}{% endblock %}
-{% block breadcrumb %}
+
-    {% for crumb in breadcrumbs %}
-        {% if not forloop.last %}
-            <li><a href="{{ crumb.url }}">{{ crumb.name }}</a></li>
-        {% else %}
-            <li class="active">{{ crumb.name }}</li>
-        {% endif %}
-    {% endfor %}
-{% endblock %}
 {% block content %}
 <div class="container comic_box">
 <center>

comic/templates/comic/settings_page.html

@@ -2,15 +2,6 @@
 
 {% block title %}{{ title }}{% endblock %}
 
-{% block breadcrumb %}
-    {% for crumb in breadcrumbs %}
-        {% if not forloop.last %}
-            <li><a href="{{ crumb.url }}">{{ crumb.name }}</a></li>
-        {% else %}
-            <li class="active">{{ crumb.name }}</li>
-        {% endif %}
-    {% endfor %}
-{% endblock %}
 
 {% block content %}
 <div class="container">

comic/templates/comic/setup.html

@@ -2,16 +2,6 @@
 
 {% block title %}{{ title }}{% endblock %}
 
-{% block breadcrumb %}
-    {% for crumb in breadcrumbs %}
-        {% if not forloop.last %}
-            <li><a href="{{ crumb.url }}">{{ crumb.name }}</a></li>
-        {% else %}
-            <li class="active">{{ crumb.name }}</li>
-        {% endif %}
-    {% endfor %}
-{% endblock %}
-
 {% block content %}
 {% if error_message %}
 <div class="alert alert-danger" role="alert">{{ error_message|safe }}</div>

comic/templates/comic/users_page.html

@@ -2,16 +2,6 @@
 
 {% block title %}CBWebReader - Users{% endblock %}
 
-{% block breadcrumb %}
-    {% for crumb in breadcrumbs %}
-        {% if not forloop.last %}
-            <li><a href="{{ crumb.url }}">{{ crumb.name }}</a></li>
-        {% else %}
-            <li class="active">{{ crumb.name }}</li>
-        {% endif %}
-    {% endfor %}
-{% endblock %}
-
 {% block content %}
 <div class="container">
     <table class="table table-striped table-bordered table-hover">

comic/views.py

@@ -149,6 +149,9 @@ def user_add_page(request):
 @user_passes_test(lambda u: u.is_superuser)
 def settings_page(request):
     success_message = []
+    crumbs = [
+        ('Settings', '/comic/settings/'),
+    ]
     if request.POST:
         form = SettingsForm(request.POST)
         if form.is_valid():
@@ -169,13 +172,14 @@ def settings_page(request):
             recaptcha_public_key.save()
             success_message.append('Settings updated.')
     form = SettingsForm(initial=SettingsForm.get_initial_values())
-    context = RequestContext(request, {
+    context = {
         'error_message': form.errors,
         'success_message': '</br>'.join(success_message),
         'form': form,
         'menu': Menu(request.user, 'Settings'),
         'title': 'CBWebReader - Settings',
-    })
+        'breadcrumbs': generate_breadcrumbs_from_menu(crumbs),
+    }
     return render(request, 'comic/settings_page.html', context)