From 87a7438d3821351a15ddee97a51d8a677e9f2d0a Mon Sep 17 00:00:00 2001
From: Shivam Mishra <scm.mymail@gmail.com>
Date: Tue, 21 May 2019 12:05:19 +0530
Subject: [PATCH] chore: added macro for page hero

---
 erpnext/www/lms/course.html      | 24 ++++++++++++++----------
 erpnext/www/lms/macros/hero.html | 20 ++++++++++++++++++++
 erpnext/www/lms/program.html     | 13 ++++---------
 3 files changed, 38 insertions(+), 19 deletions(-)
 create mode 100644 erpnext/www/lms/macros/hero.html

diff --git a/erpnext/www/lms/course.html b/erpnext/www/lms/course.html
index 2e414a03695..68128dca1d9 100644
--- a/erpnext/www/lms/course.html
+++ b/erpnext/www/lms/course.html
@@ -1,5 +1,6 @@
 {% extends "templates/base.html" %}
 {% block title %}{{ course.course_name }}{% endblock %}
+{% from "www/lms/macros/hero.html" import hero %}
 
 {% block head_include %}
 	<style>
@@ -34,7 +35,7 @@
 {% 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">
-		<a href="/lms/content?topic={{ topic.name }}&type=Content" class="no-decoration no-underline">
+		<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">
 			{% if topic.hero_image %}
 			<div class="card-hero-img" style="background-image: url({{ topic.hero_image }})"></div>
 			{% else %}
@@ -47,25 +48,28 @@
 				<div>
 					<ol class="list-unstyled">
 					{% for content in topic.topic_content %}
-						<li><a class="text-muted" href="/lms/content?topic={{ topic.name }}&type={{ content.content_type }}&content={{ content.content }}">{{ content.content }}</li></a>
+						<li><a class="text-muted" href="/lms/content?course={{ course.name }}&topic={{ topic.name }}&type={{ content.content_type }}&content={{ content.content }}">{{ content.content }}</li></a>
 					{% endfor %}
 					</ol>
 				</div>
 			</div>
+			{% if index==1 %}
+			<div class='card-footer'>
+				<i class="fa fa-check-circle text-success"></i> Completed
+			</div>
+			{% else %}
+			<div class='card-footer'>
+				<i class="fa fa-circle-o text-warning"></i> In Progress
+			</div>
+			{% endif %}
 		</a>
 	</div>
 </div>
 {% endmacro %}
 
 {% block content %}
-<section class="top-section ">
-	<div class='container pb-5'>
-		<h1>{{ course.course_name }}</h1>
-		<p class='lead'>{{ course.course_intro }}</p>
-		<p class="mt-4">
-			<a class="btn btn-primary btn-lg" style="width: 170px;" href="/login#signup">Sign Up</a>
-		</p>
-	</div>
+<section class="section ">
+	{{ hero(course.course_name, course.course_intro) }}
 	<div class='container'>
 		<div class="row mt-5">
 			{% for topic in topics %}
diff --git a/erpnext/www/lms/macros/hero.html b/erpnext/www/lms/macros/hero.html
new file mode 100644
index 00000000000..080221b0e75
--- /dev/null
+++ b/erpnext/www/lms/macros/hero.html
@@ -0,0 +1,20 @@
+{% macro hero(title, description) %}
+	<div class='container pb-5'>
+		<h1>{{ title}}</h1>
+		<p class='lead' style="max-width: 100%;">{{ description }}</p>
+		<p class="mt-4">
+			<a id="signup" class="btn btn-primary btn-lg" style="width: 170px;" href="/login#signup" hidden=true>Sign Up</a>
+		</p>
+	</div>
+
+{% block script %}
+<script type="text/javascript">
+	frappe.ready(() => {
+		btn = document.getElementById('signup')
+		if(frappe.session.user == 'Guest') {
+			btn.hidden = false;
+		}
+	})
+</script>
+{% endblock %}
+{% endmacro %}
\ No newline at end of file
diff --git a/erpnext/www/lms/program.html b/erpnext/www/lms/program.html
index 35f7f8a69c0..65c883fb4c6 100644
--- a/erpnext/www/lms/program.html
+++ b/erpnext/www/lms/program.html
@@ -1,5 +1,6 @@
 {% extends "templates/base.html" %}
 {% block title %}{{ program.program_name }}{% endblock %}
+{% from "www/lms/macros/hero.html" import hero %}
 
 {% block head_include %}
 	<style>
@@ -34,7 +35,7 @@
 {% macro card(course, 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">
-		<a href="/lms/course?name={{ course.name }}" class="no-decoration no-underline">
+		<a href="/lms/course?name={{ course.name }}&program={{ program.name }}" class="no-decoration no-underline">
 			{% if course.hero_image %}
 			<div class="card-hero-img" style="background-image: url({{ course.hero_image }})"></div>
 			{% else %}
@@ -52,14 +53,8 @@
 {% endmacro %}
 
 {% block content %}
-<section class="top-section ">
-	<div class='container pb-5'>
-		<h1>{{ program.program_name }}</h1>
-		<p class='lead'>{{ program.description }}</p>
-		<p class="mt-4">
-			<a class="btn btn-primary btn-lg" style="width: 170px;" href="/login#signup">Sign Up</a>
-		</p>
-	</div>
+<section class="section">
+	{{ hero(program.program_name, program.description) }}
 	<div class='container'>
 		<div class="row mt-5">
 			{% for course in program.courses %}