Skip to content

First attempt at breadcrumbs. #2738

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
LukasMansour wants to merge 10 commits into
base: main
Choose a base branch
from
Open

First attempt at breadcrumbs. #2738

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
a97d89d
First attempt at breadcrumbs.
LukasMansour Jun 1, 2026
1284806
Second attempt at breadcrumbs.
LukasMansour Jun 22, 2026
9996ccf
URL resolving by block tag.
LukasMansour Jul 6, 2026
28e8898
Two forgotten files
LukasMansour Jul 6, 2026
33dd7b9
Fix small mistakes that fail tests
LukasMansour Jul 6, 2026
3c3640a
Fix disabled breadcrumbs, remove redundant variable.
LukasMansour Jul 13, 2026
5829859
Use breadcrumb block tag everywhere
LukasMansour Jul 13, 2026
0961bbe
Merge remote-tracking branch 'upstream/main' into link-if-not-self-br…
LukasMansour Jul 20, 2026
e389c81
Fix djangofmt with custom breadcrumb tag.
LukasMansour Jul 20, 2026
5933f3c
Small improvements to some html templates.
LukasMansour Jul 20, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 8 additions & 2 deletions evap/staff/templates/staff_base.html
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
{% extends 'base.html' %}
{% load staff_templatetags %}

{% block title %}{% translate 'Manage' %} - {{ block.super }}{% endblock %}

Expand All @@ -7,9 +8,14 @@
<ul class="breadcrumb">
{% block breadcrumb %}
{% if disable_breadcrumb_manager or not user.is_manager %}
<li class="breadcrumb-item">{% translate 'Manage' %}</li>
{% create_breadcrumb %}
{% translate 'Manage' %}
{% endcreate_breadcrumb %}
{% else %}
<li class="breadcrumb-item"><a href="{% url 'staff:index' %}">{% translate 'Manage' %}</a></li>
{% url 'staff:index' as staff_index_url %}
{% create_breadcrumb url=staff_index_url %}
{% translate 'Manage' %}
{% endcreate_breadcrumb %}
{% endif %}
{% endblock %}
</ul>
Expand Down
16 changes: 10 additions & 6 deletions evap/staff/templates/staff_course_base.html
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
{% extends 'staff_semester_base.html' %}
{% load staff_templatetags %}

{% block title %}
{% if course %}
Expand All @@ -11,13 +12,16 @@

{% block breadcrumb %}
{{ block.super }}

{% if course %}
{% if disable_breadcrumb_course %}
Comment thread
LukasMansour marked this conversation as resolved.
<li class="breadcrumb-item">{{ course.name }}</li>
{% else %}
<li class="breadcrumb-item"><a href="{% url 'staff:course_edit' course.id %}">{{ course.name }}</a></li>
{% endif %}
{% url 'staff:course_edit' course.id as course_edit_url %}
{% create_breadcrumb course_edit_url %}
{{ course.name }}
{% endcreate_breadcrumb %}
{% elif evaluation.course %}
<li class="breadcrumb-item"><a href="{% url 'staff:course_edit' evaluation.course.id %}">{{ evaluation.course.name }}</a></li>
{% url 'staff:course_edit' evaluation.course.id as course_edit_url %}
{% create_breadcrumb course_edit_url %}
{{ evaluation.course.name }}
{% endcreate_breadcrumb %}
{% endif %}
{% endblock %}
6 changes: 5 additions & 1 deletion evap/staff/templates/staff_course_type_index.html
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,10 +1,14 @@
{% extends 'staff_base.html' %}

{% load staff_templatetags %}
{% load static %}

{% block breadcrumb %}
{{ block.super }}
<li class="breadcrumb-item">{% translate 'Course types' %}</li>
{% url request.resolver_match.view_name as self_url %}
{% create_breadcrumb self_url %}
{% translate 'Course types' %}
{% endcreate_breadcrumb %}
{% endblock %}

{% block content %}
Expand Down
12 changes: 10 additions & 2 deletions evap/staff/templates/staff_course_type_merge.html
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,11 +1,19 @@
{% extends 'staff_base.html' %}

{% load evaluation_filters %}
{% load staff_templatetags %}

