From dc3843087e6324b58a50bb4730327dd7105024dc Mon Sep 17 00:00:00 2001
From: Jannat Patel <pateljannat2308@gmail.com>
Date: Fri, 18 Feb 2022 12:26:08 +0530
Subject: [PATCH] feat: course page redesign

---
 school/hooks.py                       |  3 +-
 school/lms/utils.py                   |  5 ++-
 school/lms/widgets/CourseOutline.html | 35 ++++++++++-----
 school/plugins.py                     |  3 +-
 school/public/css/style.css           | 26 ++++++-----
 school/templates/quiz.html            |  6 +--
 school/www/batch/learn.html           | 62 +++++++++++++++++++--------
 school/www/courses/course.js          |  9 ----
 school/www/utils.py                   |  3 +-
 9 files changed, 96 insertions(+), 56 deletions(-)

diff --git a/school/hooks.py b/school/hooks.py
index 391e73a1..929dbd4f 100644
--- a/school/hooks.py
+++ b/school/hooks.py
@@ -183,7 +183,8 @@ jinja = {
         "school.lms.utils.get_reviews",
         "school.lms.utils.is_eligible_to_review",
         "school.lms.utils.get_initial_members",
-        "school.lms.utils.get_sorted_reviews"
+        "school.lms.utils.get_sorted_reviews",
+        "school.lms.utils.is_instructor"
     ],
     "filters": []
 }
diff --git a/school/lms/utils.py b/school/lms/utils.py
index 51ec23c6..9246bc5a 100644
--- a/school/lms/utils.py
+++ b/school/lms/utils.py
@@ -81,7 +81,7 @@ def get_lesson_details(chapter):
 
     for row in lesson_list:
         lesson_details = frappe.db.get_value("Course Lesson", row.lesson,
-            ["name", "title", "include_in_preview", "body"], as_dict=True)
+            ["name", "title", "include_in_preview", "body", "creation"], as_dict=True)
         lesson_details.number = flt("{}.{}".format(chapter.idx, row.idx))
         lessons.append(lesson_details)
     return lessons
@@ -289,3 +289,6 @@ def get_initial_members(course):
             member.member, ["name", "username", "full_name", "user_image"], as_dict=True))
 
     return member_details
+
+def is_instructor(course):
+    return len(list(filter(lambda x: x.name == frappe.session.user, get_instructors(course)))) > 0
diff --git a/school/lms/widgets/CourseOutline.html b/school/lms/widgets/CourseOutline.html
index ab44b730..970de9ed 100644
--- a/school/lms/widgets/CourseOutline.html
+++ b/school/lms/widgets/CourseOutline.html
@@ -41,14 +41,14 @@
           {% elif is_instructor and not lesson.include_in_preview %}
           <a class="lesson-links"
           title="This lesson is not available for preview. As you are the Instructor of the course only you can see it."
-          href="{{ course.get_lesson_url(lesson.number) }}{{course.query_parameter}}"
+          href="{{ get_lesson_url(lesson.number) }}{{course.query_parameter}}"
           data-course="{{ course.name }}">
           <img class="mr-3" src="/assets/school/icons/lock.svg">
           <div>{{ lesson.title }}</div>
           </a>
 
           {% else %}
-          <div class="no-preview" title="This lesson is not available for preview">
+          <div class="no-preview" title="This lesson is not available for preview" data-course="{{ course.name }}">
             <div class="lesson-links">
               <img class="mr-3" src="/assets/school/icons/lock.svg">
               <div>{{ lesson.title }}</div>
@@ -76,9 +76,13 @@ frappe.ready(() => {
     rotate_chapter_icon(e);
   });
 
-})
+  $(".no-preview").click((e) => {
+    show_no_preview_dialog(e);
+  });
 
-var expand_the_first_chapter = () => {
+});
+
+const expand_the_first_chapter = () => {
   var elements = $(".course-outline .collapse");
   elements.each((i, element) => {
     if (i < 1) {
@@ -86,9 +90,9 @@ var expand_the_first_chapter = () => {
       return false;
     }
   });
-}
+};
 
