From f22793f91b31ad1e7c3e038a7001f5c57bc2c1cd Mon Sep 17 00:00:00 2001
From: Shivam Mishra <scm.mymail@gmail.com>
Date: Thu, 30 May 2019 16:34:53 +0530
Subject: [PATCH] refactor: added program card macro

---
 erpnext/www/lms/all-programs.html | 29 +++--------------------------
 erpnext/www/lms/index.html        | 28 +++-------------------------
 erpnext/www/lms/macros/card.html  | 22 ++++++++++++++++++++++
 3 files changed, 28 insertions(+), 51 deletions(-)
 create mode 100644 erpnext/www/lms/macros/card.html

diff --git a/erpnext/www/lms/all-programs.html b/erpnext/www/lms/all-programs.html
index f5b7595ec8..088498b4a9 100644
--- a/erpnext/www/lms/all-programs.html
+++ b/erpnext/www/lms/all-programs.html
@@ -1,5 +1,6 @@
 {% extends "templates/base.html" %}
 {% block title %}All Programs{% endblock %}
+{% from "www/lms/macros/card.html" import program_card %}
 
 {% block head_include %}
 	<style>
@@ -30,42 +31,18 @@
 	</style>
 {% endblock %}
 
-
-{% macro card(program, is_enrolled) %}
-<div class="col-sm-6 mb-4 text-left">
-	<a href="/lms/program?name={{ program.name }}" class="no-decoration no-underline">
-	<div class="card h-100">
-		{% if program.hero_image %}
-		<div class="card-hero-img" style="background-image: url({{ program.hero_image }})"></div>
-		{% else %}
-		<div class="card-image-wrapper text-center">
-			<div class="image-body">{{ program.program_name }}</div>
-		</div>
-		{% endif %}
-		<div class='card-body'>
-			<h5 class='card-title'>{{ program.program_name }}</h5>
-			<div>{{ program.description }}</div>
-		</div>
-		<div class='card-footer'>
-			<span class='small'>Watch Intro</span>
-		</div>
-	</div>
-	</a>
-</div>
-{% endmacro %}
-
 {% block content %}
 <section class="top-section ">
 	<div class='container pb-5'>
 		<h1>All Programs</h1>
 		<p class="mt-4">
-			<a class="btn btn-primary btn-lg" style="width: 170px;" href="/pricing">Start Learning</a>
+			<a class="btn btn-primary btn-lg" href="{{ '/login#signup' if frappe.session.user == 'Guest' else '/lms/all-programs' }}">{{ 'Start Learning' if frappe.session.user == 'Guest' else 'Explore Programs'}}</a>
 		</p>
 	</div>
 	<div class='container'>
 		<div class="row mt-5">
 			{% for program in all_programs %}
-				{{ card(program.program, program.is_enrolled) }}
+				{{ program_card(program) }}
 			{% endfor %}
 		</div>
 	</div>
diff --git a/erpnext/www/lms/index.html b/erpnext/www/lms/index.html
index ffa46e123c..ba3034c79e 100644
--- a/erpnext/www/lms/index.html
+++ b/erpnext/www/lms/index.html
@@ -1,5 +1,6 @@
 {% extends "templates/base.html" %}
 {% block title %}{{ education_settings.portal_title }}{% endblock %}
+{% from "www/lms/macros/card.html" import program_card %}
 
 {% block head_include %}
 	<meta name="description" content="{{ education_settings.description }}" />
@@ -32,43 +33,20 @@
 	</style>
 {% endblock %}
 
-{% macro card(program) %}
-<div class="col-sm mb-4 text-left">
-	<a href="/lms/program?name={{ program.name }}" class="no-decoration no-underline">
-	<div class="card h-100">
-		{% if program.hero_image %}
-		<div class="card-hero-img" style="background-image: url({{ program.hero_image }})"></div>
-		{% else %}
-		<div class="card-image-wrapper text-center">
-			<div class="image-body">{{ program.program_name }}</div>
-		</div>
-		{% endif %}
-		<div class='card-body'>
-			<h5 class='card-title'>{{ program.program_name }}</h5>
-			<div>{{ program.description }}</div>
-		</div>
-		<div class='card-footer'>
-			<span class='small'>Watch Intro</span>
-		</div>
-	</div>
-	</a>
-</div>
-{% endmacro %}
-
 {% block content %}
 <section class="top-section ">
 	<div class='container pb-5'>
 		<h1>{{ education_settings.portal_title }}</h1>
 		<p class='lead'>{{ education_settings.description }}</p>
 		<p class="mt-4">
-			<a class="btn btn-primary btn-lg" style="width: 170px;" href="{{ '/login#signup' if frappe.session.user == 'Guest' else '/lms/all-programs' }}">Start Learning</a>
+			<a class="btn btn-primary btn-lg" href="{{ '/login#signup' if frappe.session.user == 'Guest' else '/lms/all-programs' }}">{{ 'Start Learning' if frappe.session.user == 'Guest' else 'Explore Programs'}}</a>
 		</p>
 		<a href="https://erpnext.com" target="blank" class="mt-0 small text-muted">Go to erpnext.com</a>
 	</div>
 	<div class='container'>
 		<div class="row mt-5">
 			{% for program in featured_programs %}
-				{{ card(program.program) }}
+				{{ program_card(program) }}
 			{% endfor %}
 		</div>
 		<p class='mt-4'>
diff --git a/erpnext/www/lms/macros/card.html b/erpnext/www/lms/macros/card.html
new file mode 100644
index 0000000000..8cf8a78e56
--- /dev/null
+++ b/erpnext/www/lms/macros/card.html
@@ -0,0 +1,22 @@
+{% macro program_card(program) %}
+<div class="col-sm mb-4 text-left">
+	<a href="/lms/program?program={{ program.name }}" class="no-decoration no-underline">
+	<div class="card h-100">
+		{% if program.hero_image %}
+		<div class="card-hero-img" style="background-image: url({{ program.hero_image }})"></div>
+		{% else %}
+		<div class="card-image-wrapper text-center">
+			<div class="image-body"><i class="fa fa-picture-o" aria-hidden="true"></i></div>
+		</div>
+		{% endif %}
+		<div class='card-body'>
+			<h5 class='card-title'>{{ program.program_name }}</h5>
+			<div>{{ program.description }}</div>
+		</div>
+		<div class='card-footer'>
+			<span class='small'>Watch Intro</span>
+		</div>
+	</div>
+	</a>
+</div>
+{% endmacro %}
\ No newline at end of file