Make 'Grader is unstable' banner sticky

Users might not see the alert sometimes, since it was always at the top of the page. Fixes #1309
apluslms · Feb 1, 2024 · 2673aa9 · 2673aa9
1 parent 0ef09c8
commit 2673aa9
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/course/templates/course/course_base.html b/course/templates/course/course_base.html
@@ -75,6 +75,9 @@ <h4>{% trans "Calendar" %}</h4>
 	</div>
 	{% endblock course_sidebar %}
 	<div class="col-sm-10" id="course-content">
+		{% if instance %}
+		{% course_alert instance %}
+		{% endif %}
 		{% if instance.is_past and instance.has_enrollment_closed %}
 			<div class="alert sticky-alert alert-default">
 				{% translate "COURSE_HAS_ALREADY_ENDED" %}

diff --git a/course/templatetags/base.py b/course/templatetags/base.py
@@ -51,7 +51,7 @@ def course_alert(instance):
             _('GRADER_PROBLEMS_ALERT -- {exercises}'),
             exercises=exercises,
         )
-        return mark_safe(format_lazy('<div class="alert alert-danger">{message}</div>', message=message))
+        return mark_safe(format_lazy('<div class="alert alert-danger sticky-alert">{message}</div>', message=message))
     return ''
 
 

diff --git a/templates/base.html b/templates/base.html
@@ -235,9 +235,6 @@ <h4>{% translate "SITE" %}</h4>
 		<main class="site-content container-fluid" id="content">
 			{% endblock %}
 			{% site_alert %}
-			{% if instance %}
-			{% course_alert instance %}
-			{% endif %}
 			{% include "_messages.html" %}
 			{% block content %}
 			<div class="error">