-var expand_the_active_chapter = () => {
+const expand_the_active_chapter = () => {
 
   /* Find anchor matching the URL for course details page */
   var selector = $(`a[href="${decodeURIComponent(window.location.pathname)}"]`).parent();
@@ -111,21 +115,30 @@ var expand_the_active_chapter = () => {
   else {
     expand_the_first_chapter();
   }
-}
+};
 
-var show_section = (element) => {
+const show_section = (element) => {
   $(element).addClass("show");
   $(element).siblings(".chapter-title").children(".chapter-icon").css("transform", "rotate(90deg)");
   $(element).siblings(".chapter-title").attr("aria-expanded", true);
-}
+};
 
-var rotate_chapter_icon = (e) => {
+const rotate_chapter_icon = (e) => {
   var icon = $(e.currentTarget).children(".chapter-icon");
   if (icon.css("transform") == "none") {
     icon.css("transform", "rotate(90deg)");
   } else {
     icon.css("transform", "none");
   }
-}
+};
+
+const show_no_preview_dialog = (e) => {
+  frappe.warn(__("Not available for preview"),
+      __("This lesson is not available for preview. Please join the course to access it."),
+      () => {
+        window.location.href = `/courses/${ $(e.currentTarget).data("course") }`
+      },
+      __("Start Learning"), false);
+};
 
 </script>
diff --git a/school/plugins.py b/school/plugins.py
index b5c72a9a..7a30eba2 100644
--- a/school/plugins.py
+++ b/school/plugins.py
@@ -114,10 +114,11 @@ def exercise_renderer(argument):
 
 def youtube_video_renderer(video_id):
     return f"""
-    <iframe width="100%" height="315"
+    <iframe width="100%" height="400"
         src="https://www.youtube.com/embed/{video_id}"
         title="YouTube video player"
         frameborder="0"
+        style="border-radius: var(--border-radius-lg)"
         allow="accelerometer; autoplay; clipboard-write; encrypted-media; gyroscope; picture-in-picture"
         allowfullscreen>
     </iframe>
diff --git a/school/public/css/style.css b/school/public/css/style.css
index 6496c0c2..194f867f 100644
--- a/school/public/css/style.css
+++ b/school/public/css/style.css
@@ -403,6 +403,8 @@ input[type=checkbox] {
   font-size: var(--text-md);
   line-height: 20px;
   box-shadow: var(--btn-shadow);
+  border: none;
+  width: fit-content;
 }
 
 .button:disabled {
@@ -435,10 +437,8 @@ input[type=checkbox] {
   color: var(--gray-700);
 }
 
-@media (max-width: 600px) {
-  .video-preview {
-    margin-top: 16px;
-  }
+.is-default:disabled {
+  color: var(--gray-500);
 }
 
 .is-primary {
@@ -561,7 +561,7 @@ input[type=checkbox] {
 }
 
 .course-content-parent .lesson-links {
-  padding: 0 0 0 1rem;
+  padding: 0.5rem 1rem;
   margin-bottom: 0.75rem;
   font-size: 0.85rem;
   line-height: 200%;
@@ -669,7 +669,7 @@ input[type=checkbox] {
   font-weight: 600;
   font-size: var(--text-3-5xl);
   letter-spacing: -0.0175em;
-  color: var(--gray-800);
+  color: var(--gray-900);
   margin-bottom: 1rem;
 }
 
@@ -859,8 +859,7 @@ input[type=checkbox] {
 }
 
 .lesson-content-card {
-  padding: 24px;
-  flex-direction: column;
+  margin-top: 2rem;
 }
 
 .lesson-content-card .alert-dismissible .close {
@@ -905,8 +904,15 @@ input[type=checkbox] {
 }
 
 .active-lesson {
-  background-color: var(--gray-100);
+  background-color: var(--blue-100);
   border-radius: var(--border-radius-md);
+  color: var(--blue-500);
+}
+
+.lesson-title {
+  display: flex;
+  align-items: center;
+  margin-bottom: 1rem;
 }
 
 .lesson-progress {
@@ -914,7 +920,7 @@ input[type=checkbox] {
   padding: 4px 8px 4px;
   font-size: 10px;
   line-height: 120%;
-  margin: 0px 10px 20px;
+  margin-left: 1rem;
   border-radius: var(--border-radius-md);
   font-weight: bold;
 }
diff --git a/school/templates/quiz.html b/school/templates/quiz.html
index 4a4f3325..640f0569 100644
--- a/school/templates/quiz.html
+++ b/school/templates/quiz.html
@@ -47,9 +47,9 @@
     </div>
     <div class="quiz-footer">
       <span class="font-weight-bold"> <span class="current-question">1</span> of {{ quiz.questions | length }}</span>
-      <button class="button pull-right" id="check" disabled>Check</button>
-      <button class="button hide" id="next">Next Question</button>
-      <button class="button is-default hide" id="summary">Summary</button>
+      <button class="button pull-right is-default" id="check" disabled>Check</button>
+      <div class="button is-secondary hide" id="next">Next Question</div>
+      <div class="button is-secondary is-default hide" id="summary">Summary</div>
       <small id="submission-message" class="font-weight-bold hide"> Please join the course to submit the Quiz.</small>
     </div>
     <div class="button is-secondary pull-right hide" id="try-again">Try Again</div>
diff --git a/school/www/batch/learn.html b/school/www/batch/learn.html
index 959a9c3d..444e6676 100644
--- a/school/www/batch/learn.html
+++ b/school/www/batch/learn.html
@@ -33,34 +33,60 @@
 {% endblock %}
 
 {% macro LessonContent(lesson) %}
-{% set is_instructor = frappe.session.user == course.instructor %}
+{% set instructors = get_instructors(course.name) %}
+{% set is_instructor = is_instructor(course.name) %}
 <div class="lesson-content">
-  <div class="course-home-headings title
-  {% if membership %} is-member {% endif %}
-  {% if membership or is_instructor %} eligible-for-submission {% endif %}" data-lesson="{{ lesson.name }}"
-    data-course="{{ course.name }}">
-    {{ lesson.title }}
+  <div class="lesson-title">
+    <div class="course-home-headings title mb-0
+    {% if membership %} is-member {% endif %}
+    {% if membership or is_instructor %} eligible-for-submission {% endif %}" data-lesson="{{ lesson.name }}"
+      data-course="{{ course.name }}">{{ lesson.title }}</div>
     <span class="lesson-progress {{hide if get_progress(course.name, lesson.name) != 'Complete' else ''}}">COMPLETED</span>
   </div>
 
+  <div class="d-flex align-items-center">
+    {% set instructors = instructors %}
+    {% set ins_len = instructors | length %}
+    {% for instructor in instructors %}
+    {% if ins_len > 1 and loop.index == 1 %}
+    <div class="avatar-group overlap">
+    {% endif %}
+    {{ widgets.Avatar(member=instructor, avatar_class="avatar-small") }}
 
+    {% if ins_len > 1 and loop.index == ins_len %}
+    </div>
+    {% endif %}
+    {% endfor %}
+    <a class="button-links ml-2" href="{{ get_profile_url(instructors[0].username) }}">
+      <span class="course-creator-name">
+      {% if ins_len == 1 %}
+        {{ instructors[0].full_name }}
+      {% else %}
+        {% set suffix = "other" if ins_len - 1 == 1 else "others"  %}
+        {{ instructors[0].full_name.split(" ")[0] }} and {{ ins_len - 1 }} {{ suffix }}
+      {% endif %}
+      </span>
+    </a>
+    <div class="ml-3 course-meta"> {{ frappe.utils.format_date(lesson.creation, "medium") }} </div>
+  </div>
+
+
+  <div class="markdown-source lesson-content-card">
   {% if membership or lesson.include_in_preview or is_instructor %}
-  <div class="common-card-style lesson-content-card markdown-source">
-    {% if is_instructor and not lesson.include_in_preview %}
-    <small class="alert alert-secondary alert-dismissible">
+  {% if is_instructor and not lesson.include_in_preview %}
+    <div class="small alert alert-secondary alert-dismissible mt-4 mb-4">
       This lesson is not available for preview. As you are the Instructor of the course only you can see it.
       <a href="#" class="close" data-dismiss="alert" aria-label="close">&times;</a>
-    </small>
-    {% endif %}
-    {{ render_html(lesson.body) }}</div>
-  {% else %}
-  <div class="common-card-style lesson-content-card">
-    <div class="w-25 text-center" style="margin: 0 auto;">
-      <small>This lesson is not available for preview. Please join the course to access it.</small>
-      <a class="button is-primary ml-auto mr-auto mt-3" href="/courses/{{ course.name }}"> Start Learning </a>
     </div>
-  </div>
   {% endif %}
+    {{ render_html(lesson.body) }}
+  {% else %}
+    <div class="">
+      <a class="button is-primary pull-right" href="/courses/{{ course.name }}"> Start Learning </a>
+      <div class="">This lesson is not available for preview. Please join the course to access it.</div>
+    </div>
+  {% endif %}
+  </div>
 
 </div>
 {% endmacro %}
diff --git a/school/www/courses/course.js b/school/www/courses/course.js
index f59c3552..e5f8c8ab 100644
--- a/school/www/courses/course.js
+++ b/school/www/courses/course.js
@@ -21,10 +21,6 @@ frappe.ready(() => {
     view_all_mentors(e);
   });
 
-  $(".video-preview").click((e) => {
-    show_video_dialog(e);
-  });
-
   $(".review-link").click((e) => {
     show_review_dialog(e);
   });
@@ -158,11 +154,6 @@ var view_all_mentors = (e) => {
   }
 }
 
-var show_video_dialog = (e) => {
-  e.preventDefault();
-  $("#video-modal").modal("show");
-}
-
 var show_review_dialog = (e) => {
   e.preventDefault();
   $("#review-modal").modal("show");
diff --git a/school/www/utils.py b/school/www/utils.py
index 21fa2676..9cb00bba 100644
--- a/school/www/utils.py
+++ b/school/www/utils.py
@@ -1,6 +1,5 @@
 import frappe
-from school.lms.utils import slugify, get_membership, get_lessons, get_batch
-
+from school.lms.utils import slugify, get_membership, get_lessons, get_batch, get_lesson_url
 def get_common_context(context):
     context.no_cache = 1