From ad42a3d41cdd7e82e9e6066b7050db46b7e4899c Mon Sep 17 00:00:00 2001
From: Jannat Patel <pateljannat2308@gmail.com>
Date: Fri, 17 Jun 2022 18:55:26 +0530
Subject: [PATCH] fix: testimonials new design

---
 .../multiple_testimonials.html                | 59 ++++++-------------
 1 file changed, 18 insertions(+), 41 deletions(-)

diff --git a/lms/lms/web_template/multiple_testimonials/multiple_testimonials.html b/lms/lms/web_template/multiple_testimonials/multiple_testimonials.html
index f0d8fb3f..b316f910 100644
--- a/lms/lms/web_template/multiple_testimonials/multiple_testimonials.html
+++ b/lms/lms/web_template/multiple_testimonials/multiple_testimonials.html
@@ -1,50 +1,27 @@
-<div class="course-home-headings section-heading text-center" style="margin-bottom: 40px"> {{ _(title) }} </div>
-<div class="carousel slide" id="carouselExampleControls" data-ride="carousel" data-interval="false">
-  <div class="carousel-inner">
-    {% for testimonial_row in testimonials_table %}
-    {% set review = frappe.db.get_value("LMS Course Review", testimonial_row.testimonials, ["owner", "review"], as_dict=True) %}
-    {% set member = frappe.db.get_value("User", review.owner, ["name", "username", "full_name", "user_image", "headline"], as_dict=True) %}
+<h3 class="section-title text-center mb-10">{{ _(title) }}</h3>
+<div id="carouselExampleControls" class="carousel slide" data-ride="carousel" data-interval="5000">
+    <div class="carousel-inner">
 
-    {% if loop.index % 3 == 1 %}
-    <div class="carousel-item {% if loop.index == 1 %} active {% endif %}"> <div class="cards-parent">
-    {% endif %}
-      <div class="common-card-style testimonial-card">
-        <div class="testimonial-review">
-          {{ review.review }}
+        {% for testimonial_row in testimonials_table %}
+        {% set review = frappe.db.get_value("LMS Course Review", testimonial_row.testimonials, ["owner", "review"], as_dict=True) %}
+        {% set member = frappe.db.get_value("User", review.owner, ["name", "username", "full_name", "user_image", "headline"], as_dict=True) %}
+        <div class="testimonial carousel-item {% if loop.index == 1 %} active {% endif %} ">
+            {{ widgets.Avatar(member=member, avatar_class="avatar-large") }}
+            <div class="testimonial-content">
+                “{{ review.review }}”
+            </div>
+            <div class="testimonial-by">
+                {{ member.full_name }}
+            </div>
         </div>
-        <div class="d-flex align-items-center">
-          <div>
-            {{ widgets.Avatar(member=member, avatar_class="avatar-small") }}
-          </div>
-          <div class="ml-3">
-            <div class="testimonial-author-name" >{{ member.full_name }}</div>
-            {% if member.headline %}<div class="testimonial-profession">{{ member.headline }}</div> {% endif %}
-          </div>
-        </div>
-      </div>
-    {% if loop.index % 3 == 0 or loop.index == testimonials_table | length %} </div> </div> {% endif %}
-    {% endfor %}
-  </div>
-
-  {% if testimonials_table | length > 3 %}
-  <div class="slider-controls">
+        {% endfor %}
+    </div>
     <a class="carousel-control-prev" href="#carouselExampleControls" role="button" data-slide="prev">
       <span class="carousel-control-prev-icon" aria-hidden="true"></span>
-      <span class="sr-only">{{ _("Previous") }}</span>
+      <span class="sr-only">Previous</span>
     </a>
-
-    <ol class="carousel-indicators">
-      {% set number_of_slides = (testimonials_table|length / 3)|round(method="ceil")|int %}
-      {% for i in  range(number_of_slides) %}
-      <li data-target="#carouselExampleControls" data-slide-to="{{ i }}" {% if i == 0 %} class="active" {% endif %} ></li>
-      {% endfor %}
-    </ol>
-
     <a class="carousel-control-next" href="#carouselExampleControls" role="button" data-slide="next">
       <span class="carousel-control-next-icon" aria-hidden="true"></span>
-      <span class="sr-only">{{ _("Next") }}</span>
+      <span class="sr-only">Next</span>
     </a>
-  </div>
-  {% endif %}
-
 </div>