From 7c12d094c59d041505445666474c1623b2fcd251 Mon Sep 17 00:00:00 2001
From: Jannat Patel <pateljannat2308@gmail.com>
Date: Mon, 3 Jan 2022 09:40:30 +0530
Subject: [PATCH] fix: logo dimensions

---
 school/overrides/user.py         |  4 ++--
 school/www/profiles/profile.html | 14 ++++++++------
 2 files changed, 10 insertions(+), 8 deletions(-)

diff --git a/school/overrides/user.py b/school/overrides/user.py
index c4e0bb4a..5b452e06 100644
--- a/school/overrides/user.py
+++ b/school/overrides/user.py
@@ -71,8 +71,8 @@ class CustomUser(User):
                 frappe.throw(_("Skills must be unique"))
 
     def validate_completion(self):
-        all_fields_have_value = True
         if frappe.db.get_single_value("LMS Settings", "force_profile_completion"):
+            all_fields_have_value = True
             profile_mandatory_fields = frappe.get_hooks("profile_mandatory_fields")
             docfields = frappe.get_meta(self.doctype).fields
 
@@ -81,7 +81,7 @@ class CustomUser(User):
                     all_fields_have_value = False
                     break
 
-        self.profile_complete = all_fields_have_value
+            self.profile_complete = all_fields_have_value
 
     def get_authored_courses(self) -> int:
         """Returns the number of courses authored by this user.
diff --git a/school/www/profiles/profile.html b/school/www/profiles/profile.html
index 9124e2f5..2dc8c904 100644
--- a/school/www/profiles/profile.html
+++ b/school/www/profiles/profile.html
@@ -81,20 +81,20 @@
     <div class="common-card-style overview-card small-title">
       {% if enrollment %}
       <div class="overview-item">
-        <img class="icon-background mr-1" src="/assets/school/icons/course.svg" />
-        {{ enrollment }} {% if enrollment > 1 %} Courses {% else %} Course {% endif %} Taken
+        <img class="mr-1" src="/assets/school/icons/course.svg" />
+        <span> {{ enrollment }} {% if enrollment > 1 %} Courses {% else %} Course {% endif %} Taken </span>
       </div>
       {% endif %}
       {% if reviews %}
       <div class="overview-item">
-        <img class="icon-background mr-1" src="/assets/school/icons/rating.svg" />
-        {{ reviews }} {% if reviews > 1 %} Courses {% else %} Course {% endif %} Reviewed
+        <img class="mr-1" src="/assets/school/icons/rating.svg" />
+        <span> {{ reviews }} {% if reviews > 1 %} Courses {% else %} Course {% endif %} Reviewed </span>
       </div>
       {% endif %}
       {% if mentorship %}
       <div class="overview-item">
-        <img class="icon-background mr-1" src="/assets/school/icons/calendar.svg" />
-        {{ mentorship }} {% if mentorship > 1 %} Courses {% else %} Course {% endif %} Mentored
+        <img class="mr-1" src="/assets/school/icons/calendar.svg" />
+        <span> {{ mentorship }} {% if mentorship > 1 %} Courses {% else %} Course {% endif %} Mentored </span>
       </div>
       {% endif %}
     </div>
@@ -238,6 +238,7 @@
 {% endmacro %}
 
 {% macro Contact(member) %}
+{% if not hide_primary_contact and member.email and member.mobile_no and member.linkedin and member.github and member.medium %}
 <div class="education-details">
   <div class="course-home-headings"> {{ _("Contact") }} </div>
   <div class="common-card-style overview-card">
@@ -269,6 +270,7 @@
     {% endif %}
   </div>
 </div>
+{% endif %}
 {% endmacro %}
 
 {% macro Skills(member) %}