{% block breadcrumb %}
{{ block.super }}
<li class="breadcrumb-item"><a href="{% url 'staff:course_type_index' %}">{% translate 'Course types' %}</a></li>
<li class="breadcrumb-item">{% translate 'Merge course types' %}</li>
{% url 'staff:course_type_index' as course_type_index_url %}
{% create_breadcrumb course_type_index_url %}
{% translate 'Course types' %}
{% endcreate_breadcrumb %}

{% url 'staff:course_type_merge_selection' as course_type_merge_url %}
{% create_breadcrumb course_type_merge_url %}
{% translate 'Merge course types' %}
{% endcreate_breadcrumb %}
{% endblock %}

{% block content %}
Expand Down
14 changes: 12 additions & 2 deletions evap/staff/templates/staff_course_type_merge_selection.html
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,9 +1,19 @@
{% extends 'staff_base.html' %}

{% load staff_templatetags %}

{% block breadcrumb %}
{{ block.super }}
<li class="breadcrumb-item"><a href="{% url 'staff:course_type_index' %}">{% translate 'Course types' %}</a></li>
<li class="breadcrumb-item">{% translate 'Merge course types' %}</li>
{% url 'staff:course_type_index' as course_type_index_url %}
{% create_breadcrumb course_type_index_url %}
{% translate 'Course types' %}
{% endcreate_breadcrumb %}

{% url request.resolver_match.view_name as self_url %}
{% create_breadcrumb self_url %}
{% translate 'Merge course types' %}
{% endcreate_breadcrumb %}

{% endblock %}

{% block content %}
Expand Down
11 changes: 9 additions & 2 deletions evap/staff/templates/staff_evaluation_base.html
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
{% extends 'staff_course_base.html' %}

{% load staff_templatetags %}
{% block title %}
{% if evaluation.name %}
{{ evaluation.name }} -
Expand All @@ -11,9 +12,15 @@
{{ block.super }}
{% if evaluation %}
{% if evaluation.name %}
<li class="breadcrumb-item">{{ evaluation.name }}</li>
{% url 'staff:evaluation_edit' evaluation.id as eval_edit_id %}
{% create_breadcrumb eval_edit_id %}
{{ evaluation.name }}
{% endcreate_breadcrumb %}
{% else %}
<li class="breadcrumb-item">{{ evaluation.course.name }}</li>
{% url 'staff:evaluation_edit' evaluation.id as eval_edit_id %}
{% create_breadcrumb eval_edit_id %}
{{ evaluation.course.name }}
Comment thread
LukasMansour marked this conversation as resolved.
{% endcreate_breadcrumb %}
{% endif %}
{% endif %}
{% endblock %}
12 changes: 10 additions & 2 deletions evap/staff/templates/staff_evaluation_textanswer_edit.html
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,9 +1,17 @@
{% extends 'staff_evaluation_base.html' %}

{% load staff_templatetags %}
{% block breadcrumb %}
{{ block.super }}
<li class="breadcrumb-item"><a href="{% url 'staff:evaluation_textanswers' evaluation.id %}#{{ textanswer.id }}">{% translate 'Text answers' %}</a></li>
<li class="breadcrumb-item">{{ textanswer.id }}</li>

{% url 'staff:evaluation_textanswers' evaluation.id as staff_evaluation_textanswer_url %}
{% create_breadcrumb staff_evaluation_textanswer_url %}
{% translate 'Text answers' %}
{% endcreate_breadcrumb %}

{% create_breadcrumb request.path %}
{{ textanswer.id }}
{% endcreate_breadcrumb %}
{% endblock %}

{% block content %}
Expand Down
6 changes: 5 additions & 1 deletion evap/staff/templates/staff_evaluation_textanswers.html
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,10 +1,14 @@
{% extends 'staff_evaluation_base.html' %}

{% load evaluation_filters %}
{% load staff_templatetags %}

{% block breadcrumb %}
{{ block.super }}
<li class="breadcrumb-item">{% translate 'Text answers' %}</li>
{% url request.resolver_match.view_name as self_url %}
{% create_breadcrumb self_url %}
{% translate 'Text answers' %}
{% endcreate_breadcrumb %}
{% endblock %}

{% block content %}
Expand Down
7 changes: 6 additions & 1 deletion evap/staff/templates/staff_exam_type_index.html
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,10 +1,15 @@
{% extends 'staff_base.html' %}

