fix: icon background and space cleanup

school/www/batch/learn.html

@@ -15,7 +15,7 @@
 {% block content %}
 <div class="common-page-style">
   <div class="container course-details-page">
-    {{ widgets.BreadCrumb(course=course, lesson=lesson) }}
+    {{ BreadCrumb(course, lesson) }}
     <div class="course-content-parent">
       <div class="course-details-outline">
         {{ widgets.CourseOutline(course=course, membership=membership) }}
@@ -32,6 +32,16 @@
 </div>
 {% endblock %}
 
+{% macro BreadCrumb(course, lesson) %}
+<div class="breadcrumb">
+  <a class="dark-links" href="/courses">{{ _("All Courses") }}</a>
+  <img class="ml-1 mr-1" src="/assets/school/icons/chevron-right.svg">
+  <a class="dark-links" href="/courses">{{ course.title }}</a>
+  <img class="ml-1 mr-1" src="/assets/school/icons/chevron-right.svg">
+  <span class="breadcrumb-destination">{{ lesson.title }}</span>
+</div>
+{% endmacro %}
+
 {% macro LessonContent(lesson) %}
 {% set instructors = get_instructors(course.name) %}
 {% set is_instructor = is_instructor(course.name) %}
@@ -57,8 +67,8 @@
     </div>
     {% endif %}
     {% endfor %}
-    <a class="button-links ml-2" href="{{ get_profile_url(instructors[0].username) }}">
-      <span class="course-creator-name">
+    <a class="button-links ml-1" href="{{ get_profile_url(instructors[0].username) }}">
+      <span class="course-meta">
       {% if ins_len == 1 %}
         {{ instructors[0].full_name }}
       {% else %}

school/www/courses/course.html

@@ -6,15 +6,17 @@
 <link rel="stylesheet" href="/assets/frappe/css/font-awesome.css">
 {% endblock %}
 {% block content %}
-<div class="common-page-style pt-0">
+<div class="common-page-style pt-0 pb-0">
   <div class="course-home-top-container">
     {{ CourseHomeHeader(course) }}
-    <div class="container course-home-page">
-      <div class="course-body-container">
-        {{ CourseHeaderOverlay(course) }}
-        {{ Description(course) }}
-        {{ widgets.CourseOutline(course=course, membership=membership) }}
-        {{ widgets.Reviews(course=course, membership=membership) }}
+    <div class="course-home-page">
+      <div class="container">
+        <div class="course-body-container">
+          {{ CourseHeaderOverlay(course) }}
+          {{ Description(course) }}
+          {{ widgets.CourseOutline(course=course, membership=membership) }}
+          {{ widgets.Reviews(course=course, membership=membership) }}
+        </div>
       </div>
     </div>
   </div>
@@ -65,25 +67,21 @@
   <div class="course-intructor-rating-section">
 
     <div class="vertically-center">
-      <svg class="icon icon-md mr-1">
+      <svg class="icon icon-md">
         <use class="" href="#icon-users">
       </svg>
       {{ get_students(course.name) | length }} {{ _("Enrolled") }}
     </div>
 
-    <span class="seperator"></span>
-
     {% if get_lessons(course.name) | length %}
     <div class="vertically-center">
-      <svg class="icon icon-md mr-1">
+      <svg class="icon icon-md">
         <use href="#icon-education"></use>
       </svg>
       {{ get_lessons(course.name) | length }} {{ _("Lessons") }}
     </div>
     {% endif %}
 
-    <span class="seperator"></span>
-
     {% set avg_rating = get_average_rating(course.name) %}
     {% if avg_rating %}
     <div class="rating mr-2">
@@ -92,8 +90,8 @@
         <use href="#icon-star"></use>
       </svg>
       {% endfor %}
+      <span> {{ avg_rating }} {{ _(" Rating ") }} </span>
     </div>
-    <span> {{ avg_rating }} {{ _(" Rating ") }} </span>
     {% endif %}
 
   </div>