From a779e53d72ba7ccbceb4d457c9d5b2d28a09c6a4 Mon Sep 17 00:00:00 2001
From: Prasanta Kumar Dutta <7580745+pkd2512@users.noreply.github.com>
Date: Thu, 7 Sep 2023 14:11:42 +0530
Subject: [PATCH] Matt edits #116

---
 src/components/Byline/Byline.svelte           |  4 ++--
 src/components/Headline/Headline.svelte       |  2 +-
 .../SiteHeadline/SiteHeadline.svelte          |  2 +-
 src/scss/reset/_typography.scss               | 21 +++++--------------
 .../tokens/spacers/mixins/_fluid-margin.scss  |  2 +-
 src/scss/tokens/text/_text-role.scss          |  4 ++++
 src/scss/tokens/text/mixins/_text-role.scss   | 10 ++++++++-
 7 files changed, 23 insertions(+), 22 deletions(-)

diff --git a/src/components/Byline/Byline.svelte b/src/components/Byline/Byline.svelte
index 72706de7..9723c81c 100644
--- a/src/components/Byline/Byline.svelte
+++ b/src/components/Byline/Byline.svelte
@@ -75,7 +75,7 @@
         {#if authors.length > 0}
           {#each authors as author, i}
             <a
-              class="no-underline whitespace-nowrap text-primary font-medium"
+              class="no-underline whitespace-nowrap text-primary font-bold"
               href="{getAuthorPage(author)}"
               rel="author"
             >
@@ -86,7 +86,7 @@
         {:else}
           <a
             href="https://www.reuters.com"
-            class="no-underline whitespace-nowrap text-primary font-medium"
+            class="no-underline whitespace-nowrap text-primary font-bold"
             >Reuters</a
           >
         {/if}
diff --git a/src/components/Headline/Headline.svelte b/src/components/Headline/Headline.svelte
index e1c71f4b..194ffa94 100644
--- a/src/components/Headline/Headline.svelte
+++ b/src/components/Headline/Headline.svelte
@@ -85,7 +85,7 @@
       <div class="title">
         {#if section}
           <p
-            class="section-title mb-0 font-subhed text-xxs text-secondary font-bold uppercase whitespace-nowrap tracking-wider"
+            class="section-title mb-0 font-subhed text-xs text-secondary font-bold uppercase whitespace-nowrap tracking-wider"
           >
             {section}
           </p>
diff --git a/src/components/SiteHeadline/SiteHeadline.svelte b/src/components/SiteHeadline/SiteHeadline.svelte
index efc6b634..7bbb562e 100644
--- a/src/components/SiteHeadline/SiteHeadline.svelte
+++ b/src/components/SiteHeadline/SiteHeadline.svelte
@@ -75,7 +75,7 @@
     <div class="title">
       {#if section}
         <p
-          class="section-title mb-0 font-subhed text-xxs text-secondary font-bold uppercase whitespace-nowrap tracking-wider"
+          class="section-title mb-0 font-subhed text-xs text-secondary font-bold uppercase whitespace-nowrap tracking-wider"
         >
           {#if sectionUrl}
             <a class="no-underline !text-secondary" href="{sectionUrl}"
diff --git a/src/scss/reset/_typography.scss b/src/scss/reset/_typography.scss
index 7a47ce2f..54e4e919 100644
--- a/src/scss/reset/_typography.scss
+++ b/src/scss/reset/_typography.scss
@@ -123,29 +123,18 @@ figcaption {
 
 blockquote {
   @include fmx-0;
-  @include fpl-3;
-  @include fpy-3;
-  @include fpr-1;
-  border-left: 2px solid $theme-colour-brand-rules;
-  @include fmy-4;
+  @include fmy-6;
   p {
-    @include font-body;
+    @include body-note;
+    @include font-light;
     @include text-lg;
-    @include leading-tight;
-    @include fmb-1;
-    font-style: italic;
   }
   blockquote {
-    @include fmb-0;
     @include fmt-3;
-    border: 0;
-    @include fpy-0;
-    @include fpl-0;
     p {
+      @include body-caption;
       @include text-sm;
-      @include font-note;
-      @include fm-0;
-      font-style: normal;
+      @include fmb-0;
     }
   }
 }
diff --git a/src/scss/tokens/spacers/mixins/_fluid-margin.scss b/src/scss/tokens/spacers/mixins/_fluid-margin.scss
index 5c76d3f3..6d56a761 100644
--- a/src/scss/tokens/spacers/mixins/_fluid-margin.scss
+++ b/src/scss/tokens/spacers/mixins/_fluid-margin.scss
@@ -80,7 +80,7 @@ $fluid-margin-levels: (
 @mixin fml-1 {
   margin-left: map.get($fluid-margin-levels, '1');
 }
-@mixin \!fmr-1 {
+@mixin fmr-1 {
   margin-right: map.get($fluid-margin-levels, '1') !important;
 }
 @mixin \!fm-1 {
diff --git a/src/scss/tokens/text/_text-role.scss b/src/scss/tokens/text/_text-role.scss
index 74cccbdf..7624e455 100644
--- a/src/scss/tokens/text/_text-role.scss
+++ b/src/scss/tokens/text/_text-role.scss
@@ -31,3 +31,7 @@
 .body-link {
   @include body-link;
 }
+
+.drop-cap {
+  @include drop-cap;
+}
diff --git a/src/scss/tokens/text/mixins/_text-role.scss b/src/scss/tokens/text/mixins/_text-role.scss
index 230a7198..398e56f5 100644
--- a/src/scss/tokens/text/mixins/_text-role.scss
+++ b/src/scss/tokens/text/mixins/_text-role.scss
@@ -83,8 +83,16 @@
 @mixin body-link {
   color: currentColor;
   text-decoration: underline;
-  text-underline-position: under;
+  text-underline-position: from-font;
   text-decoration-style: dotted;
   text-decoration-thickness: 1px;
   text-decoration-color: var(--theme-colour-text-secondary);
 }
+
+@mixin drop-cap {
+  float: left;
+  @include text-5xl;
+  line-height: 0.9;
+  @include fmr-1;
+  @include font-light;
+}