{% load static %}
{% load staff_templatetags %}

{% block breadcrumb %}
{{ block.super }}
<li class="breadcrumb-item">{% translate 'Exam types' %}</li>

{% url request.resolver_match.view_name as self_url %}
{% create_breadcrumb self_url %}
{% translate 'Exam types' %}
{% endcreate_breadcrumb %}
{% endblock %}

{% block content %}
Expand Down
7 changes: 6 additions & 1 deletion evap/staff/templates/staff_faq_index.html
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,10 +1,15 @@
{% extends 'staff_base.html' %}

{% load static %}
{% load staff_templatetags %}

{% block breadcrumb %}
{{ block.super }}
<li class="breadcrumb-item">{% translate 'FAQ Sections' %}</li>

{% url request.resolver_match.view_name as self_url %}
{% create_breadcrumb self_url %}
{% translate 'FAQ Sections' %}
{% endcreate_breadcrumb %}
{% endblock %}

{% block content %}
Expand Down
12 changes: 10 additions & 2 deletions evap/staff/templates/staff_faq_section.html
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,11 +1,19 @@
{% extends 'staff_base.html' %}

{% load static %}
{% load staff_templatetags %}

{% block breadcrumb %}
{{ block.super }}
<li class="breadcrumb-item"><a href="{% url 'staff:faq_index' %}">{% translate 'FAQ Sections' %}</a></li>
<li class="breadcrumb-item">{{ section.title }}</li>

{% url 'staff:faq_index' as staff_faq_url %}
{% create_breadcrumb staff_faq_url %}
{% translate 'FAQ Sections' %}
{% endcreate_breadcrumb %}

{% create_breadcrumb %}
{{ section.title }}
{% endcreate_breadcrumb %}
{% endblock %}

{% block content %}
Expand Down
8 changes: 7 additions & 1 deletion evap/staff/templates/staff_infotexts.html
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,8 +1,14 @@
{% extends 'staff_base.html' %}

{% load staff_templatetags %}

{% block breadcrumb %}
{{ block.super }}
<li class="breadcrumb-item">{% translate 'Infotexts' %}</li>

{% url request.resolver_match.view_name as self_url %}
{% create_breadcrumb self_url %}
Comment thread
LukasMansour marked this conversation as resolved.
Outdated
{% translate 'Infotexts' %}
{% endcreate_breadcrumb %}
{% endblock %}

{% block content %}
Expand Down
7 changes: 6 additions & 1 deletion evap/staff/templates/staff_program_index.html
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,10 +1,15 @@
{% extends 'staff_base.html' %}

{% load static %}
{% load staff_templatetags %}

{% block breadcrumb %}
{{ block.super }}
<li class="breadcrumb-item">{% translate 'Programs' %}</li>

{% url request.resolver_match.view_name as self_url %}
{% create_breadcrumb self_url %}
{% translate 'Programs' %}
{% endcreate_breadcrumb %}
{% endblock %}

{% block content %}
Expand Down
12 changes: 10 additions & 2 deletions evap/staff/templates/staff_program_merge.html
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,11 +1,19 @@
{% extends 'staff_base.html' %}

{% load evaluation_filters %}
{% load staff_templatetags %}

{% block breadcrumb %}
{{ block.super }}
<li class="breadcrumb-item"><a href="{% url 'staff:program_index' %}">{% translate 'Programs' %}</a></li>
<li class="breadcrumb-item">{% translate 'Merge programs' %}</li>

{% url 'staff:program_index' as programm_index_url %}
{% create_breadcrumb programm_index_url %}
{% translate 'Programs' %}
{% endcreate_breadcrumb %}

{% create_breadcrumb %}
{% translate 'Merge programs' %}
{% endcreate_breadcrumb %}
{% endblock %}

{% block content %}
Expand Down
13 changes: 11 additions & 2 deletions evap/staff/templates/staff_program_merge_selection.html
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,9 +1,18 @@
{% extends 'staff_base.html' %}

{% load staff_templatetags %}

{% block breadcrumb %}
{{ block.super }}
<li class="breadcrumb-item"><a href="{% url 'staff:program_index' %}">{% translate 'Programs' %}</a></li>
<li class="breadcrumb-item">{% translate 'Merge programs' %}</li>

