From 24bd07d000d1e747a453bfd6ba5588fc6086922b Mon Sep 17 00:00:00 2001
From: Shivam Mishra <scm.mymail@gmail.com>
Date: Thu, 30 May 2019 16:36:58 +0530
Subject: [PATCH] refactor: style changes and template refactor

---
 erpnext/www/lms/content.html |  2 +-
 erpnext/www/lms/course.html  | 24 ++++++++++++------------
 erpnext/www/lms/program.html |  4 ++--
 3 files changed, 15 insertions(+), 15 deletions(-)

diff --git a/erpnext/www/lms/content.html b/erpnext/www/lms/content.html
index a908d85da3..68fb9f32fe 100644
--- a/erpnext/www/lms/content.html
+++ b/erpnext/www/lms/content.html
@@ -90,7 +90,7 @@
 		{% endif %}
 	</div>
 </div>
-<div>
+<div style="line-height: 1.8em;">
 	{{ content.content }}
 </div>
 {% endmacro %}
diff --git a/erpnext/www/lms/course.html b/erpnext/www/lms/course.html
index 30b594d903..ee3b9758cb 100644
--- a/erpnext/www/lms/course.html
+++ b/erpnext/www/lms/course.html
@@ -35,16 +35,16 @@
 {% macro card(topic, index, length) %}
 <div class="col-sm-{{ 12 if length%3 == 1 and index == 1 else 6 if length%3 == 2 and index in [1,2] else 4}} mb-4 text-left">
 	<div class="card h-100">
-		{% if frappe.session.user == 'Guest' %}
-			 <div>
+		{% if has_access %}
+			<a href="/lms/content?program={{ program }}&course={{ course.name }}&topic={{ topic.name }}&type={{ topic.topic_content[0].content_type }}&content={{ topic.topic_content[0].content }}" class="no-decoration no-underline">
 		{% else %}
-			<a href="/lms/content?course={{ course.name }}&topic={{ topic.name }}&type={{ topic.topic_content[0].content_type }}&content={{ topic.topic_content[0].content }}" class="no-decoration no-underline">
+			<div>
 		{% endif %}
 			{% if topic.hero_image %}
 			<div class="card-hero-img" style="background-image: url({{ topic.hero_image }})"></div>
 			{% else %}
 			<div class="card-image-wrapper text-center">
-				<div class="image-body">{{ topic.topic_name }}</div>
+				<div class="image-body"><i class="fa fa-picture-o" aria-hidden="true"></i></div>
 			</div>
 			{% endif %}
 			<div class='card-body'>
@@ -53,27 +53,27 @@
 					<ol class="list-unstyled">
 					{% for content in topic.topic_content %}
 						<li>
-							{% if frappe.session.user == 'Guest' %}
-								 <span class="text-muted">{{ content.content }}</span>
-							{% else %}
-								<a class="text-muted" href="/lms/content?course={{ course.name }}&topic={{ topic.name }}&type={{ content.content_type }}&content={{ content.content }}">
+							{% if has_access %}
+								<a class="text-muted" href="/lms/content?program={{ program }}&course={{ course.name }}&topic={{ topic.name }}&type={{ content.content_type }}&content={{ content.content }}">
 									{{ content.content }}
 								</a>
+							{% else %}
+								<span class="text-muted">{{ content.content }}</span>
 							{% endif %}
 						</li>
 					{% endfor %}
 					</ol>
 				</div>
 			</div>
-		{% if frappe.session.user == 'Guest' %}
-			</div>
-		{% else %}
+		{% if has_access %}
 			<div class='card-footer'>
 				{% if index==1 %} <span class="indicator green"> Completed </span>
 				{% else %} <span class="indicator orange"> Completed </span>
 				{% endif %}
 			</div>
 			</a>
+		{% else %}
+			</div>
 		{% endif %}
 	</div>
 </div>
@@ -81,7 +81,7 @@
 
 {% block content %}
 <section class="section ">
-	{{ hero(course.course_name, course.course_intro) }}
+	{{ hero(course.course_name, course.course_intro, has_access) }}
 	<div class='container'>
 		<div class="row mt-5">
 			{% for topic in topics %}
diff --git a/erpnext/www/lms/program.html b/erpnext/www/lms/program.html
index 65c883fb4c..d364e5e1d9 100644
--- a/erpnext/www/lms/program.html
+++ b/erpnext/www/lms/program.html
@@ -40,7 +40,7 @@
 			<div class="card-hero-img" style="background-image: url({{ course.hero_image }})"></div>
 			{% else %}
 			<div class="card-image-wrapper text-center">
-				<div class="image-body">{{ course.course_name }}</div>
+				<div class="image-body"><i class="fa fa-picture-o" aria-hidden="true"></i></div>
 			</div>
 			{% endif %}
 			<div class='card-body'>
@@ -54,7 +54,7 @@
 
 {% block content %}
 <section class="section">
-	{{ hero(program.program_name, program.description) }}
+	{{ hero(program.program_name, program.description, has_access) }}
 	<div class='container'>
 		<div class="row mt-5">
 			{% for course in program.courses %}