From 02cc4ed0783063de5bfe19677fad4c22f67440c8 Mon Sep 17 00:00:00 2001
From: Aditya Unnithan <a_unnithan@hotmail.com>
Date: Fri, 6 Oct 2023 13:26:00 +0100
Subject: [PATCH] Remove redundant code

---
 src/components/card/_card.scss |  8 ++++----
 src/components/card/_macro.njk | 29 ++++++++---------------------
 2 files changed, 12 insertions(+), 25 deletions(-)

diff --git a/src/components/card/_card.scss b/src/components/card/_card.scss
index eafbf0966d..c5acdf2d26 100644
--- a/src/components/card/_card.scss
+++ b/src/components/card/_card.scss
@@ -7,11 +7,11 @@
     flex-direction: column;
     margin-bottom: 1rem;
     width: fit-content;
-  }
 
-  &__remove--margin {
-    margin-bottom: 0;
-  }
+    & > .ons-card__title {
+      margin-bottom: 0;
+    }
+  } 
 
   &__image--reorder {
     order: 1;
diff --git a/src/components/card/_macro.njk b/src/components/card/_macro.njk
index 42516328a8..e962d3b15b 100644
--- a/src/components/card/_macro.njk
+++ b/src/components/card/_macro.njk
@@ -6,36 +6,23 @@
 
   <div class="ons-card">
 
-    {%- if params.image -%}
       {%- if params.url -%}
         <a href="{{ params.url }}" class="ons-card__link">
       {%- endif -%}
-          <h{{ titleSize }} class="ons-card__title ons-card__title--reorder {{ params.titleClasses | default('ons-u-fs-m')}} ons-card__remove--margin" id="{{ params.id }}">
+          <h{{ titleSize }} class="ons-card__title ons-card__title--reorder {{ params.titleClasses | default('ons-u-fs-m')}}" id="{{ params.id }}">
               {{ params.title }}
           </h{{ titleSize }}>
-          {% if params.image.smallSrc %}
-            <img class="ons-card__image ons-u-mb-s ons-card__image--reorder" height="200" width="303"{% if params.image.largeSrc %} srcset="{{ params.image.smallSrc }} 1x, {{ params.image.largeSrc }} 2x"{% endif %} src="{{ params.image.smallSrc }}" alt="{{ params.image.alt }}" loading="lazy">
-          {% elif params.image == true or params.image.placeholderURL %}
-            <img class="ons-card__image ons-u-mb-s ons-card__image--reorder" height="100" width="303" srcset="{{ params.image.placeholderURL if params.image.placeholderURL }}/img/small/placeholder-card.png 1x, {{ params.image.placeholderURL if params.image.placeholderURL }}/img/large/placeholder-card.png 2x" src="{{ params.image.placeholderURL if params.image.placeholderURL }}/img/small/placeholder-card.png" alt="" loading="lazy">
-          {% endif %}
+          {%- if params.image -%}
+            {% if params.image.smallSrc %}
+              <img class="ons-card__image ons-u-mb-s ons-card__image--reorder" height="200" width="303"{% if params.image.largeSrc %} srcset="{{ params.image.smallSrc }} 1x, {{ params.image.largeSrc }} 2x"{% endif %} src="{{ params.image.smallSrc }}" alt="{{ params.image.alt }}" loading="lazy">
+            {% elif params.image == true or params.image.placeholderURL %}
+              <img class="ons-card__image ons-u-mb-s ons-card__image--reorder" height="100" width="303" srcset="{{ params.image.placeholderURL if params.image.placeholderURL }}/img/small/placeholder-card.png 1x, {{ params.image.placeholderURL if params.image.placeholderURL }}/img/large/placeholder-card.png 2x" src="{{ params.image.placeholderURL if params.image.placeholderURL }}/img/small/placeholder-card.png" alt="" loading="lazy">
+            {% endif %}
+          {%- endif -%}
       {%- if params.url -%}
         </a>
       {%- endif -%}
 
-    {%- else -%}
-
-      {%- if params.url -%}
-        <a class="ons-card__link" href="{{ params.url }}">
-      {%- endif -%}
-        <h{{ titleSize }} class="ons-card__title ons-card__title--reorder {{ params.titleClasses | default('ons-u-fs-m')}} ons-card__remove--margin" id="{{ params.id }}">
-          {{ params.title }}
-        </h{{ titleSize }}>
-      {%- if params.url -%}
-        </a>
-      {%- endif -%}
-
-    {%- endif -%}
-
     <p id="{{ params.textId }}">{{ params.text }}</p>
 
     {% if params.itemsList -%}