{% url 'staff:program_index' as programm_index_url %}
{% create_breadcrumb programm_index_url %}
{% translate 'Programs' %}
{% endcreate_breadcrumb %}

{% create_breadcrumb request.path %}
{% translate 'Merge programs' %}
{% endcreate_breadcrumb %}
{% endblock %}

{% block content %}
Expand Down
22 changes: 18 additions & 4 deletions evap/staff/templates/staff_questionnaire_base.html
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,16 +1,30 @@
{% extends 'staff_base.html' %}

{% load staff_templatetags %}

{% block breadcrumb %}
{{ block.super }}
{% if questionnaire %}
<li class="breadcrumb-item"><a href="{% url 'staff:questionnaire_index' %}">{% translate 'Questionnaires' %}</a></li>
{% url 'staff:questionnaire_index' as question_index_url %}
{% create_breadcrumb question_index_url %}
{% translate 'Questionnaires' %}
{% endcreate_breadcrumb %}

{% if disable_breadcrumb_questionnaire %}
<li class="breadcrumb-item">{{ questionnaire.name }}</li>
{% create_breadcrumb %}
{{ questionnaire.name }}
{% endcreate_breadcrumb %}
{% else %}
<li class="breadcrumb-item"><a href="{% url 'staff:questionnaire_edit' questionnaire.id %}">{{ questionnaire.name }}</a></li>

{% url 'staff:questionnaire_edit' questionnaire.id as question_edit_url %}
{% create_breadcrumb question_edit_url %}
{{ questionnaire.name }}
{% endcreate_breadcrumb %}
{% endif %}
{% else %}
<li class="breadcrumb-item">{% translate 'Questionnaires' %}</li>
{% create_breadcrumb %}
{% translate 'Questionnaires' %}
{% endcreate_breadcrumb %}
{% endif %}
Comment thread
LukasMansour marked this conversation as resolved.
{% endblock %}

Expand Down
7 changes: 6 additions & 1 deletion evap/staff/templates/staff_questionnaire_usage.html
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,8 +1,13 @@
{% extends 'staff_questionnaire_base.html' %}

{% load staff_templatetags %}

{% block breadcrumb %}
{{ block.super }}
<li class="breadcrumb-item">{% translate 'Usage' %}</li>

{% create_breadcrumb %}
{% translate 'Usage' %}
{% endcreate_breadcrumb %}
{% endblock %}

{% block content %}
Expand Down
6 changes: 5 additions & 1 deletion evap/staff/templates/staff_questionnaire_view.html
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,8 +1,12 @@
{% extends 'staff_questionnaire_base.html' %}

{% load staff_templatetags %}

{% block breadcrumb %}
{{ block.super }}
<li class="breadcrumb-item">{% translate 'Preview' %}</li>
{% create_breadcrumb %}
{% translate 'Preview' %}
{% endcreate_breadcrumb %}
{% endblock %}

{% block content %}
Expand Down
11 changes: 5 additions & 6 deletions evap/staff/templates/staff_semester_base.html
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
{% extends 'staff_base.html' %}
{% load staff_templatetags %}

{% block title %}
{% if semester %}
Expand All @@ -9,12 +10,10 @@

{% block breadcrumb %}
{{ block.super }}
<li class="breadcrumb-item">{% translate 'Semesters' %}</li>

@LukasMansour LukasMansour Jul 6, 2026 •
edited
Loading

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would argue that the breadcrumbs look a lot better without this singular breadcrumb. This breadcrumb links to nowhere, just makes the breadcrumb bar longer and looks ugly when something clickable appears to the right of it in the breadcrumb bar.

@richardebeling richardebeling Jul 20, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@janno42 opinions?

{% if semester %}
{% if disable_breadcrumb_semester %}
Comment thread
LukasMansour marked this conversation as resolved.
<li class="breadcrumb-item">{{ semester.name }}</li>
{% else %}
<li class="breadcrumb-item"><a href="{% url 'staff:semester_view' semester.id %}">{{ semester.name }}</a></li>
{% endif %}
{% url 'staff:semester_view' semester.id as semester_view_url %}
{% create_breadcrumb url=semester_view_url %}
Comment thread
LukasMansour marked this conversation as resolved.
Outdated
{{ semester.name }}
{% endcreate_breadcrumb %}
{% endif %}
{% endblock %}
Loading